Ruff: Add PLW0603 (#13097)

Author: manuel-sommer

ruff.toml

@@ -85,7 +85,7 @@ select = [
    "PLC01", "PLC02", "PLC0414", "PLC18", "PLC24", "PLC28", "PLC3",
    "PLE",
    "PLR01", "PLR02", "PLR04", "PLR0915", "PLR1711", "PLR1704", "PLR1714", "PLR1716", "PLR172", "PLR173", "PLR2044", "PLR5", "PLR6104", "PLR6201", 
-   "PLW01", "PLW02", "PLW04", "PLW0602", "PLW0604", "PLW07", "PLW1", "PLW2", "PLW3",
+   "PLW01", "PLW02", "PLW04", "PLW0602", "PLW0603", "PLW0604", "PLW07", "PLW1", "PLW2", "PLW3",
    "UP",
    "FURB",
    "RUF",

tests/base_test_class.py

@@ -57,12 +57,11 @@ def setUpClass(cls):
         # Path for automatic downloads, mapped to the media path
         cls.export_path = "/app"
 
-        global dd_driver
+        global dd_driver  # noqa: PLW0603 global variables are dirty, but in unit tests scenario's like these they are acceptable
         if not dd_driver:
             # setupModule and tearDownModule are not working in our scenario, so for now we use setupClass and a global variable
-            # global variables are dirty, but in unit tests scenario's like these they are acceptable
             logger.info(f"launching browser for: {cls.__name__}")
-            global dd_driver_options
+            global dd_driver_options  # noqa: PLW0603 global variables are dirty, but in unit tests scenario's like these they are acceptable
             dd_driver_options = Options()
 
             # headless means no UI, if you want to see what is happening remove headless. Adding detach will leave the window open after the test