From b7ecfcff55e1fb2ea1c085e10449832dc6770a93 Mon Sep 17 00:00:00 2001 From: Lubomír Sedlář Date: Jun 08 2016 08:33:51 +0000 Subject: Fix tests for retire with new rpkg The `push` method gained a `force` argument, which will be passed in new rpkg, but missing in old ones. This patch changes the assert to verify that push was called, but not specific arguments (which are not interesting anyway). Signed-off-by: Lubomír Sedlář --- diff --git a/test/test_retire.py b/test/test_retire.py index e80b94e..9da76cb 100644 --- a/test/test_retire.py +++ b/test/test_retire.py @@ -80,7 +80,7 @@ class RetireTestCase(unittest.TestCase): client.retire() self.assertRetired('my reason') - self.assertEqual(client.cmd.push.call_args_list, [mock.call()]) + self.assertEqual(len(client.cmd.push.call_args_list), 1) self.assertEqual(PkgDB.return_value.retire_packages.call_args_list, [mock.call('fedpkg', 'master', namespace='rpms')]) @@ -96,6 +96,6 @@ class RetireTestCase(unittest.TestCase): client.retire() self.assertRetired('my reason') - self.assertEqual(client.cmd.push.call_args_list, [mock.call()]) + self.assertEqual(len(client.cmd.push.call_args_list), 1) self.assertEqual(PkgDB.return_value.retire_packages.call_args_list, [mock.call('fedpkg', 'master', namespace='rpms')])