Skip to content

Commit 60f4520

Browse files
committed
feat: simplify logging in the dispatch
1 parent cfe9f23 commit 60f4520

File tree

1 file changed

+13
-27
lines changed

1 file changed

+13
-27
lines changed

.github/workflows/android-regression.yml

Lines changed: 13 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -51,30 +51,15 @@ on:
5151
- '2'
5252
default: '0'
5353

54-
PRINT_FAILED_TEST_LOGS:
55-
description: 'print failed test logs (1 to enable) - DONT DO FOR FULL REGRESSION (it crashes github)'
56-
required: true
57-
type: choice
58-
options:
59-
- '0'
60-
- '1'
61-
default: '0'
62-
PRINT_ONGOING_TEST_LOGS:
63-
description: 'print ongoing test logs (1 to enable) - DONT DO FOR FULL REGRESSION (it crashes github)'
64-
required: true
65-
type: choice
66-
options:
67-
- '0'
68-
- '1'
69-
default: '0'
70-
HIDE_WEBDRIVER_LOGS:
71-
description: 'print webdriver logs (1 to hide, 0 to show). PRINT_ONGOING_TEST_LOGS or PRINT_FAILED_TEST_LOGS must be 1'
72-
required: true
54+
LOG_LEVEL:
55+
description: 'Test logging verbosity'
56+
required: false
7357
type: choice
7458
options:
75-
- '0'
76-
- '1'
77-
default: '1'
59+
- 'minimal' # No test logs
60+
- 'failures' # Only failed test logs
61+
- 'verbose' # All test logs
62+
default: 'minimal'
7863

7964
jobs:
8065
android-regression:
@@ -93,8 +78,9 @@ jobs:
9378
IOS_APP_PATH_PREFIX: '<just_not_empty>'
9479
ANDROID_APK: './extracted/session-android.apk'
9580
PLAYWRIGHT_RETRIES_COUNT: ${{ github.event.inputs.PLAYWRIGHT_RETRIES_COUNT }}
96-
PRINT_FAILED_TEST_LOGS: ${{ github.event.inputs.PRINT_FAILED_TEST_LOGS }}
97-
PRINT_ONGOING_TEST_LOGS: ${{ github.event.inputs.PRINT_ONGOING_TEST_LOGS }}
81+
PRINT_FAILED_TEST_LOGS: ${{ github.event.inputs.LOG_LEVEL == 'failures' || github.event.inputs.LOG_LEVEL == 'verbose' ? '1' : '0' }}
82+
PRINT_ONGOING_TEST_LOGS: ${{ github.event.inputs.LOG_LEVEL == 'verbose' ? '1' : '0' }}
83+
HIDE_WEBDRIVER_LOGS: ${{ github.event.inputs.LOG_LEVEL == 'minimal' ? '1' : '0' }}
9884
IOS_1_SIMULATOR: '<just_not_empty>'
9985
IOS_2_SIMULATOR: '<just_not_empty>'
10086
IOS_3_SIMULATOR: '<just_not_empty>'
@@ -206,7 +192,7 @@ jobs:
206192
RISK: ${{ github.event.inputs.RISK }}
207193

208194
- name: Run 1-device tests
209-
uses: ./github/actions/run-playwright-tests
195+
uses: ./github/actions/run-tests
210196
with:
211197
PLATFORM: ${{ env.PLATFORM }}
212198
RISK: ${{ github.event.inputs.RISK }}
@@ -218,7 +204,7 @@ jobs:
218204
UPLOAD_IDENTIFIER: 'devices-1-test-run'
219205

220206
- name: Run 2-device tests
221-
uses: ./github/actions/run-playwright-tests
207+
uses: ./github/actions/run-tests
222208
with:
223209
PLATFORM: ${{ env.PLATFORM }}
224210
RISK: ${{ github.event.inputs.RISK }}
@@ -230,7 +216,7 @@ jobs:
230216
UPLOAD_IDENTIFIER: 'devices-2-test-run'
231217

232218
- name: Run 3-device tests
233-
uses: ./github/actions/run-playwright-tests
219+
uses: ./github/actions/run-tests
234220
with:
235221
PLATFORM: ${{ env.PLATFORM }}
236222
RISK: ${{ github.event.inputs.RISK }}

0 commit comments

Comments
 (0)