simplify py3 tests

Makefile

@@ -80,8 +80,9 @@ test: test2 test3
 
 test2:
 	coverage2 erase
-	PYTHONPATH=hub/.:plugins/hub/.:plugins/builder/.:plugins/cli/.:cli/.:www/lib coverage2 run \
-	    --source . -m nose
+	PYTHONPATH=.:plugins/builder/.:plugins/cli/.:cli/.:www/lib coverage2 run \
+	    --source . -m nose tests/test_builder tests/test_cli tests/test_lib \
+	    tests/test_plugins/test*builder.py tests/test_plugins/test*cli.py
 	coverage2 report
 	coverage2 html
 	@echo Full coverage report at file://${CURDIR}/htmlcov/py2/index.html

tests/test_hub/test_add_archivetype.py

@@ -1,8 +1,4 @@
-from __future__ import absolute_import
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 import mock
 
 import koji

tests/test_hub/test_add_btype.py

@@ -1,8 +1,4 @@
-from __future__ import absolute_import
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 import mock
 
 import koji

tests/test_hub/test_add_external_rpm.py

@@ -1,9 +1,5 @@
-from __future__ import absolute_import
 import mock
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 
 import koji
 import kojihub

tests/test_hub/test_add_host.py

@@ -1,9 +1,5 @@
-from __future__ import absolute_import
 import mock
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 
 import koji
 import kojihub

tests/test_hub/test_add_host_to_channel.py

@@ -1,9 +1,5 @@
-from __future__ import absolute_import
 import mock
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 
 import koji
 import kojihub

tests/test_hub/test_apply_query_opts.py

@@ -1,9 +1,5 @@
-from __future__ import absolute_import
 import copy
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 from nose.tools import eq_
 
 import kojihub

tests/test_hub/test_cg_importer.py

@@ -1,11 +1,7 @@
-from __future__ import absolute_import
 import mock
 import os
 import shutil
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 
 import koji
 import kojihub

tests/test_hub/test_check_volume_policy.py

@@ -1,9 +1,5 @@
-from __future__ import absolute_import
 import mock
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 import koji
 import koji.policy
 import kojihub

tests/test_hub/test_complete_image_build.py

@@ -1,4 +1,3 @@
-from __future__ import absolute_import
 import copy
 import json
 import mock
@@ -6,10 +5,8 @@ import os
 import os.path
 import shutil
 import tempfile
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
+
 import koji
 import koji.util
 import kojihub

tests/test_hub/test_complete_maven_build.py

@@ -1,4 +1,3 @@
-from __future__ import absolute_import
 import copy
 import json
 import mock
@@ -6,10 +5,8 @@ import os
 import os.path
 import shutil
 import tempfile
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
+
 import koji
 import koji.util
 import kojihub

tests/test_hub/test_create_tag.py

@@ -1,10 +1,7 @@
 # coding: utf-8
-from __future__ import absolute_import
 import mock
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
+
 import koji
 import kojihub
 

tests/test_hub/test_delete_build.py

@@ -1,9 +1,5 @@
-from __future__ import absolute_import
 import mock
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 import kojihub
 import time
 from koji import GenericError

tests/test_hub/test_delete_tag.py

@@ -1,9 +1,5 @@
-from __future__ import absolute_import
 import mock
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 import koji
 import kojihub
 

tests/test_hub/test_dist_repo.py

@@ -1,8 +1,4 @@
-from __future__ import absolute_import
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 import json
 import mock
 import os

tests/test_hub/test_edit_host.py

@@ -1,9 +1,5 @@
-from __future__ import absolute_import
 import mock
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 
 import koji
 import kojihub

tests/test_hub/test_edit_tag.py

@@ -1,10 +1,6 @@
 # coding: utf-8
-from __future__ import absolute_import
 import mock
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 import koji
 import kojihub
 

tests/test_hub/test_edit_tag_external_repo.py

@@ -1,8 +1,5 @@
 import mock
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 
 import koji
 import kojihub

tests/test_hub/test_edit_user.py

@@ -1,8 +1,5 @@
 import mock
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 
 import koji
 import kojihub

tests/test_hub/test_ensure_volume_symlink.py

@@ -1,13 +1,9 @@
-from __future__ import absolute_import
 import mock
 import os
 import os.path
 import shutil
 import tempfile
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 import koji
 import kojihub
 

tests/test_hub/test_getPackageID.py

@@ -1,5 +1,3 @@
-from __future__ import absolute_import
-
 import mock
 
 from .utils import DBQueryTestCase

tests/test_hub/test_getRPMDeps.py

@@ -1,10 +1,6 @@
-from __future__ import absolute_import
 import mock
 import os
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 import koji
 import kojihub
 

tests/test_hub/test_getRPMFile.py

@@ -1,13 +1,9 @@
-from __future__ import absolute_import
 
 import os
 
 import mock
 
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 import koji
 import kojihub
 

tests/test_hub/test_get_active_repos.py

@@ -1,9 +1,5 @@
-from __future__ import absolute_import
 import mock
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 import koji
 import kojihub
 import koji.db

tests/test_hub/test_get_archive_file.py

@@ -1,9 +1,5 @@
-from __future__ import absolute_import
 import mock
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 
 import koji
 import kojihub

tests/test_hub/test_get_build_logs.py

@@ -1,12 +1,8 @@
-from __future__ import absolute_import
 import mock
 import os
 import shutil
 import tempfile
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 import koji
 import kojihub
 

tests/test_hub/test_get_build_notifications.py

@@ -1,8 +1,5 @@
 import mock
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 import koji
 import kojihub
 

tests/test_hub/test_get_build_type.py

@@ -1,9 +1,5 @@
-from __future__ import absolute_import
 import mock
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 
 import kojihub
 

tests/test_hub/test_get_external_repos.py

@@ -1,5 +1,3 @@
-from __future__ import absolute_import
-
 import koji
 import kojihub
 from .utils import DBQueryTestCase

tests/test_hub/test_get_host.py

@@ -1,9 +1,5 @@
-from __future__ import absolute_import
 import mock
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 
 import koji
 import kojihub

tests/test_hub/test_get_next_release.py

@@ -1,9 +1,5 @@
-from __future__ import absolute_import
 import mock
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 import koji
 import kojihub
 

tests/test_hub/test_get_task_children.py

@@ -1,9 +1,5 @@
-from __future__ import absolute_import
 import mock
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 
 import koji
 import kojihub

tests/test_hub/test_get_upload_path.py

@@ -1,12 +1,8 @@
-from __future__ import absolute_import
 import os
 import mock
 import shutil
 import tempfile
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 
 import kojihub
 from koji import GenericError

tests/test_hub/test_get_user_perms.py

@@ -1,9 +1,5 @@
-from __future__ import absolute_import
 import mock
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 import koji
 import kojihub
 

tests/test_hub/test_get_verify_class.py

@@ -1,9 +1,5 @@
-from __future__ import absolute_import
 import hashlib
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 
 import kojihub
 from koji import GenericError

tests/test_hub/test_group_operations.py

@@ -1,9 +1,5 @@
-from __future__ import absolute_import
 import mock
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 import koji
 import kojihub
 

tests/test_hub/test_import_build.py

@@ -1,12 +1,8 @@
-from __future__ import absolute_import
 import copy
 import mock
 import shutil
 import tempfile
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 
 import koji
 import kojihub

tests/test_hub/test_import_image_internal.py

@@ -1,12 +1,8 @@
-from __future__ import absolute_import
 import mock
 import os
 import shutil
 import tempfile
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 
 import kojihub
 

tests/test_hub/test_insert_processor.py

@@ -1,9 +1,5 @@
-from __future__ import absolute_import
 import mock
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 
 import koji
 import kojihub

tests/test_hub/test_list_archive_files.py

@@ -1,9 +1,5 @@
-from __future__ import absolute_import
 import mock
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 
 import koji
 import kojihub

tests/test_hub/test_list_archives.py

@@ -1,4 +1,3 @@
-from __future__ import absolute_import
 import mock
 from .utils import DBQueryTestCase
 

tests/test_hub/test_list_btypes.py

@@ -1,9 +1,5 @@
-from __future__ import absolute_import
 import mock
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 
 import kojihub
 

tests/test_hub/test_list_channels.py

@@ -1,9 +1,5 @@
-from __future__ import absolute_import
 import mock
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 
 import koji
 import kojihub

tests/test_hub/test_list_hosts.py

@@ -1,9 +1,5 @@
-from __future__ import absolute_import
 import mock
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 
 import koji
 import kojihub

tests/test_hub/test_list_task_output.py

@@ -1,9 +1,5 @@
-from __future__ import absolute_import
 import mock
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 
 import kojihub
 

tests/test_hub/test_listing.py

@@ -1,9 +1,5 @@
-from __future__ import absolute_import
 import mock
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 
 import kojihub
 

tests/test_hub/test_new_build.py

@@ -1,9 +1,5 @@
-from __future__ import absolute_import
 import mock
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 
 import koji
 import kojihub

tests/test_hub/test_new_typed_build.py

@@ -1,9 +1,5 @@
-from __future__ import absolute_import
 import mock
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 
 import kojihub
 

tests/test_hub/test_notifications.py

@@ -1,9 +1,5 @@
-from __future__ import absolute_import
 import mock
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 
 import koji
 import kojihub

tests/test_hub/test_perm_operations.py

@@ -1,11 +1,7 @@
-from __future__ import absolute_import
 
 import mock
 
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 
 import koji
 import kojihub

tests/test_hub/test_pkglist.py

@@ -1,9 +1,5 @@
-from __future__ import absolute_import
 import mock
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 
 import koji
 import kojihub

tests/test_hub/test_policy_tests.py

@@ -1,9 +1,5 @@
-from __future__ import absolute_import
 import mock
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 
 import koji
 import kojihub

tests/test_hub/test_query_processor.py

@@ -1,9 +1,5 @@
-from __future__ import absolute_import
 import mock
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 
 import kojihub
 

tests/test_hub/test_recycle_build.py

@@ -1,4 +1,3 @@
-from __future__ import absolute_import
 import mock
 
 import koji

tests/test_hub/test_remove_host_from_channel.py

@@ -1,9 +1,5 @@
-from __future__ import absolute_import
 import mock
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 
 import koji
 import kojihub

tests/test_hub/test_repos.py

@@ -1,10 +1,6 @@
-from __future__ import absolute_import
 import mock
 
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 
 import koji
 import kojihub

tests/test_hub/test_rpmdiff.py

@@ -1,11 +1,7 @@
-from __future__ import absolute_import
 import copy
 import mock
 import os
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 
 import koji
 import kojihub

tests/test_hub/test_savepoint.py

@@ -1,10 +1,6 @@
-from __future__ import absolute_import
 import mock
 
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 
 import kojihub
 

tests/test_hub/test_set_host_enabled.py

@@ -1,9 +1,5 @@
-from __future__ import absolute_import
 import mock
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 
 import koji
 import kojihub

tests/test_hub/test_tag_operations.py

@@ -1,12 +1,5 @@
-from __future__ import absolute_import
-import copy
 import mock
-import shutil
-import tempfile
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 
 import koji
 import kojihub

tests/test_hub/test_task_wait_results.py

@@ -1,10 +1,6 @@
-from __future__ import absolute_import
 import mock
-import six.moves.xmlrpc_client
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import xmlrpc.client
+import unittest
 
 import koji
 import kojihub
@@ -66,8 +62,8 @@ class TestTaskWaitResults(unittest.TestCase):
             task = self.getTask(t)
             task.getResult.return_value = "OK"
             task.isCanceled.return_value = False
-        self.tasks[6].getResult.side_effect = six.moves.xmlrpc_client.Fault(1, "error")
-        with self.assertRaises(six.moves.xmlrpc_client.Fault):
+        self.tasks[6].getResult.side_effect = xmlrpc.client.Fault(1, "error")
+        with self.assertRaises(xmlrpc.client.Fault):
             results = self.host_exports.taskWaitResults(parent, task_ids)
             self.assertEqual(results, [])
         self.tasks[6].getResult.side_effect = koji.GenericError('problem')

tests/test_hub/test_update_processor.py

@@ -1,9 +1,5 @@
-from __future__ import absolute_import
 import mock
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 
 import kojihub
 

tests/test_hub/test_user_groups.py

@@ -1,9 +1,5 @@
-from __future__ import absolute_import
 import mock
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 
 import koji
 import kojihub

tests/test_hub/test_write_maven_repo_metadata.py

@@ -1,11 +1,7 @@
-from __future__ import absolute_import, with_statement
 import mock
 import os
 import datetime
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 
 from kojihub import _write_maven_repo_metadata
 

tests/test_hub/utils.py

@@ -1,10 +1,5 @@
-from __future__ import absolute_import
 import mock
-import six
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 
 import kojihub
 
@@ -36,7 +31,7 @@ class DBQueryTestCase(unittest.TestCase):
         return query
 
     def assertQueryEqual(self, query, **kwargs):
-        for k, v in six.iteritems(kwargs):
+        for k, v in kwargs.items():
             self.assertEqual(getattr(query, k, None), v)
 
     def assertLastQueryEqual(self, **kwargs):

tests/test_www/loadwebindex.py

@@ -1,4 +1,3 @@
-from __future__ import absolute_import
 import os
 import imp
 

tests/test_www/test_taskinfo.py

@@ -1,16 +1,11 @@
-from __future__ import absolute_import
-from __future__ import print_function
 import mock
-import koji
 import copy
 import datetime
-
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 
 from mock import call
+
+import koji
 from .loadwebindex import webidx
 
 

tests/test_www/test_util.py

@@ -1,9 +1,4 @@
-from __future__ import absolute_import
-
-try:
-    import unittest2 as unittest
-except ImportError:
-    import unittest
+import unittest
 
 from kojiweb.util import formatMode