Skip to content

Commit 54f73cb

Browse files
committed
Try stripping the ftl result json
1 parent ac2afbe commit 54f73cb

2 files changed

Lines changed: 3 additions & 3 deletions

File tree

.github/workflows/integration_tests.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1056,7 +1056,7 @@ jobs:
10561056
if: ${{ steps.device-info.outputs.device_type == 'ftl' && !cancelled() }}
10571057
shell: bash
10581058
run: |
1059-
python scripts/gha/read_ftl_test_result.py --test_result ${STEPS_FTL_TEST_OUTPUTS_TEST_SUMMARY} \
1059+
python scripts/gha/read_ftl_test_result.py --test_result '${STEPS_FTL_TEST_OUTPUTS_TEST_SUMMARY}' \
10601060
--output_path testapps/test-results-android-${MATRIX_BUILD_OS}-${MATRIX_ANDROID_DEVICE}-${{ matrix.test_type }}.log
10611061
env:
10621062
STEPS_FTL_TEST_OUTPUTS_TEST_SUMMARY: ${{ steps.ftl_test.outputs.test_summary }}
@@ -1247,7 +1247,7 @@ jobs:
12471247
if: ${{ steps.device-info.outputs.device_type == 'ftl' && !cancelled() }}
12481248
shell: bash
12491249
run: |
1250-
python scripts/gha/read_ftl_test_result.py --test_result ${STEPS_FTL_TEST_OUTPUTS_TEST_SUMMARY} \
1250+
python scripts/gha/read_ftl_test_result.py --test_result '${STEPS_FTL_TEST_OUTPUTS_TEST_SUMMARY}' \
12511251
--output_path testapps/test-results-ios-${{ matrix.build_os }}-${MATRIX_IOS_DEVICE}-${{ matrix.test_type }}.log
12521252
env:
12531253
STEPS_FTL_TEST_OUTPUTS_TEST_SUMMARY: ${{ steps.ftl_test.outputs.test_summary }}

scripts/gha/read_ftl_test_result.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@ def main(argv):
5656
if len(argv) > 1:
5757
raise app.UsageError("Too many command-line arguments.")
5858

59-
test_result = json.loads(FLAGS.test_result)
59+
test_result = json.loads(FLAGS.test_result.strip("'"))
6060
tests = []
6161
for app in test_result.get("apps"):
6262
app_path = app.get("testapp_path")

0 commit comments

Comments
 (0)