Skip to content

Commit 0ed289c

Browse files
philimon/l10n_report_fix (#702)
* fix for l10n report * logging * remove test script
1 parent 9e55a43 commit 0ed289c

File tree

1 file changed

+7
-7
lines changed

1 file changed

+7
-7
lines changed

modules/testrail_integration.py

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -196,22 +196,22 @@ def reportable(platform_to_test=None):
196196
# keeping this logic to still see how many mappings are reported.
197197
for entry in plan_entries:
198198
if entry.get("name") in distributed_mappings:
199-
report = False
200199
site = entry.get("name")
201200
for run_ in entry.get("runs"):
202201
if run_.get("config"):
203202
run_region, run_platform = run_.get("config").split("-")
204-
covered_mappings += (
205-
1
206-
if run_region in distributed_mappings[site]
203+
if (
204+
run_region in distributed_mappings[site]
207205
and platform in run_platform
208-
else 0
209-
)
206+
):
207+
covered_mappings += 1
208+
report = False
210209
logging.warning(
211210
f"Potentially matching run found for {platform}, may be reportable. (Found {covered_mappings} site/region mappings reported.)"
212211
)
212+
logging.warning(f"Run is reportable: {report}")
213213
# Only report when there is a new beta and no other site/region mappings are reported.
214-
return report or covered_mappings == 0
214+
return report
215215
else:
216216
covered_suites = 0
217217
for entry in plan_entries:

0 commit comments

Comments
 (0)