Use Python 3 style super() without arguments

tcms/kiwi_auth/forms.py

@@ -35,7 +35,7 @@ def clean_email(self):
             _("A user with that email already exists."))
 
     def save(self, commit=True):
-        user = super(RegistrationForm, self).save(commit=False)
+        user = super().save(commit=False)
         user.email = self.cleaned_data['email']
         user.is_active = False
         user.set_password(self.cleaned_data["password1"])

tcms/rpc/tests/test_category.py

@@ -8,7 +8,7 @@
 class TestCategory(APITestCase):
 
     def _fixture_setup(self):
-        super(TestCategory, self)._fixture_setup()
+        super()._fixture_setup()
 
         self.product_nitrate = ProductFactory(name='nitrate')
         self.product_xmlrpc = ProductFactory(name='xmlrpc')

tcms/rpc/tests/test_component.py

@@ -10,7 +10,7 @@
 
 class TestFilterComponents(APITestCase):
     def _fixture_setup(self):
-        super(TestFilterComponents, self)._fixture_setup()
+        super()._fixture_setup()
 
         self.product = ProductFactory(name='StarCraft')
         self.component = ComponentFactory(name='application', product=self.product,
@@ -34,7 +34,7 @@ def test_filter_non_existing(self):
 class TestCreateComponent(APITestCase):
 
     def _fixture_setup(self):
-        super(TestCreateComponent, self)._fixture_setup()
+        super()._fixture_setup()
 
         self.product = ProductFactory()
 
@@ -60,7 +60,7 @@ def test_add_component_with_no_perms(self):
 class TestUpdateComponent(APITestCase):
     # pylint: disable=objects-update-used
     def _fixture_setup(self):
-        super(TestUpdateComponent, self)._fixture_setup()
+        super()._fixture_setup()
 
         self.product = ProductFactory(name='StarCraft')
         self.component = ComponentFactory(name="application", product=self.product,

tcms/rpc/tests/test_plantype.py

@@ -8,7 +8,7 @@
 class PlanTypeMethods(APITestCase):
 
     def _fixture_setup(self):
-        super(PlanTypeMethods, self)._fixture_setup()
+        super()._fixture_setup()
 
         self.plan_type = PlanTypeFactory(name='xmlrpc plan type', description='')
 

tcms/rpc/tests/test_product.py

@@ -8,7 +8,7 @@
 class TestFilter(APITestCase):
 
     def _fixture_setup(self):
-        super(TestFilter, self)._fixture_setup()
+        super()._fixture_setup()
 
         self.product = ProductFactory(name='Nitrate')
         self.product_xmlrpc = ProductFactory(name='XMLRPC API')

tcms/rpc/tests/test_tag.py

@@ -7,7 +7,7 @@
 
 class Tag(APITestCase):
     def _fixture_setup(self):
-        super(Tag, self)._fixture_setup()
+        super()._fixture_setup()
 
         self.tag_db = TagFactory(name='db')
         self.tag_fedora = TagFactory(name='fedora')

tcms/rpc/tests/test_testbuild.py

@@ -11,7 +11,7 @@
 class BuildCreate(APITestCase):
 
     def _fixture_setup(self):
-        super(BuildCreate, self)._fixture_setup()
+        super()._fixture_setup()
 
         self.product = ProductFactory()
 
@@ -84,7 +84,7 @@ def test_build_create(self):
 class BuildUpdate(APITestCase):
 
     def _fixture_setup(self):
-        super(BuildUpdate, self)._fixture_setup()
+        super()._fixture_setup()
 
         self.product = ProductFactory()
         self.another_product = ProductFactory()
@@ -128,7 +128,7 @@ def test_build_update(self):
 class BuildFilter(APITestCase):
 
     def _fixture_setup(self):
-        super(BuildFilter, self)._fixture_setup()
+        super()._fixture_setup()
 
         self.product = ProductFactory()
         self.build = BuildFactory(product=self.product)

tcms/rpc/tests/test_testcase.py

@@ -19,14 +19,14 @@ class TestNotificationRemoveCC(APITestCase):
     """ Tests the XML-RPC testcase.notication_remove_cc method """
 
     def _fixture_setup(self):
-        super(TestNotificationRemoveCC, self)._fixture_setup()
+        super()._fixture_setup()
 
         self.default_cc = 'example@MrSenko.com'
         self.testcase = TestCaseFactory()
         self.testcase.emailing.add_cc(self.default_cc)
 
     def tearDown(self):
-        super(TestNotificationRemoveCC, self).tearDown()
+        super().tearDown()
         self.rpc_client.Auth.logout()
 
     def test_remove_existing_cc(self):
@@ -42,7 +42,7 @@ def test_remove_existing_cc(self):
 class TestFilterCases(APITestCase):
 
     def _fixture_setup(self):
-        super(TestFilterCases, self)._fixture_setup()
+        super()._fixture_setup()
 
         self.tester = UserFactory(username='great tester')
         self.product = ProductFactory(name='StarCraft')

tcms/rpc/tests/test_testplan.py

@@ -20,7 +20,7 @@
 class TestFilter(APITestCase):
 
     def _fixture_setup(self):
-        super(TestFilter, self)._fixture_setup()
+        super()._fixture_setup()
 
         self.product = ProductFactory()
         self.version = VersionFactory(product=self.product)
@@ -51,7 +51,7 @@ def test_filter_out_all_plans(self):
 class TestAddTag(APITestCase):
 
     def _fixture_setup(self):
-        super(TestAddTag, self)._fixture_setup()
+        super()._fixture_setup()
 
         self.product = ProductFactory()
         self.plans = [
@@ -90,7 +90,7 @@ def test_add_tag_without_permissions(self):
 class TestRemoveTag(APITestCase):
 
     def _fixture_setup(self):
-        super(TestRemoveTag, self)._fixture_setup()
+        super()._fixture_setup()
 
         self.product = ProductFactory()
         self.plans = [
@@ -174,7 +174,7 @@ class TestRemoveCase(APITestCase):
     """ Test the XML-RPC method TestPlan.remove_case() """
 
     def _fixture_setup(self):
-        super(TestRemoveCase, self)._fixture_setup()
+        super()._fixture_setup()
         self.testcase_1 = TestCaseFactory()
         self.testcase_2 = TestCaseFactory()
         self.plan_1 = TestPlanFactory()

tcms/rpc/tests/test_testrun.py

@@ -108,7 +108,7 @@ def test_should_remove_a_case_run(self):
 
 class TestAddTag(APITestCase):
     def _fixture_setup(self):
-        super(TestAddTag, self)._fixture_setup()
+        super()._fixture_setup()
 
         self.product = ProductFactory()
         self.version = VersionFactory()
@@ -156,7 +156,7 @@ def test_add_tag_without_permissions(self):
 
 class TestRemoveTag(APITestCase):
     def _fixture_setup(self):
-        super(TestRemoveTag, self)._fixture_setup()
+        super()._fixture_setup()
 
         self.product = ProductFactory()
         self.version = VersionFactory()

tcms/rpc/tests/test_user.py

@@ -74,7 +74,7 @@ class TestUserJoin(APITestCase):
     """Test User.join_group"""
 
     def _fixture_setup(self):
-        super(TestUserJoin, self)._fixture_setup()
+        super()._fixture_setup()
         # needs auth.change_user
         self.api_user.is_superuser = True
         self.api_user.save()
@@ -102,7 +102,7 @@ class TestUserUpdate(APITestCase):
     """Test User.update"""
 
     def _fixture_setup(self):
-        super(TestUserUpdate, self)._fixture_setup()
+        super()._fixture_setup()
 
         self.another_user = UserFactory()
         self.another_user.set_password('another-password')
@@ -115,12 +115,12 @@ def _fixture_setup(self):
         }
 
     def setUp(self):
-        super(TestUserUpdate, self).setUp()
+        super().setUp()
         # clear permissions b/c we set them inside individual tests
         self.api_user.user_permissions.all().delete()
 
     def tearDown(self):
-        super(TestUserUpdate, self).tearDown()
+        super().tearDown()
         self.api_user.set_password('api-testing')
         self.api_user.save()
 

tcms/rpc/tests/test_version.py

@@ -14,7 +14,7 @@
 class TestFilterVersions(APITestCase):
 
     def _fixture_setup(self):
-        super(TestFilterVersions, self)._fixture_setup()
+        super()._fixture_setup()
 
         self.product = ProductFactory(name='StarCraft')
         self.version = VersionFactory(value='0.7', product=self.product)

tcms/testcases/fields.py

@@ -29,7 +29,7 @@ def to_python(self, value):
 
     def clean(self, value):
         email_addrs = self.to_python(value)
-        super_instance = super(MultipleEmailField, self)
+        super_instance = super()
 
         valid_email_addrs = []
         invalid_email_addrs = []

tcms/tests/storage.py

@@ -28,4 +28,4 @@ def url(self, name):
         if (name not in self._files_found) and (name not in self._white_list):
             raise Exception('Static file "%s" does not exist and will cause 404 errors!' % name)
 
-        return super(RaiseWhenFileNotFound, self).url(name)
+        return super().url(name)