diff --git a/tests/test_cli/test_add_channel.py b/tests/test_cli/test_add_channel.py index 99f1ea17..0445e3d7 100644 --- a/tests/test_cli/test_add_channel.py +++ b/tests/test_cli/test_add_channel.py @@ -26,6 +26,9 @@ class TestAddChannel(utils.CliTestCase): %s: error: {message} """ % (self.progname, self.progname) + def tearDown(self): + mock.patch.stopall() + @mock.patch('sys.stdout', new_callable=six.StringIO) def test_handle_add_channel(self, stdout): self.session.addChannel.return_value = self.channel_id diff --git a/tests/test_cli/test_add_external_repo.py b/tests/test_cli/test_add_external_repo.py index afcccc6d..24cabb40 100644 --- a/tests/test_cli/test_add_external_repo.py +++ b/tests/test_cli/test_add_external_repo.py @@ -25,6 +25,9 @@ class TestAddExternalRepo(utils.CliTestCase): self.tag = 'test-tag' self.priority = 10 + def tearDown(self): + mock.patch.stopall() + def test_add_external_repo_invalid_mode(self): mode = 'test-mode' arguments = ['--mode', mode] diff --git a/tests/test_cli/test_add_group.py b/tests/test_cli/test_add_group.py index 396f1725..97622efb 100644 --- a/tests/test_cli/test_add_group.py +++ b/tests/test_cli/test_add_group.py @@ -21,6 +21,9 @@ class TestAddGroup(utils.CliTestCase): %s: error: {message} """ % (self.progname, self.progname) + def tearDown(self): + mock.patch.stopall() + @mock.patch('sys.stdout', new_callable=six.StringIO) def test_handle_add_group(self, stdout): tag = 'tag' diff --git a/tests/test_cli/test_add_host.py b/tests/test_cli/test_add_host.py index 7a71469f..12615b99 100644 --- a/tests/test_cli/test_add_host.py +++ b/tests/test_cli/test_add_host.py @@ -19,6 +19,9 @@ class TestAddHost(utils.CliTestCase): %s: error: {message} """ % (self.progname, self.progname) + def tearDown(self): + mock.patch.stopall() + @mock.patch('sys.stdout', new_callable=six.StringIO) def test_handle_add_host(self, stdout): host = 'host' diff --git a/tests/test_cli/test_add_host_to_channel.py b/tests/test_cli/test_add_host_to_channel.py index eb4cb2a3..9dce6672 100644 --- a/tests/test_cli/test_add_host_to_channel.py +++ b/tests/test_cli/test_add_host_to_channel.py @@ -19,6 +19,9 @@ class TestAddHostToChannel(utils.CliTestCase): %s: error: {message} """ % (self.progname, self.progname) + def tearDown(self): + mock.patch.stopall() + @mock.patch('sys.stdout', new_callable=six.StringIO) def test_handle_add_host_to_channel(self, stdout): host = 'host' diff --git a/tests/test_cli/test_add_notification.py b/tests/test_cli/test_add_notification.py index ff3c5ee0..6e85bf4e 100644 --- a/tests/test_cli/test_add_notification.py +++ b/tests/test_cli/test_add_notification.py @@ -21,6 +21,9 @@ class TestAddNotification(utils.CliTestCase): %s: error: {message} """ % (self.progname, self.progname) + def tearDown(self): + mock.patch.stopall() + def test_handle_add_notification(self): self.session.getPackageID.return_value = 1234 self.session.getTagID.return_value = 4321 diff --git a/tests/test_cli/test_add_pkg.py b/tests/test_cli/test_add_pkg.py index b876fa1f..50739da8 100644 --- a/tests/test_cli/test_add_pkg.py +++ b/tests/test_cli/test_add_pkg.py @@ -27,6 +27,9 @@ class TestAddPkg(utils.CliTestCase): %s: error: {message} """ % (self.progname, self.progname) + def tearDown(self): + mock.patch.stopall() + @mock.patch('sys.stdout', new_callable=six.StringIO) def test_handle_add_pkg(self, stdout): tag = 'tag' diff --git a/tests/test_cli/test_add_tag.py b/tests/test_cli/test_add_tag.py index c5d8640a..5b7b2dd7 100644 --- a/tests/test_cli/test_add_tag.py +++ b/tests/test_cli/test_add_tag.py @@ -21,6 +21,9 @@ class TestAddTag(utils.CliTestCase): %s: error: {message} """ % (self.progname, self.progname) + def tearDown(self): + mock.patch.stopall() + def test_handle_add_tag(self): """Test handle_add_tag function""" # Case 1. no argument error diff --git a/tests/test_cli/test_add_tag_inheritance.py b/tests/test_cli/test_add_tag_inheritance.py index fe5cd325..892be393 100644 --- a/tests/test_cli/test_add_tag_inheritance.py +++ b/tests/test_cli/test_add_tag_inheritance.py @@ -56,6 +56,9 @@ class TestAddTagInheritance(utils.CliTestCase): 'perm': None, 'perm_id': None} + def tearDown(self): + mock.patch.stopall() + def test_add_tag_inheritance_without_option(self): arguments = [] expected = self.format_error_message( diff --git a/tests/test_cli/test_add_target.py b/tests/test_cli/test_add_target.py index a453cdc2..78aa9249 100644 --- a/tests/test_cli/test_add_target.py +++ b/tests/test_cli/test_add_target.py @@ -20,6 +20,9 @@ class TestAddTarget(utils.CliTestCase): %s: error: {message} """ % (self.progname, self.progname) + def tearDown(self): + mock.patch.stopall() + def test_add_target_without_option(self,): expected = self.format_error_message( "Please specify a target name, a build tag, and destination tag") diff --git a/tests/test_cli/test_add_user.py b/tests/test_cli/test_add_user.py index a7099d76..afea994f 100644 --- a/tests/test_cli/test_add_user.py +++ b/tests/test_cli/test_add_user.py @@ -25,6 +25,9 @@ class TestAddUser(utils.CliTestCase): %s: error: {message} """ % (self.progname, self.progname) + def tearDown(self): + mock.patch.stopall() + @mock.patch('sys.stdout', new_callable=six.StringIO) def test_handle_add_user(self, stdout): """Test handle_add_user function""" diff --git a/tests/test_cli/test_add_volume.py b/tests/test_cli/test_add_volume.py index 5fa8436a..ca4f7406 100644 --- a/tests/test_cli/test_add_volume.py +++ b/tests/test_cli/test_add_volume.py @@ -24,6 +24,9 @@ class TestAddVolume(utils.CliTestCase): %s: error: {message} """ % (self.progname, self.progname) + def tearDown(self): + mock.patch.stopall() + @mock.patch('sys.stdout', new_callable=six.StringIO) def test_handle_add_volume(self, stdout): """Test handle_add_volume function""" diff --git a/tests/test_cli/test_assign_task.py b/tests/test_cli/test_assign_task.py index ed38f485..ed9f62c3 100644 --- a/tests/test_cli/test_assign_task.py +++ b/tests/test_cli/test_assign_task.py @@ -24,6 +24,9 @@ class TestAssignTask(utils.CliTestCase): %s: error: {message} """ % (self.progname, self.progname) + def tearDown(self): + mock.patch.stopall() + @mock.patch('sys.stdout', new_callable=six.StringIO) def test_handle_assign_task(self, stdout): hostname = "host" diff --git a/tests/test_cli/test_block_group.py b/tests/test_cli/test_block_group.py index 51345a57..f483108c 100644 --- a/tests/test_cli/test_block_group.py +++ b/tests/test_cli/test_block_group.py @@ -25,6 +25,9 @@ class TestBlockGroup(utils.CliTestCase): %s: error: {message} """ % (self.progname, self.progname) + def tearDown(self): + mock.patch.stopall() + def test_handle_block_group_nonexistent_tag(self): tag = 'nonexistent-tag' group = 'group' diff --git a/tests/test_cli/test_block_notification.py b/tests/test_cli/test_block_notification.py index a1107a5f..25c1ca46 100644 --- a/tests/test_cli/test_block_notification.py +++ b/tests/test_cli/test_block_notification.py @@ -21,6 +21,9 @@ class TestBlockNotification(utils.CliTestCase): %s: error: {message} """ % (self.progname, self.progname) + def tearDown(self): + mock.patch.stopall() + def test_handle_block_notification_non_exist_tag(self): tag = 'test-tag' arguments = ['--tag', tag] diff --git a/tests/test_cli/test_block_pkg.py b/tests/test_cli/test_block_pkg.py index f637adb5..750b8237 100644 --- a/tests/test_cli/test_block_pkg.py +++ b/tests/test_cli/test_block_pkg.py @@ -26,6 +26,9 @@ class TestBlockPkg(utils.CliTestCase): %s: error: {message} """ % (self.progname, self.progname) + def tearDown(self): + mock.patch.stopall() + @mock.patch('sys.stdout', new_callable=six.StringIO) def test_handle_block_pkg(self, stdout): tag = 'tag' diff --git a/tests/test_cli/test_build.py b/tests/test_cli/test_build.py index b66d391a..215a577d 100644 --- a/tests/test_cli/test_build.py +++ b/tests/test_cli/test_build.py @@ -48,6 +48,9 @@ https://docs.pagure.org/koji/HOWTO/#package-organization %s: error: {message} """ % (self.progname, self.progname, self.progname) + def tearDown(self): + mock.patch.stopall() + @mock.patch('sys.stdout', new_callable=six.StringIO) def test_handle_build_from_srpm(self, stdout): args = [self.target, self.source_srpm] diff --git a/tests/test_cli/test_call.py b/tests/test_cli/test_call.py index 727fefc2..378849be 100644 --- a/tests/test_cli/test_call.py +++ b/tests/test_cli/test_call.py @@ -29,6 +29,9 @@ Note, that you can use global option --noauth for anonymous calls here %s: error: {message} """ % (self.progname, self.progname) + def tearDown(self): + mock.patch.stopall() + @mock.patch('sys.stdout', new_callable=six.StringIO) def test_handle_call(self, stdout): """Test handle_call function""" diff --git a/tests/test_cli/test_cancel.py b/tests/test_cli/test_cancel.py index 7a8a1c60..580165c5 100644 --- a/tests/test_cli/test_cancel.py +++ b/tests/test_cli/test_cancel.py @@ -35,6 +35,9 @@ class TestCancel(utils.CliTestCase): self.session.hub_version = (1, 33, 0) self.session.hub_version_str = '1.33.0' + def tearDown(self): + mock.patch.stopall() + def test_anon_cancel(self): args = ['123'] self.activate_session_mock.side_effect = koji.GenericError diff --git a/tests/test_cli/test_chain_build.py b/tests/test_cli/test_chain_build.py index 82383668..f5dd01b5 100644 --- a/tests/test_cli/test_chain_build.py +++ b/tests/test_cli/test_chain_build.py @@ -31,6 +31,9 @@ class TestChainBuild(utils.CliTestCase): %s: error: {message} """ % (self.progname, self.progname) + def tearDown(self): + mock.patch.stopall() + @mock.patch('sys.stdout', new_callable=six.StringIO) def test_handle_chain_build(self, stdout): target = 'target' diff --git a/tests/test_cli/test_disable_channel.py b/tests/test_cli/test_disable_channel.py index a0aeaf79..03b3d961 100644 --- a/tests/test_cli/test_disable_channel.py +++ b/tests/test_cli/test_disable_channel.py @@ -29,6 +29,9 @@ class TestDisableChannel(utils.CliTestCase): 'name': 'test-channel'} ] + def tearDown(self): + mock.patch.stopall() + def __vm(self, result): m = koji.VirtualCall('mcall_method', [], {}) if isinstance(result, dict) and result.get('faultCode'): diff --git a/tests/test_cli/test_disable_host.py b/tests/test_cli/test_disable_host.py index 1dbc154e..a101c29c 100644 --- a/tests/test_cli/test_disable_host.py +++ b/tests/test_cli/test_disable_host.py @@ -21,6 +21,9 @@ class TestDisableHost(utils.CliTestCase): %s: error: {message} """ % (self.progname, self.progname) + def tearDown(self): + mock.patch.stopall() + def test_handle_disable_host_no_such_host(self): """Test %s function""" % handle_disable_host.__name__ self.session.getHost.return_value = None diff --git a/tests/test_cli/test_disable_user.py b/tests/test_cli/test_disable_user.py index ff54eecb..7e21d22a 100644 --- a/tests/test_cli/test_disable_user.py +++ b/tests/test_cli/test_disable_user.py @@ -24,6 +24,9 @@ class TestDisableUser(utils.CliTestCase): %s: error: {message} """ % (self.progname, self.progname) + def tearDown(self): + mock.patch.stopall() + def test_handle_disable_user_no_argument(self): """Test handle_disable_user function""" expected = self.format_error_message( diff --git a/tests/test_cli/test_edit_external_repo.py b/tests/test_cli/test_edit_external_repo.py index 10a8f1e2..8cd2210b 100644 --- a/tests/test_cli/test_edit_external_repo.py +++ b/tests/test_cli/test_edit_external_repo.py @@ -24,6 +24,9 @@ class TestEditExternalRepo(utils.CliTestCase): %s: error: {message} """ % (self.progname, self.progname) + def tearDown(self): + mock.patch.stopall() + def test_handle_edit_external_repo_error(self): """Test handle_edit_external_repo function""" # [(expected, args),...] diff --git a/tests/test_cli/test_edit_host.py b/tests/test_cli/test_edit_host.py index b7cca310..7f6cf681 100644 --- a/tests/test_cli/test_edit_host.py +++ b/tests/test_cli/test_edit_host.py @@ -28,6 +28,9 @@ class TestEditHost(utils.CliTestCase): self.description = 'description' self.comment = 'comment' + def tearDown(self): + mock.patch.stopall() + @mock.patch('sys.stdout', new_callable=six.StringIO) def test_handle_edit_host(self, stdout): host_info = mock.ANY diff --git a/tests/test_cli/test_edit_notification.py b/tests/test_cli/test_edit_notification.py index 889335d3..09954dc6 100644 --- a/tests/test_cli/test_edit_notification.py +++ b/tests/test_cli/test_edit_notification.py @@ -19,6 +19,9 @@ class TestEditNotification(utils.CliTestCase): %s: error: {message} """ % (self.progname, self.progname) + def tearDown(self): + mock.patch.stopall() + def test_handle_edit_notification(self): self.session.getPackageID.return_value = 1234 self.session.getTagID.return_value = 4321 diff --git a/tests/test_cli/test_edit_permission.py b/tests/test_cli/test_edit_permission.py index 0d9a1042..c03462f1 100644 --- a/tests/test_cli/test_edit_permission.py +++ b/tests/test_cli/test_edit_permission.py @@ -26,6 +26,9 @@ class TestEditPermission(utils.CliTestCase): self.perm = 'test-perm' self.description = 'test-description' + def tearDown(self): + mock.patch.stopall() + def test_handle_edit_permission_argument_error(self): expected = self.format_error_message("Please specify a permission and a description") for args in [[], [self.perm]]: diff --git a/tests/test_cli/test_edit_tag.py b/tests/test_cli/test_edit_tag.py index 19e302d4..ae6661f5 100644 --- a/tests/test_cli/test_edit_tag.py +++ b/tests/test_cli/test_edit_tag.py @@ -30,6 +30,9 @@ class TestEditTag(utils.CliTestCase): self.extra = {'extraA': 'A', 'extraB': True} self.remove_extra = ['extraC', 'extraD'] + def tearDown(self): + mock.patch.stopall() + @mock.patch('sys.stdout', new_callable=six.StringIO) def test_handle_edit_tag_1(self, stdout): args = [self.tag] diff --git a/tests/test_cli/test_edit_target.py b/tests/test_cli/test_edit_target.py index 34a449d7..57a6c53a 100644 --- a/tests/test_cli/test_edit_target.py +++ b/tests/test_cli/test_edit_target.py @@ -42,6 +42,9 @@ class TestEditTarget(utils.CliTestCase): self.new_dest_tag = 'new-dest-tag' self.new_build_tag = 'new-build-tag' + def tearDown(self): + mock.patch.stopall() + def test_edit_target_without_option(self): expected = self.format_error_message("Please specify a build target") self.assert_system_exit( diff --git a/tests/test_cli/test_edit_user.py b/tests/test_cli/test_edit_user.py index 0b5784a5..1bc31ed2 100644 --- a/tests/test_cli/test_edit_user.py +++ b/tests/test_cli/test_edit_user.py @@ -26,6 +26,9 @@ class TestEditUser(utils.CliTestCase): self.user = 'user' self.rename = 'user2' + def tearDown(self): + mock.patch.stopall() + @mock.patch('sys.stdout', new_callable=six.StringIO) def test_handle_edit_user(self, stdout): args = [self.user] diff --git a/tests/test_cli/test_enable_channel.py b/tests/test_cli/test_enable_channel.py index 2c985e2b..f1243e15 100644 --- a/tests/test_cli/test_enable_channel.py +++ b/tests/test_cli/test_enable_channel.py @@ -30,6 +30,9 @@ class TestEnableChannel(utils.CliTestCase): 'name': 'test-channel'} ] + def tearDown(self): + mock.patch.stopall() + def __vm(self, result): m = koji.VirtualCall('mcall_method', [], {}) if isinstance(result, dict) and result.get('faultCode'): diff --git a/tests/test_cli/test_enable_host.py b/tests/test_cli/test_enable_host.py index 276f692f..bb9721e3 100644 --- a/tests/test_cli/test_enable_host.py +++ b/tests/test_cli/test_enable_host.py @@ -22,6 +22,9 @@ class TestEnableHost(utils.CliTestCase): %s: error: {message} """ % (self.progname, self.progname) + def tearDown(self): + mock.patch.stopall() + def test_handle_enable_host_no_such_host(self): """Test %s function""" % handle_enable_host.__name__ diff --git a/tests/test_cli/test_enable_user.py b/tests/test_cli/test_enable_user.py index 58ce95cd..9438ef08 100644 --- a/tests/test_cli/test_enable_user.py +++ b/tests/test_cli/test_enable_user.py @@ -21,6 +21,9 @@ class TestEnableUser(utils.CliTestCase): """ % (self.progname, self.progname) self.username = 'user' + def tearDown(self): + mock.patch.stopall() + def test_handle_enable_user_no_argument(self): """Test handle_enable_user function""" expected = self.format_error_message("You must specify the username of the user to enable") diff --git a/tests/test_cli/test_free_task.py b/tests/test_cli/test_free_task.py index 705afb6b..5e1f40f2 100644 --- a/tests/test_cli/test_free_task.py +++ b/tests/test_cli/test_free_task.py @@ -19,6 +19,9 @@ class TestFreeTask(utils.CliTestCase): %s: error: {message} """ % (self.progname, self.progname) + def tearDown(self): + mock.patch.stopall() + def test_free_task_without_arg(self): expected = self.format_error_message('please specify at least one task_id') self.assert_system_exit( diff --git a/tests/test_cli/test_grant_cg_access.py b/tests/test_cli/test_grant_cg_access.py index ed71eb3e..d2ba71bc 100644 --- a/tests/test_cli/test_grant_cg_access.py +++ b/tests/test_cli/test_grant_cg_access.py @@ -22,6 +22,9 @@ class TestGrantCGAccess(utils.CliTestCase): self.cg = 'cg' self.user = 'user' + def tearDown(self): + mock.patch.stopall() + def test_handle_grant_cg_access_arg_error(self): """Test handle_grant_cg_access function""" expected = self.format_error_message("Please specify a user and content generator") diff --git a/tests/test_cli/test_grant_permission.py b/tests/test_cli/test_grant_permission.py index d70e9aa7..f5df6ef4 100644 --- a/tests/test_cli/test_grant_permission.py +++ b/tests/test_cli/test_grant_permission.py @@ -26,6 +26,9 @@ class TestGrantPermission(utils.CliTestCase): self.perm = 'createuser' self.user = 'user' + def tearDown(self): + mock.patch.stopall() + def test_handle_grant_permission_argument_error(self): expected = self.format_error_message( "Please specify a permission and at least one user") diff --git a/tests/test_cli/test_hello.py b/tests/test_cli/test_hello.py index 6e35638c..079e68cb 100644 --- a/tests/test_cli/test_hello.py +++ b/tests/test_cli/test_hello.py @@ -39,6 +39,9 @@ class TestHello(utils.CliTestCase): """ % (self.progname, self.progname) self.huburl = "https://%s.local/%shub" % (self.progname, self.progname) + def tearDown(self): + mock.patch.stopall() + @mock.patch('sys.stdout', new_callable=six.StringIO) @mock.patch('koji_cli.commands._printable_unicode') def test_handle_moshimoshi(self, print_unicode_mock, stdout): diff --git a/tests/test_cli/test_import.py b/tests/test_cli/test_import.py index e91a5a76..381efad8 100644 --- a/tests/test_cli/test_import.py +++ b/tests/test_cli/test_import.py @@ -84,6 +84,9 @@ class TestImport(utils.CliTestCase): %s: error: {message} """ % (self.progname, self.progname) + def tearDown(self): + mock.patch.stopall() + def __do_import_test(self, options, session, arguments, **kwargs): expected = kwargs.get('expected', None) rpm_header = kwargs.get('rpm_header', {}) diff --git a/tests/test_cli/test_import_archive.py b/tests/test_cli/test_import_archive.py index dbdfe6bf..b6f07061 100644 --- a/tests/test_cli/test_import_archive.py +++ b/tests/test_cli/test_import_archive.py @@ -23,6 +23,9 @@ class TestImportArchive(utils.CliTestCase): %s: error: {message} """ % (self.progname, self.progname) + def tearDown(self): + mock.patch.stopall() + def test_import_archive_without_option(self): expected = self.format_error_message( "You must specify a build ID or N-V-R and an archive to import") diff --git a/tests/test_cli/test_import_cg.py b/tests/test_cli/test_import_cg.py index 3854702f..852945a3 100644 --- a/tests/test_cli/test_import_cg.py +++ b/tests/test_cli/test_import_cg.py @@ -34,6 +34,9 @@ class TestImportCG(utils.CliTestCase): %s: error: {message} """ % (self.progname, self.progname) + def tearDown(self): + mock.patch.stopall() + @mock.patch('sys.stdout', new_callable=six.StringIO) @mock.patch('koji_cli.commands._progress_callback') @mock.patch('koji.json') diff --git a/tests/test_cli/test_import_comps.py b/tests/test_cli/test_import_comps.py index 466b0fdd..de2f82f5 100644 --- a/tests/test_cli/test_import_comps.py +++ b/tests/test_cli/test_import_comps.py @@ -38,6 +38,9 @@ class TestImportComps(utils.CliTestCase): %s: error: {message} """ % (self.progname, self.progname) + def tearDown(self): + mock.patch.stopall() + @mock.patch('sys.stdout', new_callable=six.StringIO) @mock.patch('koji_cli.commands.libcomps') @mock.patch('koji_cli.commands._import_comps') diff --git a/tests/test_cli/test_import_sig.py b/tests/test_cli/test_import_sig.py index 12041fcb..2f1519ac 100644 --- a/tests/test_cli/test_import_sig.py +++ b/tests/test_cli/test_import_sig.py @@ -76,6 +76,9 @@ class TestImportSIG(utils.CliTestCase): %s: error: {message} """ % (self.progname, self.progname) + def tearDown(self): + mock.patch.stopall() + @mock.patch('sys.stderr', new_callable=six.StringIO) @mock.patch('sys.stdout', new_callable=six.StringIO) @mock.patch('koji.rip_rpm_sighdr') diff --git a/tests/test_cli/test_list_api.py b/tests/test_cli/test_list_api.py index 59606cf8..3d007cab 100644 --- a/tests/test_cli/test_list_api.py +++ b/tests/test_cli/test_list_api.py @@ -19,6 +19,9 @@ class TestListApi(utils.CliTestCase): self.options = mock.MagicMock() self.ensure_connection = mock.patch('koji_cli.commands.ensure_connection').start() + def tearDown(self): + mock.patch.stopall() + @mock.patch('sys.stdout', new_callable=six.StringIO) def test_anon_handle_list_api_all_method(self, stdout): """Test anon_handle_list_api function""" diff --git a/tests/test_cli/test_list_buildroot.py b/tests/test_cli/test_list_buildroot.py index 7b4b6ce7..89e7b613 100644 --- a/tests/test_cli/test_list_buildroot.py +++ b/tests/test_cli/test_list_buildroot.py @@ -22,6 +22,9 @@ class TestListBuilds(utils.CliTestCase): %s: error: {message} """ % (self.progname, self.progname) + def tearDown(self): + mock.patch.stopall() + def test_list_buildroot_without_args(self): self.assert_system_exit( anon_handle_list_buildroot, diff --git a/tests/test_cli/test_list_channels.py b/tests/test_cli/test_list_channels.py index 2d28b897..747ebed3 100644 --- a/tests/test_cli/test_list_channels.py +++ b/tests/test_cli/test_list_channels.py @@ -64,6 +64,9 @@ class TestListChannels(utils.CliTestCase): ]] ] + def tearDown(self): + mock.patch.stopall() + @mock.patch('sys.stdout', new_callable=StringIO) def test_list_channels_not_quiet(self, stdout): self.session.listChannels.return_value = self.list_channels diff --git a/tests/test_cli/test_list_notifications.py b/tests/test_cli/test_list_notifications.py index 5e7869d2..ef8f643f 100644 --- a/tests/test_cli/test_list_notifications.py +++ b/tests/test_cli/test_list_notifications.py @@ -23,6 +23,9 @@ class TestListNotifications(utils.CliTestCase): %s: error: {message} """ % (self.progname, self.progname) + def tearDown(self): + mock.patch.stopall() + @mock.patch('sys.stdout', new_callable=StringIO) def test_list_notifications(self, stdout): self.session.getBuildNotifications.return_value = [ diff --git a/tests/test_cli/test_list_permissions.py b/tests/test_cli/test_list_permissions.py index d9dc0771..9fb51578 100644 --- a/tests/test_cli/test_list_permissions.py +++ b/tests/test_cli/test_list_permissions.py @@ -36,6 +36,9 @@ class TestListPermissions(utils.CliTestCase): self.user = 'tester' self.userinfo = {'id': 101, 'name': self.user} + def tearDown(self): + mock.patch.stopall() + def test_handle_list_permissions_arg_error(self): """Test handle_list_permissions argument error (no argument is required)""" expected = self.format_error_message("This command takes no arguments") diff --git a/tests/test_cli/test_list_pkgs.py b/tests/test_cli/test_list_pkgs.py index 307c7835..cf4a88ca 100644 --- a/tests/test_cli/test_list_pkgs.py +++ b/tests/test_cli/test_list_pkgs.py @@ -52,6 +52,9 @@ class TestListPkgs(utils.CliTestCase): 'perm': None, 'perm_id': None} + def tearDown(self): + mock.patch.stopall() + def test_list_pkgs_non_exist_tag(self): self.session.getTag.return_value = None self.assert_system_exit( diff --git a/tests/test_cli/test_list_signed.py b/tests/test_cli/test_list_signed.py index 1f8b0694..9be5e43a 100644 --- a/tests/test_cli/test_list_signed.py +++ b/tests/test_cli/test_list_signed.py @@ -20,6 +20,9 @@ class TestListSigned(utils.CliTestCase): %s: error: {message} """ % (self.progname, self.progname) + def tearDown(self): + mock.patch.stopall() + def __vm(self, result): m = koji.VirtualCall('mcall_method', [], {}) if isinstance(result, dict) and result.get('faultCode'): diff --git a/tests/test_cli/test_list_tags.py b/tests/test_cli/test_list_tags.py index 023feb34..aa8f59c2 100644 --- a/tests/test_cli/test_list_tags.py +++ b/tests/test_cli/test_list_tags.py @@ -38,6 +38,9 @@ class TestListTags(utils.CliTestCase): %s: error: {message} """ % (self.progname, self.progname) + def tearDown(self): + mock.patch.stopall() + def test_list_tags_non_exist_package(self): pkg = 'test-pkg' self.session.getPackage.return_value = None diff --git a/tests/test_cli/test_mock_config.py b/tests/test_cli/test_mock_config.py index cc7fd605..9f58ad01 100644 --- a/tests/test_cli/test_mock_config.py +++ b/tests/test_cli/test_mock_config.py @@ -65,6 +65,9 @@ config_opts['macros']['%distribution'] = 'Koji Testing' %s: error: {message} """ % (self.progname, self.progname) + def tearDown(self): + mock.patch.stopall() + @mock.patch('sys.stdout', new_callable=six.StringIO) def test_handle_mock_config_buildroot_option(self, stdout): """Test anon_handle_mock_config buildroot options""" diff --git a/tests/test_cli/test_remove_external_repo.py b/tests/test_cli/test_remove_external_repo.py index 25d1847d..268b07e1 100644 --- a/tests/test_cli/test_remove_external_repo.py +++ b/tests/test_cli/test_remove_external_repo.py @@ -18,6 +18,9 @@ class TestRemoveExternalRepo(utils.CliTestCase): %s: error: {message} """ % (self.progname, self.progname) + def tearDown(self): + mock.patch.stopall() + def test_remove_external_repo_help(self): self.assert_help( handle_remove_external_repo, diff --git a/tests/test_cli/test_remove_pkg.py b/tests/test_cli/test_remove_pkg.py index 04f90769..8d28c6b8 100644 --- a/tests/test_cli/test_remove_pkg.py +++ b/tests/test_cli/test_remove_pkg.py @@ -28,6 +28,9 @@ class TestRemovePkg(utils.CliTestCase): %s: error: {message} """ % (self.progname, self.progname) + def tearDown(self): + mock.patch.stopall() + @mock.patch('sys.stderr', new_callable=six.StringIO) def test_handle_remove_pkg(self, stderr): tag = 'tag' diff --git a/tests/test_cli/test_remove_tag_inheritance.py b/tests/test_cli/test_remove_tag_inheritance.py index dd02d265..a0c261d1 100644 --- a/tests/test_cli/test_remove_tag_inheritance.py +++ b/tests/test_cli/test_remove_tag_inheritance.py @@ -50,6 +50,9 @@ class TestRemoveTagInheritance(utils.CliTestCase): 'perm': None, 'perm_id': None} + def tearDown(self): + mock.patch.stopall() + def test_remove_tag_inheritance_without_option(self): expected = self.format_error_message( "This command takes at least one argument: a tag name or ID") diff --git a/tests/test_cli/test_restart_host.py b/tests/test_cli/test_restart_host.py index 78011489..4d81b29c 100644 --- a/tests/test_cli/test_restart_host.py +++ b/tests/test_cli/test_restart_host.py @@ -24,6 +24,9 @@ class TestRestartHosts(utils.CliTestCase): self.watch_tasks_mock = mock.patch('koji_cli.commands.watch_tasks').start() self.task_id = 101 + def tearDown(self): + mock.patch.stopall() + def test_handle_restart_hosts_force_options(self): """Test %s function with --force option""" % handle_restart_hosts.__name__ arguments = ['--force'] diff --git a/tests/test_cli/test_set_task_priority.py b/tests/test_cli/test_set_task_priority.py index fb2c23a0..dda93d39 100644 --- a/tests/test_cli/test_set_task_priority.py +++ b/tests/test_cli/test_set_task_priority.py @@ -25,6 +25,9 @@ class TestSetTaskPriority(utils.CliTestCase): %s: error: {message} """ % (self.progname, self.progname) + def tearDown(self): + mock.patch.stopall() + @mock.patch('sys.stderr', new_callable=six.StringIO) @mock.patch('sys.stdout', new_callable=six.StringIO) def test_handle_set_task_priority(self, stdout, stderr): diff --git a/tests/test_cli/test_show_groups.py b/tests/test_cli/test_show_groups.py index 73e6deb6..15ef148a 100644 --- a/tests/test_cli/test_show_groups.py +++ b/tests/test_cli/test_show_groups.py @@ -52,6 +52,9 @@ class TestShowGroups(utils.CliTestCase): 'group_id': 3} ] + def tearDown(self): + mock.patch.stopall() + def test_show_groups_incorrect_num_of_args(self): arguments = [] self.assert_system_exit( diff --git a/tests/test_cli/test_untag_build.py b/tests/test_cli/test_untag_build.py index e1c42c64..5d2d6caf 100644 --- a/tests/test_cli/test_untag_build.py +++ b/tests/test_cli/test_untag_build.py @@ -34,6 +34,9 @@ class TestUntagBuild(utils.CliTestCase): 'name': 'test-package', 'release': '1.f35', 'tag.name': 'test-tag', 'tag_id': 460, 'version': '1.1'}]} + def tearDown(self): + mock.patch.stopall() + def __vm(self, result): m = koji.VirtualCall('mcall_method', [], {}) if isinstance(result, dict) and result.get('faultCode'): diff --git a/tests/test_cli/test_watch_logs.py b/tests/test_cli/test_watch_logs.py index 0e304cd6..24630677 100644 --- a/tests/test_cli/test_watch_logs.py +++ b/tests/test_cli/test_watch_logs.py @@ -21,6 +21,9 @@ class TestWatchLogs(utils.CliTestCase): %s: error: {message} """ % (self.progname, self.progname) + def tearDown(self): + mock.patch.stopall() + def test_handle_watch_logs_help(self): self.assert_help( anon_handle_watch_logs, diff --git a/tests/test_cli/test_win_build.py b/tests/test_cli/test_win_build.py index bb66f59b..bac4a674 100644 --- a/tests/test_cli/test_win_build.py +++ b/tests/test_cli/test_win_build.py @@ -39,6 +39,9 @@ class TestWinBuild(utils.CliTestCase): %s: error: {message} """ % (self.progname, self.progname) + def tearDown(self): + mock.patch.stopall() + def test_win_build_without_option(self): self.assert_system_exit( handle_win_build, diff --git a/tests/test_hub/test_get_next_release.py b/tests/test_hub/test_get_next_release.py index f199723c..457e1796 100644 --- a/tests/test_hub/test_get_next_release.py +++ b/tests/test_hub/test_get_next_release.py @@ -12,9 +12,6 @@ class TestGetNextRelease(DBQueryTestCase): self.get_build = mock.patch('kojihub.kojihub.get_build').start() self.binfo = {'name': 'name', 'version': 'version'} - def tearDown(self): - mock.patch.stopall() - def test_get_next_release_new(self): # no previous build self.qp_execute_one_return_value = None diff --git a/tests/test_hub/test_get_user.py b/tests/test_hub/test_get_user.py index ba040356..51bcf2e0 100644 --- a/tests/test_hub/test_get_user.py +++ b/tests/test_hub/test_get_user.py @@ -15,9 +15,6 @@ class TestGetUser(DBQueryTestCase): self.list_user_krb_principals = mock.patch( 'kojihub.kojihub.list_user_krb_principals').start() - def tearDown(self): - mock.patch.stopall() - def test_wrong_format_user_info(self): userinfo = ['test-user'] with self.assertRaises(koji.GenericError) as cm: diff --git a/tests/test_hub/test_models/test_host.py b/tests/test_hub/test_models/test_host.py index 248c9364..80daa53a 100644 --- a/tests/test_hub/test_models/test_host.py +++ b/tests/test_hub/test_models/test_host.py @@ -32,6 +32,9 @@ class TestHost(unittest.TestCase): self.queries = [] self.query_execute = mock.MagicMock() + def tearDown(self): + mock.patch.stopall() + @mock.patch('kojihub.kojihub.context') def test_instantiation_not_a_host(self, context): context.session.getHostId.return_value = None diff --git a/tests/test_hub/test_verify_names.py b/tests/test_hub/test_verify_names.py index c36e4c58..2fcc3979 100644 --- a/tests/test_hub/test_verify_names.py +++ b/tests/test_hub/test_verify_names.py @@ -46,6 +46,9 @@ class TestVerifyUser(unittest.TestCase): self.context.opts = {'MaxNameLengthInternal': 15, 'RegexUserName.compiled': re.compile('^[A-Za-z0-9/_.@-]+$')} + def tearDown(self): + mock.patch.stopall() + def test_verify_user_type_name(self): expected_error = "Name should be string" with self.assertRaises(koji.GenericError) as cm: diff --git a/tests/test_lib/test_auth.py b/tests/test_lib/test_auth.py index a8672dfe..f385b916 100644 --- a/tests/test_lib/test_auth.py +++ b/tests/test_lib/test_auth.py @@ -44,6 +44,9 @@ class TestAuthSession(unittest.TestCase): # start with "assert" self.context.session.assertLogin = mock.MagicMock() + def tearDown(self): + mock.patch.stopall() + def test_instance(self): """Simple kojihub.auth.Session instance""" self.context.opts = {