arno/trezor-firmware
1
0
Fork 0
mirror of https://github.com/trezor/trezor-firmware.git synced 2024-11-19 05:58:09 +00:00
Code Issues Releases Wiki Activity

fix(tests): hotfix - fix update_fixtures ci --remove-missing for lang awareness

Browse Source 
but the correct fix is to make langs a separate group
This commit is contained in:
matejcik 2024-06-06 11:38:01 +02:00 committed by matejcik
parent 2bf2549a31
commit b868ccbbdd
1 changed files with 4 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
tests/update_fixtures.py
View File

@ -97,7 +97,9 @@ def ci(
is_error = True is_error = True
print("No results found.") print("No results found.")
continue continue
_model, lang, _job = job_name.split("-")
model = next(iter(ui_res_dict.keys())) model = next(iter(ui_res_dict.keys()))
assert model == _model
group = next(iter(ui_res_dict[model].keys())) group = next(iter(ui_res_dict[model].keys()))
current_model = current_fixtures.setdefault(model, {}) current_model = current_fixtures.setdefault(model, {})
current_group = current_model.setdefault(group, {}) # type: ignore current_group = current_model.setdefault(group, {}) # type: ignore
@ -106,6 +108,8 @@ def ci(
# get rid of tests that were not run in CI # get rid of tests that were not run in CI
removed = 0 removed = 0
for key in list(current_group.keys()): for key in list(current_group.keys()):
if not key.startswith(f"{model}_{lang}_"):
continue
if key not in ui_res_dict[model][group]: if key not in ui_res_dict[model][group]:
current_group.pop(key) current_group.pop(key)
removed += 1 removed += 1