More pylint fixes. (#72)

spotfiresoftware · Sep 23, 2024 · 0b39ecb · 0b39ecb
1 parent 6c86a6b
commit 0b39ecb
Showing 1 changed file with 9 additions and 8 deletions.
diff --git a/spotfire/test/__main__.py b/spotfire/test/__main__.py
@@ -26,16 +26,17 @@ def load_tests(loader, tests, pattern):  # pylint: disable=unused-argument
     return test_suite
 
 
-py_version = f"{sys.version_info.major}.{sys.version_info.minor}"
-test_env = os.getenv("TEST_ENVIRONMENT")
-report_title_suffix, report_name = "", f"{platform.system().lower()}-{py_version}"
-if test_env:
-    report_title_suffix = f" ({test_env} test environment)"
-    report_name += f"-{test_env}"
+PYTHON_VERSION = f"{sys.version_info.major}.{sys.version_info.minor}"
+TEST_ENVIRONMENT = os.getenv("TEST_ENVIRONMENT")
+REPORT_SUFFIX = ""
+REPORT_NAME = f"{platform.system().lower()}-{PYTHON_VERSION}"
+if TEST_ENVIRONMENT:
+    REPORT_SUFFIX = f" ({TEST_ENVIRONMENT} test environment)"
+    REPORT_NAME += f"-{TEST_ENVIRONMENT}"
 runner = HtmlTestRunner.HTMLTestRunner(combine_reports=True,
                                        output="build/test-results/",
                                        template=utils.get_test_data_file("template.html.in"),
-                                       report_title=f"Python {py_version} on {platform.system()}{report_title_suffix}",
-                                       report_name=report_name,
+                                       report_title=f"Python {PYTHON_VERSION} on {platform.system()}{REPORT_SUFFIX}",
+                                       report_name=REPORT_NAME,
                                        add_timestamp=False)
 unittest.main(testRunner=runner, failfast=False, buffer=False, catchbreak=False)