case test_kill_process_with_different_signal is not necessary

[ZongKai LI]

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

[Armando Migliaccio (armando-migliaccio)]

Without actual links to the files it's difficult to see what you mean but from what *I* can see, _kill_process does use "execute" and that does need to be mocked.

[Launchpad Janitor (janitor)]

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