Addon Manager: Black and lint

src/Mod/AddonManager/AddonManagerTest/app/test_addon.py

@@ -36,9 +36,7 @@ class TestAddon(unittest.TestCase):
     MODULE = "test_addon"  # file name without extension
 
     def setUp(self):
-        self.test_dir = os.path.join(
-            os.path.dirname(__file__), "..", "data"
-        )
+        self.test_dir = os.path.join(os.path.dirname(__file__), "..", "data")
 
     def test_display_name(self):
 
@@ -178,7 +176,8 @@ class TestAddon(unittest.TestCase):
         self.assertEqual(addon.repo_type, Addon.Kind.MACRO)
         self.assertEqual(addon.name, "DoNothing")
         self.assertEqual(
-            addon.macro.comment, "Do absolutely nothing. For Addon Manager integration tests."
+            addon.macro.comment,
+            "Do absolutely nothing. For Addon Manager integration tests.",
         )
         self.assertEqual(addon.url, "https://github.com/FreeCAD/FreeCAD")
         self.assertEqual(addon.macro.version, "1.0")

src/Mod/AddonManager/AddonManagerTest/app/test_installer.py

@@ -116,7 +116,9 @@ class TestAddonInstaller(unittest.TestCase):
                 os.path.join(self.test_data_dir, "good_package.xml"),
                 os.path.join(addon_dir, "package.xml"),
             )
-            good_metadata = MetadataReader.from_file(os.path.join(addon_dir, "package.xml"))
+            good_metadata = MetadataReader.from_file(
+                os.path.join(addon_dir, "package.xml")
+            )
             installer._update_metadata()
             self.assertEqual(self.real_addon.installed_version, good_metadata.version)
 

src/Mod/AddonManager/AddonManagerTest/app/test_metadata.py

@@ -204,9 +204,13 @@ class TestUrlType(unittest.TestCase):
 
 
 class TestMetadataAuxiliaryFunctions(unittest.TestCase):
-
     def test_get_first_supported_freecad_version_simple(self):
-        from addonmanager_metadata import Metadata, Version, get_first_supported_freecad_version
+        from addonmanager_metadata import (
+            Metadata,
+            Version,
+            get_first_supported_freecad_version,
+        )
+
         expected_result = Version(from_string="0.20.2beta")
         metadata = self.given_metadata_with_freecadmin_set(expected_result)
         first_version = get_first_supported_freecad_version(metadata)
@@ -215,12 +219,18 @@ class TestMetadataAuxiliaryFunctions(unittest.TestCase):
     @staticmethod
     def given_metadata_with_freecadmin_set(min_version):
         from addonmanager_metadata import Metadata
+
         metadata = Metadata()
         metadata.freecadmin = min_version
         return metadata
 
     def test_get_first_supported_freecad_version_with_content(self):
-        from addonmanager_metadata import Metadata, Version, get_first_supported_freecad_version
+        from addonmanager_metadata import (
+            Metadata,
+            Version,
+            get_first_supported_freecad_version,
+        )
+
         expected_result = Version(from_string="0.20.2beta")
         metadata = self.given_metadata_with_freecadmin_in_content(expected_result)
         first_version = get_first_supported_freecad_version(metadata)
@@ -229,17 +239,24 @@ class TestMetadataAuxiliaryFunctions(unittest.TestCase):
     @staticmethod
     def given_metadata_with_freecadmin_in_content(min_version):
         from addonmanager_metadata import Metadata, Version
+
         v_list = min_version.version_as_list
         metadata = Metadata()
         wb1 = Metadata()
-        wb1.freecadmin = Version(from_list=[v_list[0]+1,v_list[1],v_list[2],v_list[3]])
+        wb1.freecadmin = Version(
+            from_list=[v_list[0] + 1, v_list[1], v_list[2], v_list[3]]
+        )
         wb2 = Metadata()
-        wb2.freecadmin = Version(from_list=[v_list[0],v_list[1]+1,v_list[2],v_list[3]])
+        wb2.freecadmin = Version(
+            from_list=[v_list[0], v_list[1] + 1, v_list[2], v_list[3]]
+        )
         wb3 = Metadata()
-        wb3.freecadmin = Version(from_list=[v_list[0],v_list[1],v_list[2]+1,v_list[3]])
+        wb3.freecadmin = Version(
+            from_list=[v_list[0], v_list[1], v_list[2] + 1, v_list[3]]
+        )
         m1 = Metadata()
         m1.freecadmin = min_version
-        metadata.content = {"workbench":[wb1,wb2,wb3],"macro":[m1]}
+        metadata.content = {"workbench": [wb1, wb2, wb3], "macro": [m1]}
         return metadata
 
 

src/Mod/AddonManager/AddonManagerTest/gui/gui_mocks.py

@@ -67,9 +67,9 @@ class DialogInteractor(QtCore.QObject):
         # doesn't set the title of a modal dialog:
         os = QtCore.QSysInfo.productType()  # Qt5 gives "osx", Qt6 gives "macos"
         if os in ["osx", "macos"] or (
-                hasattr(widget, "windowTitle")
-                and callable(widget.windowTitle)
-                and widget.windowTitle() == self.dialog_to_watch_for
+            hasattr(widget, "windowTitle")
+            and callable(widget.windowTitle)
+            and widget.windowTitle() == self.dialog_to_watch_for
         ):
             return True
         return False

src/Mod/AddonManager/AddonManagerTest/gui/test_installer_gui.py

@@ -45,7 +45,6 @@ class TestInstallerGui(unittest.TestCase):
         self.installer_gui = AddonInstallerGUI(self.addon_to_install)
         self.finalized_thread = False
 
-
     def tearDown(self):
         pass
 

src/Mod/AddonManager/AddonManagerTest/gui/test_uninstaller_gui.py

@@ -108,7 +108,9 @@ class TestUninstallerGUI(unittest.TestCase):
             translate("AddonsInstaller", "Removing Addon"),
             QtWidgets.QDialogButtonBox.Cancel,
         )
-        QtCore.QTimer.singleShot(1000, worker.stop) # If the test fails, this kills the "worker"
+        QtCore.QTimer.singleShot(
+            1000, worker.stop
+        )  # If the test fails, this kills the "worker"
         self.uninstaller_gui._confirm_uninstallation = lambda: True
         self.uninstaller_gui._run_uninstaller = worker.work
         self.uninstaller_gui._finalize = lambda: None

src/Mod/AddonManager/AddonManagerTest/gui/test_update_all_gui.py

@@ -122,6 +122,7 @@ class TestUpdateAllGui(unittest.TestCase):
                 self.counter = 0
                 self.LIMIT = 100
                 self.limit_reached = False
+
             def run(self):
                 while self.counter < self.LIMIT:
                     if QtCore.QThread.currentThread().isInterruptionRequested():
@@ -129,6 +130,7 @@ class TestUpdateAllGui(unittest.TestCase):
                     self.counter += 1
                     sleep(0.01)
                 self.limit_reached = True
+
         worker = Worker()
         self.factory.work_function = worker.run
         self.test_object.run()

src/Mod/AddonManager/AddonManagerTest/gui/test_workers_startup.py

@@ -45,7 +45,9 @@ class TestWorkersStartup(unittest.TestCase):
 
     MODULE = "test_workers_startup"  # file name without extension
 
-    @unittest.skipUnless(run_slow_tests, "This integration test is slow and uses the network")
+    @unittest.skipUnless(
+        run_slow_tests, "This integration test is slow and uses the network"
+    )
     def setUp(self):
         """Set up the test"""
         self.test_dir = os.path.join(