Skip to content

Commit aa91f0f

Browse files
more fixes for test suite failures
1 parent 737ee15 commit aa91f0f

File tree

1 file changed

+23
-20
lines changed

1 file changed

+23
-20
lines changed

tools/patch.py

Lines changed: 23 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -30,30 +30,33 @@ def patch(article_path: str, results: dict, link: str):
3030
else:
3131
raise SystemExit("Unknown content path, pass learning paths or install guides only")
3232

33-
test_images = results.keys()
34-
results_values = defaultdict(lambda: "failed")
35-
results_values[0] = "passed"
33+
if results is not None:
34+
test_images = results.keys()
35+
results_values = defaultdict(lambda: "failed")
36+
results_values[0] = "passed"
3637

37-
content_data = data["sw_categories"][sw_category][content_title]
38+
content_data = data["sw_categories"][sw_category][content_title]
3839

39-
# Create 'tests_and_status' if it doesn't exist
40-
if "tests_and_status" not in content_data or not isinstance(content_data["tests_and_status"], list):
41-
content_data["tests_and_status"] = [{} for _ in range(len(test_images))]
40+
# Create 'tests_and_status' if it doesn't exist
41+
if "tests_and_status" not in content_data or not isinstance(content_data["tests_and_status"], list):
42+
content_data["tests_and_status"] = [{} for _ in range(len(test_images))]
4243

43-
# If 'tests_and_status' exists but is too short, extend it
44-
if len(content_data["tests_and_status"]) < len(test_images):
45-
additional_entries = [{} for _ in range(len(test_images) - len(content_data["tests_and_status"]))]
46-
content_data["tests_and_status"].extend(additional_entries)
44+
# If 'tests_and_status' exists but is too short, extend it
45+
if len(content_data["tests_and_status"]) < len(test_images):
46+
additional_entries = [{} for _ in range(len(test_images) - len(content_data["tests_and_status"]))]
47+
content_data["tests_and_status"].extend(additional_entries)
4748

48-
# Now safe to index
49-
for i, image in enumerate(test_images):
50-
idx = min(i, len(content_data["tests_and_status"]) - 1)
51-
content_data["tests_and_status"][idx][image] = results_values[results[image]]
49+
# Now safe to index
50+
for i, image in enumerate(test_images):
51+
idx = min(i, len(content_data["tests_and_status"]) - 1)
52+
content_data["tests_and_status"][idx][image] = results_values[results[image]]
5253

53-
if link:
54-
data["sw_categories"][sw_category][content_title]["test_link"] = link
54+
if link:
55+
data["sw_categories"][sw_category][content_title]["test_link"] = link
5556

5657

57-
with open(stats_file, mode='w') as f:
58-
yaml.dump(data, f)
59-
f.close()
58+
with open(stats_file, mode='w') as f:
59+
yaml.dump(data, f)
60+
f.close()
61+
else:
62+
pass

0 commit comments

Comments
 (0)