[Expired for neutron because there has been no activity for 60 days.]

** Changed in: neutron
       Status: Incomplete => Expired

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1509941

Title:
  case test_kill_process_with_different_signal is not necessary

Status in neutron:
  Expired

Bug description:
  in _test__kill_process, utils.execute has been mocked in following code:
          ...
          with mock.patch.object(utils, 'execute',
                                 side_effect=exc) as mock_execute:
          ...
  while in _kill_process, signal will be used in:
      ...
      utils.execute(['kill', '-%d' % kill_signal, pid],
                     run_as_root=self.run_as_root)
      ...
  so indeed, no signal will be sent at all, for that, we don't need case 
test_kill_process_with_different_signal, and don't need signal as parameter for 
_test__kill_process

To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/1509941/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp

Reply via email to