Skip to content

Commit d79eb90

Browse files
author
Konrad Dysput
committed
Conditional smoketest execution
1 parent ddc10d5 commit d79eb90

File tree

2 files changed

+76
-56
lines changed

2 files changed

+76
-56
lines changed

‎.github/actions/smoke-tests/action.yml‎

Lines changed: 48 additions & 40 deletions
Original file line numberDiff line numberDiff line change
@@ -1,47 +1,55 @@
11
name: Smoke test
2-
description: "Execute smoke tests"
2+
description: 'Execute smoke tests'
33

44
inputs:
5-
submit-url:
6-
description: 'Submit layer url'
7-
required: true
8-
direct-submit-url:
9-
description: 'Direct submission URL to Backtrace instance'
10-
required: true
11-
sauce-username:
12-
description: 'Sauce labs user name'
13-
required: true
14-
sauce-access-key:
15-
description: 'Sauce labs secret'
16-
required: true
5+
submit-url:
6+
description: 'Submit layer url'
7+
required: true
8+
direct-submit-url:
9+
description: 'Direct submission URL to Backtrace instance'
10+
required: true
11+
sauce-username:
12+
description: 'Sauce labs user name'
13+
required: true
14+
sauce-access-key:
15+
description: 'Sauce labs secret'
16+
required: true
17+
ignore-browser:
18+
description: 'Ignore browser smoke-tests'
19+
required: false
20+
default: 'false'
21+
ignore-node:
22+
description: 'Ignore node smoke-tests'
23+
required: false
24+
default: 'false'
1725

1826
runs:
19-
using: "composite"
20-
steps:
21-
- uses: dorny/paths-filter@v3
22-
id: filter
23-
with:
24-
filters: |
25-
browser:
26-
- 'packages/browser/**'
27-
- 'packages/react/**'
28-
- 'packages/sdk-core/**'
29-
node:
30-
- 'packages/sdk-core/**'
31-
- 'packages/node/**'
27+
using: 'composite'
28+
steps:
29+
- uses: dorny/paths-filter@v3
30+
id: filter
31+
with:
32+
filters: |
33+
browser:
34+
- 'packages/browser/**'
35+
- 'packages/react/**'
36+
- 'packages/sdk-core/**'
37+
node:
38+
- 'packages/sdk-core/**'
39+
- 'packages/node/**'
3240
33-
- run: npm run smoketest:node
34-
shell: bash
35-
if: steps.filter.outputs.node == 'true'
36-
env:
37-
SMOKETESTS_SUBMIT_LAYER_URL: ${{ inputs.submit-url }}
38-
SMOKETESTS_DIRECT_SUBMIT_URL: ${{ inputs.direct-submit-url }}
41+
- run: npm run smoketest:node
42+
shell: bash
43+
if: steps.filter.outputs.node == 'true' && inputs.ignore-node == 'false'
44+
env:
45+
SMOKETESTS_SUBMIT_LAYER_URL: ${{ inputs.submit-url }}
46+
SMOKETESTS_DIRECT_SUBMIT_URL: ${{ inputs.direct-submit-url }}
3947

40-
- run: npm run smoketest:browser
41-
shell: bash
42-
if: steps.filter.outputs.browser == 'true'
43-
env:
44-
SMOKETESTS_SUBMIT_LAYER_URL: ${{ inputs.submit-url }}
45-
SMOKETESTS_DIRECT_SUBMIT_URL: ${{ inputs.direct-submit-url }}
46-
SMOKETESTS_SAUCE_USERNAME: ${{ inputs.sauce-username }}
47-
SMOKETESTS_SAUCE_ACCESS_KEY: ${{ inputs.sauce-access-key }}
48+
- run: npm run smoketest:browser
49+
shell: bash
50+
if: steps.filter.outputs.browser == 'true' && inputs.ignore-browser == 'false'
51+
env:
52+
SMOKETESTS_SUBMIT_LAYER_URL: ${{ inputs.submit-url }}
53+
SMOKETESTS_DIRECT_SUBMIT_URL: ${{ inputs.direct-submit-url }}
54+
SMOKETESTS_SAUCE_USERNAME: ${{ inputs.sauce-username }}
55+
SMOKETESTS_SAUCE_ACCESS_KEY: ${{ inputs.sauce-access-key }}

‎.github/workflows/test.yml‎

Lines changed: 28 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -27,10 +27,10 @@ jobs:
2727
- name: smoke-test
2828
uses: ./.github/actions/smoke-tests
2929
with:
30-
submit-url: ${{ secrets.SMOKETESTS_SUBMIT_LAYER_URL }}
31-
direct-submit-url: ${{ secrets.SMOKETESTS_DIRECT_SUBMIT_URL }}
32-
sauce-username: ${{ secrets.SMOKETESTS_SAUCE_USERNAME }}
33-
sauce-access-key: ${{ secrets.SMOKETESTS_SAUCE_ACCESS_KEY }}
30+
submit-url: ${{ secrets.SMOKETESTS_SUBMIT_LAYER_URL }}
31+
direct-submit-url: ${{ secrets.SMOKETESTS_DIRECT_SUBMIT_URL }}
32+
sauce-username: ${{ secrets.SMOKETESTS_SAUCE_USERNAME }}
33+
sauce-access-key: ${{ secrets.SMOKETESTS_SAUCE_ACCESS_KEY }}
3434

3535
test_linux:
3636
runs-on: ubuntu-latest
@@ -53,10 +53,14 @@ jobs:
5353
with:
5454
node-version: ${{ matrix.node-version }}
5555
- run: npm test
56-
- run: npm run smoketest:node
57-
env:
58-
SMOKETESTS_SUBMIT_LAYER_URL: ${{ secrets.SMOKETESTS_SUBMIT_LAYER_URL }}
59-
SMOKETESTS_DIRECT_SUBMIT_URL: ${{ secrets.SMOKETESTS_DIRECT_SUBMIT_URL }}
56+
- name: smoke-test
57+
uses: ./.github/actions/smoke-tests
58+
with:
59+
submit-url: ${{ secrets.SMOKETESTS_SUBMIT_LAYER_URL }}
60+
direct-submit-url: ${{ secrets.SMOKETESTS_DIRECT_SUBMIT_URL }}
61+
sauce-username: ${{ secrets.SMOKETESTS_SAUCE_USERNAME }}
62+
sauce-access-key: ${{ secrets.SMOKETESTS_SAUCE_ACCESS_KEY }}
63+
ignore-browser: 'true'
6064

6165
test_linux_pre_workspaces:
6266
runs-on: ubuntu-latest
@@ -80,10 +84,14 @@ jobs:
8084
node-version: ${{ matrix.node-version }}
8185
- run: npm i -g npm@8
8286
- run: npm test
83-
- run: npm run smoketest:node
84-
env:
85-
SMOKETESTS_SUBMIT_LAYER_URL: ${{ secrets.SMOKETESTS_SUBMIT_LAYER_URL }}
86-
SMOKETESTS_DIRECT_SUBMIT_URL: ${{ secrets.SMOKETESTS_DIRECT_SUBMIT_URL }}
87+
- name: smoke-test
88+
uses: ./.github/actions/smoke-tests
89+
with:
90+
submit-url: ${{ secrets.SMOKETESTS_SUBMIT_LAYER_URL }}
91+
direct-submit-url: ${{ secrets.SMOKETESTS_DIRECT_SUBMIT_URL }}
92+
sauce-username: ${{ secrets.SMOKETESTS_SAUCE_USERNAME }}
93+
sauce-access-key: ${{ secrets.SMOKETESTS_SAUCE_ACCESS_KEY }}
94+
ignore-browser: 'true'
8795

8896
test_windows:
8997
runs-on: windows-latest
@@ -101,7 +109,11 @@ jobs:
101109
- run: npm ci
102110
- run: npm run build
103111
- run: npm test
104-
- run: npm run smoketest:node
105-
env:
106-
SMOKETESTS_SUBMIT_LAYER_URL: ${{ secrets.SMOKETESTS_SUBMIT_LAYER_URL }}
107-
SMOKETESTS_DIRECT_SUBMIT_URL: ${{ secrets.SMOKETESTS_DIRECT_SUBMIT_URL }}
112+
- name: smoke-test
113+
uses: ./.github/actions/smoke-tests
114+
with:
115+
submit-url: ${{ secrets.SMOKETESTS_SUBMIT_LAYER_URL }}
116+
direct-submit-url: ${{ secrets.SMOKETESTS_DIRECT_SUBMIT_URL }}
117+
sauce-username: ${{ secrets.SMOKETESTS_SAUCE_USERNAME }}
118+
sauce-access-key: ${{ secrets.SMOKETESTS_SAUCE_ACCESS_KEY }}
119+
ignore-browser: 'true'

0 commit comments

Comments
 (0)