Ruff: Fix PTH118, merge PTH11 (#11503)

Author: kiblik

dojo/forms.py

@@ -2428,7 +2428,7 @@ def get_jira_issue_template_dir_choices():
 
         for dirname in dirnames:
             clean_base_dir = base_dir.removeprefix(settings.TEMPLATE_DIR_PREFIX)
-            template_dir_list.append((os.path.join(clean_base_dir, dirname), dirname))
+            template_dir_list.append((str(Path(clean_base_dir) / dirname), dirname))
 
     logger.debug("templates: %s", template_dir_list)
     return template_dir_list

dojo/jira_link/helper.py

@@ -1,7 +1,6 @@
 import io
 import json
 import logging
-import os
 from pathlib import Path
 from typing import Any
 
@@ -333,8 +332,8 @@ def get_jira_issue_template(obj):
         template_dir = "issue-trackers/jira_full/"
 
     if isinstance(obj, Finding_Group):
-        return os.path.join(template_dir, "jira-finding-group-description.tpl")
-    return os.path.join(template_dir, "jira-description.tpl")
+        return Path(template_dir) / "jira-finding-group-description.tpl"
+    return Path(template_dir) / "jira-description.tpl"
 
 
 def get_jira_creation(obj):

dojo/models.py

@@ -2,7 +2,6 @@
 import copy
 import hashlib
 import logging
-import os
 import re
 import warnings
 from contextlib import suppress
@@ -150,7 +149,7 @@ def __call__(self, model_instance, filename):
             filename += ext
         if self.directory is None:
             return filename
-        return os.path.join(now().strftime(self.directory), filename)
+        return Path(now().strftime(self.directory)) / filename
 
 
 class Regulation(models.Model):

dojo/settings/settings.dist.py

@@ -442,7 +442,7 @@ def generate_url(scheme, double_slashes, user, password, host, port, path, param
     # Put strings here, like "/home/html/static" or "C:/www/django/static".
     # Always use forward slashes, even on Windows.
     # Don't forget to use absolute paths, not relative paths.
-    os.path.join(Path(DOJO_ROOT).parent, "components", "node_modules"),
+    Path(DOJO_ROOT).parent / "components" / "node_modules",
 )
 
 # List of finder classes that know how to find static files in
@@ -912,8 +912,6 @@ def saml2_attrib_map_format(dict):
 SAML2_LOGIN_BUTTON_TEXT = env("DD_SAML2_LOGIN_BUTTON_TEXT")
 SAML2_LOGOUT_URL = env("DD_SAML2_LOGOUT_URL")
 if SAML2_ENABLED:
-    from os import path
-
     import saml2
     import saml2.saml
     # SSO_URL = env('DD_SSO_URL')
@@ -949,7 +947,7 @@ def saml2_attrib_map_format(dict):
         "entityid": str(SAML2_ENTITY_ID),
 
         # directory with attribute mapping
-        "attribute_map_dir": path.join(BASEDIR, "attribute-maps"),
+        "attribute_map_dir": Path(BASEDIR) / "attribute-maps",
         # do now discard attributes not specified in attribute-maps
         "allow_unknown_attributes": SAML_ALLOW_UNKNOWN_ATTRIBUTES,
         # this block states what services we provide

dojo/tools/factory.py

@@ -117,7 +117,7 @@ def requires_tool_type(scan_type):
 package_dir = str(Path(__file__).resolve().parent)
 for module_name in os.listdir(package_dir):  # noqa: PTH208
     # check if it's dir
-    if Path(os.path.join(package_dir, module_name)).is_dir():
+    if (Path(package_dir) / module_name).is_dir():
         try:
             # check if it's a Python module
             if find_spec(f"dojo.tools.{module_name}.parser"):

dojo/views.py

@@ -1,5 +1,4 @@
 import logging
-import os
 from pathlib import Path
 
 from auditlog.models import LogEntry
@@ -151,7 +150,7 @@ def manage_files(request, oid, obj_type):
 
             for o in files_formset.deleted_objects:
                 logger.debug("removing file: %s", o.file.name)
-                Path(os.path.join(settings.MEDIA_ROOT, o.file.name)).unlink()
+                (Path(settings.MEDIA_ROOT) / o.file.name).unlink()
 
             for o in files_formset.new_objects:
                 logger.debug("adding file: %s", o.file.name)
@@ -162,7 +161,7 @@ def manage_files(request, oid, obj_type):
                                                      finding__isnull=True)
             for o in orphan_files:
                 logger.debug("purging orphan file: %s", o.file.name)
-                Path(os.path.join(settings.MEDIA_ROOT, o.file.name)).unlink()
+                (Path(settings.MEDIA_ROOT) / o.file.name).unlink()
                 o.delete()
 
             messages.add_message(

ruff.toml

@@ -68,7 +68,7 @@ select = [
    "TC",
    "INT",
    "ARG003", "ARG004", "ARG005",
-   "PTH2", "PTH10", "PTH110", "PTH111", "PTH112", "PTH113", "PTH114", "PTH115", "PTH116", "PTH117", "PTH119", "PTH120", "PTH121", "PTH122", "PTH124",
+   "PTH2", "PTH10", "PTH11", "PTH120", "PTH121", "PTH122", "PTH124",
    "TD001", "TD004", "TD005",
    "FIX001", "FIX003",
    "PD",

tests/file_test.py

@@ -35,9 +35,9 @@ def test_add_file_finding_level(self):
         driver.find_element(By.LINK_TEXT, "Manage Files").click()
         # select first file input field: form-0-image
         # Set full image path for image file 'strange.png
-        image_path = os.path.join(dir_path, "finding_image.png")
+        image_path = dir_path / "finding_image.png"
         driver.find_element(By.ID, "id_form-0-title").send_keys("Finding Title")
-        driver.find_element(By.ID, "id_form-0-file").send_keys(image_path)
+        driver.find_element(By.ID, "id_form-0-file").send_keys(str(image_path))
         # Save uploaded image
         with WaitForPageLoad(driver, timeout=50):
             driver.find_element(By.CSS_SELECTOR, "button.btn.btn-success").click()
@@ -76,9 +76,9 @@ def test_add_file_test_level(self):
         driver.find_element(By.NAME, "Manage Files").click()
         # select first file input field: form-0-image
         # Set full image path for image file 'strange.png
-        image_path = os.path.join(dir_path, "finding_image.png")
+        image_path = dir_path / "finding_image.png"
         driver.find_element(By.ID, "id_form-0-title").send_keys("Test Title")
-        driver.find_element(By.ID, "id_form-0-file").send_keys(image_path)
+        driver.find_element(By.ID, "id_form-0-file").send_keys(str(image_path))
         # Save uploaded image
         with WaitForPageLoad(driver, timeout=50):
             driver.find_element(By.CSS_SELECTOR, "button.btn.btn-success").click()
@@ -116,9 +116,9 @@ def test_add_file_engagement_level(self):
         driver.find_element(By.NAME, "Manage Files").click()
         # select first file input field: form-0-image
         # Set full image path for image file 'strange.png
-        image_path = os.path.join(dir_path, "finding_image.png")
+        image_path = dir_path / "finding_image.png"
         driver.find_element(By.ID, "id_form-0-title").send_keys("Engagement Title")
-        driver.find_element(By.ID, "id_form-0-file").send_keys(image_path)
+        driver.find_element(By.ID, "id_form-0-file").send_keys(str(image_path))
         # Save uploaded image
         with WaitForPageLoad(driver, timeout=50):
             driver.find_element(By.CSS_SELECTOR, "button.btn.btn-success").click()

tests/finding_test.py

@@ -146,8 +146,8 @@ def test_add_image(self):
         driver.find_element(By.LINK_TEXT, "Manage Files").click()
         # select first file input field: form-0-image
         # Set full image path for image file 'strange.png
-        image_path = os.path.join(dir_path, "finding_image.png")
-        driver.find_element(By.ID, "id_form-0-file").send_keys(image_path)
+        image_path = dir_path / "finding_image.png"
+        driver.find_element(By.ID, "id_form-0-file").send_keys(str(image_path))
         driver.find_element(By.ID, "id_form-0-title").send_keys("Image Title")
         # Save uploaded image
         with WaitForPageLoad(driver, timeout=50):
@@ -466,8 +466,8 @@ def test_import_scan_result(self):
         # Select `Default` as the Environment
         Select(driver.find_element(By.ID, "id_environment")).select_by_visible_text("Development")
         # upload scan file
-        file_path = os.path.join(dir_path, "zap_sample.xml")
-        driver.find_element(By.NAME, "file").send_keys(file_path)
+        file_path = dir_path / "zap_sample.xml"
+        driver.find_element(By.NAME, "file").send_keys(str(file_path))
         # Click Submit button
         with WaitForPageLoad(driver, timeout=50):
             driver.find_elements(By.CSS_SELECTOR, "button.btn.btn-primary")[1].click()

tests/ibm_appscan_test.py

@@ -31,8 +31,8 @@ def test_import_ibm_app_scan_result(self):
         # Select `Default` as the Environment
         Select(driver.find_element(By.ID, "id_environment")).select_by_visible_text("Development")
         # Upload Scan result file
-        scanner_file = os.path.join(dir_path, "ibm_appscan_xml_file.xml")
-        driver.find_element(By.NAME, "file").send_keys(scanner_file)
+        scanner_file = dir_path / "ibm_appscan_xml_file.xml"
+        driver.find_element(By.NAME, "file").send_keys(str(scanner_file))
         # click on upload button
         driver.find_elements(By.CSS_SELECTOR, "button.btn.btn-primary")[1].click()
         # Query the site to determine if the finding has been added