squash commit (#11988)

Author: manuel-sommer

dojo/tools/blackduck_component_risk/parser.py

@@ -302,8 +302,7 @@ def security_impact(self, vulns):
         """
         max_impact = 0.0
         for vuln in vulns:
-            if float(vuln["Impact"]) > max_impact:
-                max_impact = float(vuln["Impact"])
+            max_impact = max(max_impact, float(vuln["Impact"]))
         return max_impact
 
     def security_references(self, vulns):

dojo/utils.py

@@ -2625,10 +2625,8 @@ def get_open_findings_burndown(product):
                         info_count -= 1
 
         f_day = [critical_count, high_count, medium_count, low_count, info_count]
-        if min(f_day) < running_min:
-            running_min = min(f_day)
-        if max(f_day) > running_max:
-            running_max = max(f_day)
+        running_min = min(running_min, *f_day)
+        running_max = max(running_max, *f_day)
 
         past_90_days["Critical"].append([d_start * 1000, critical_count])
         past_90_days["High"].append([d_start * 1000, high_count])

ruff.toml

@@ -76,7 +76,7 @@ select = [
    "PGH",
    "PLC01", "PLC0205", "PLC0208", "PLC0414", "PLC18", "PLC24", "PLC28", "PLC3",
    "PLE",
-   "PLR01", "PLR02", "PLR04", "PLR0915", "PLR1716", "PLR172", "PLR1733", "PLR1736", "PLR2044", "PLR5", "PLR6104", "PLR6201",
+   "PLR01", "PLR02", "PLR04", "PLR0915", "PLR1716", "PLR172", "PLR173", "PLR2044", "PLR5", "PLR6104", "PLR6201",
    "PLW01", "PLW02", "PLW04", "PLW0604", "PLW07", "PLW1", "PLW2", "PLW3",
    "TRY003", "TRY004", "TRY2", "TRY300", "TRY401",
    "FLY",

tests/base_test_class.py

@@ -477,8 +477,7 @@ def get_log(self, log_type):
 
                 # save the last timestamp only if newer
                 # in this set of logs
-                if entry["timestamp"] > last_timestamp:
-                    last_timestamp = entry["timestamp"]
+                last_timestamp = max(last_timestamp, entry["timestamp"])
 
         # store the very last timestamp
         self.last_timestamp = last_timestamp