From 6831a06066e7bd51da4196ce4ddc6e4be23c5b78 Mon Sep 17 00:00:00 2001 From: a-maurice Date: Mon, 15 Dec 2025 15:04:21 -0800 Subject: [PATCH 1/2] Fix build report error --- scripts/gha/summarize_test_results.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/scripts/gha/summarize_test_results.py b/scripts/gha/summarize_test_results.py index aafafdd23e..c3b996b324 100644 --- a/scripts/gha/summarize_test_results.py +++ b/scripts/gha/summarize_test_results.py @@ -387,7 +387,7 @@ def combine_config(platform, config, config_value, k): # if certain config failed for all values, add message "All *" if len(config_value) > 1 and len(config) == len(config_value): config = ["All %d %s" % (len(config_value), config_name)] - elif config_name == "ios_device": + elif config_name == "ios_device" and get_test_device(device): ftl_devices = set(filter(lambda device: get_test_device(device).get("type") in "ftl", config_value)) simulators = set(filter(lambda device: get_test_device(device).get("type") in "virtual", config_value)) if len(ftl_devices) > 1 and ftl_devices.issubset(set(config)): @@ -396,7 +396,7 @@ def combine_config(platform, config, config_value, k): if len(simulators) > 1 and simulators.issubset(set(config)): config.insert(0, "All %d Local Simulators" % len(simulators)) config = [x for x in config if (x not in simulators)] - elif config_name == "android_device": + elif config_name == "android_device" and get_test_device(device): ftl_devices = set(filter(lambda device: get_test_device(device).get("type") in "ftl", config_value)) emulators = set(filter(lambda device: get_test_device(device).get("type") in "virtual", config_value)) if len(ftl_devices) > 1 and ftl_devices.issubset(set(config)): From bede503804fafd1d0d563d5fd211e4f6420d30fb Mon Sep 17 00:00:00 2001 From: a-maurice Date: Mon, 15 Dec 2025 16:21:19 -0800 Subject: [PATCH 2/2] Update summarize_test_results.py --- scripts/gha/summarize_test_results.py | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/scripts/gha/summarize_test_results.py b/scripts/gha/summarize_test_results.py index c3b996b324..c03cb2cdec 100644 --- a/scripts/gha/summarize_test_results.py +++ b/scripts/gha/summarize_test_results.py @@ -387,18 +387,18 @@ def combine_config(platform, config, config_value, k): # if certain config failed for all values, add message "All *" if len(config_value) > 1 and len(config) == len(config_value): config = ["All %d %s" % (len(config_value), config_name)] - elif config_name == "ios_device" and get_test_device(device): - ftl_devices = set(filter(lambda device: get_test_device(device).get("type") in "ftl", config_value)) - simulators = set(filter(lambda device: get_test_device(device).get("type") in "virtual", config_value)) + elif config_name == "ios_device": + ftl_devices = set(filter(lambda device: get_test_device(device) and get_test_device(device).get("type") in "ftl", config_value)) + simulators = set(filter(lambda device: get_test_device(device) and get_test_device(device).get("type") in "virtual", config_value)) if len(ftl_devices) > 1 and ftl_devices.issubset(set(config)): config.insert(0, "All %d FTL Devices" % len(ftl_devices)) config = [x for x in config if (x not in ftl_devices)] if len(simulators) > 1 and simulators.issubset(set(config)): config.insert(0, "All %d Local Simulators" % len(simulators)) config = [x for x in config if (x not in simulators)] - elif config_name == "android_device" and get_test_device(device): - ftl_devices = set(filter(lambda device: get_test_device(device).get("type") in "ftl", config_value)) - emulators = set(filter(lambda device: get_test_device(device).get("type") in "virtual", config_value)) + elif config_name == "android_device": + ftl_devices = set(filter(lambda device: get_test_device(device) and get_test_device(device).get("type") in "ftl", config_value)) + emulators = set(filter(lambda device: get_test_device(device) and get_test_device(device).get("type") in "virtual", config_value)) if len(ftl_devices) > 1 and ftl_devices.issubset(set(config)): config.insert(0, "All %d FTL Devices" % len(ftl_devices)) config = [x for x in config if (x not in ftl_devices)]