Skip to content

Commit ae796b2

Browse files
Merge pull request #729 from mozilla/philimon/lean_l10n_reporting
Philimon/l10n_report_by_site_only
2 parents 923a011 + cd15832 commit ae796b2

File tree

1 file changed

+2
-10
lines changed

1 file changed

+2
-10
lines changed

modules/testrail_integration.py

Lines changed: 2 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -189,20 +189,12 @@ def reportable(platform_to_test=None):
189189
if os.environ.get("FX_L10N"):
190190
beta_version = int(minor_num.split("b")[-1])
191191
distributed_mappings = select_l10n_mappings(beta_version)
192-
expected_mappings = sum(map(lambda x: len(x), distributed_mappings.values()))
192+
expected_mappings = len(distributed_mappings)
193193
covered_mappings = 0
194194
# keeping this logic to still see how many mappings are reported.
195195
for entry in plan_entries:
196196
if entry.get("name") in distributed_mappings:
197-
site = entry.get("name")
198-
for run_ in entry.get("runs"):
199-
if run_.get("config"):
200-
run_region, run_platform = run_.get("config").split("-")
201-
if (
202-
run_region in distributed_mappings[site]
203-
and platform in run_platform
204-
):
205-
covered_mappings += 1
197+
covered_mappings += 1
206198
logging.warning(
207199
f"Potentially matching run found for {platform}, may be reportable. (Found {covered_mappings} site/region mappings reported, expected {expected_mappings}.)"
208200
)

0 commit comments

Comments
 (0)