ci: Non-safely fix python files

Uses ruff with --unsafe-fixes to fix python files in this folder
This is a separate commit so it can be reverted if it causes
problems

Signed-off-by: Jamie McCrae <jamie.mccrae@nordicsemi.no>

Author: nordicjm

ci/fih_test_docker/generate_test_report.py

@@ -27,11 +27,11 @@ def print_results(results):
     test_with_skip = test_stats[CATEGORIES['SUCCESS']] - test_stats[CATEGORIES['ADDRES_NOEXEC']]
     print("    {:s}: ({:d}):".format(CATEGORIES['SKIPPED'], test_with_skip))
     print("    {:s} ({:d}):".format(CATEGORIES['NO_BOOT'], test_with_skip - test_stats[CATEGORIES['BOOT']]))
-    for last_line in failed_boot_last_lines.keys():
+    for last_line in failed_boot_last_lines:
         print(f"        last line: {last_line:s} ({failed_boot_last_lines[last_line]:d})")
     print("    {:s} ({:d})".format(CATEGORIES['BOOT'], test_stats[CATEGORIES['BOOT']]))
     print("{:s} ({:d}):".format(CATEGORIES['FAILED'], test_stats[CATEGORIES['TOTAL']] - test_stats[CATEGORIES['SUCCESS']]))
-    for reason in exec_fail_reasons.keys():
+    for reason in exec_fail_reasons:
         print(f"    {reason:s} ({exec_fail_reasons[reason]:d})")
 
 

ci/fih_test_docker/utils.py

@@ -48,7 +48,7 @@ def parse_yaml_file(filepath):
         if test["test_exec_ok"]:
             test_stats.update([CATEGORIES['SUCCESS']])
 
-            if "skipped" in test.keys() and not test["skipped"]:
+            if "skipped" in test and not test["skipped"]:
                 # The debugger didn't stop at this address
                 test_stats.update([CATEGORIES['ADDRES_NOEXEC']])
                 continue