Skip to content

Commit 06a9b94

Browse files
authored
test: disabling visual regression testing in favor of manual running
1 parent a56f268 commit 06a9b94

File tree

4 files changed

+88
-4
lines changed

4 files changed

+88
-4
lines changed

.github/workflows/automerge.yml

Lines changed: 42 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
name: automerge
1+
name: Automerge PR
22
on:
33
pull_request:
44
types: [labeled, unlabeled, synchronize, edited, unlocked]
@@ -8,10 +8,49 @@ on:
88
types: [completed]
99
status: success
1010
jobs:
11-
automerge:
11+
visual_regression_tests:
12+
if: contains(github.event.pull_request.labels.*.name, 'ready to merge')
1213
runs-on: ubuntu-latest
14+
strategy:
15+
matrix:
16+
node-version: [12.x]
1317
steps:
14-
- name: automerge
18+
- uses: actions/checkout@v2
19+
20+
- name: Cache node modules
21+
uses: actions/cache@v2
22+
env:
23+
cache-name: cache-node-modules
24+
with:
25+
# npm cache files are stored in `~/.npm` on Linux/macOS
26+
path: ~/.npm
27+
key: ${{ runner.os }}-build-${{ env.cache-name }}-${{ hashFiles('**/package-lock.json') }}
28+
restore-keys: |
29+
${{ runner.os }}-build-${{ env.cache-name }}-
30+
${{ runner.os }}-build-
31+
${{ runner.os }}-
32+
33+
- name: Install Dependencies
34+
run: npm install
35+
36+
- name: Build
37+
run: npm build
38+
39+
- name: Visual regression tests
40+
uses: ianwalter/[email protected]
41+
with:
42+
args: npm run e2e -- --verbose
43+
env:
44+
BROWSERSTACK_ACCESS_KEY: ${{ secrets.BROWSERSTACK_KEY }}
45+
BROWSERSTACK_USERNAME: ${{ secrets.BROWSERSTACK_USER }}
46+
CI: true
47+
firefox: latest
48+
chrome: stable
49+
try_to_merge:
50+
if: success() && contains(github.event.pull_request.labels.*.name, 'ready to merge')
51+
runs-on: ubuntu-latest
52+
steps:
53+
- name: Try to automerge PR ${{ github.event.number }}
1554
uses: "pascalgn/automerge-action@4536e8847eb62fe2f0ee52c8fa92d17aa97f932f"
1655
env:
1756
GITHUB_TOKEN: "${{ secrets.ACTIONS }}"

.github/workflows/visualtests.yml

Lines changed: 41 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,41 @@
1+
name: Visual regression testing
2+
on: [workflow_dispatch]
3+
jobs:
4+
visual_regression_tests:
5+
if: contains(github.event.pull_request.labels.*.name, 'ready to merge')
6+
runs-on: ubuntu-latest
7+
strategy:
8+
matrix:
9+
node-version: [12.x]
10+
steps:
11+
- uses: actions/checkout@v2
12+
13+
- name: Cache node modules
14+
uses: actions/cache@v2
15+
env:
16+
cache-name: cache-node-modules
17+
with:
18+
# npm cache files are stored in `~/.npm` on Linux/macOS
19+
path: ~/.npm
20+
key: ${{ runner.os }}-build-${{ env.cache-name }}-${{ hashFiles('**/package-lock.json') }}
21+
restore-keys: |
22+
${{ runner.os }}-build-${{ env.cache-name }}-
23+
${{ runner.os }}-build-
24+
${{ runner.os }}-
25+
26+
- name: Install Dependencies
27+
run: npm install
28+
29+
- name: Build
30+
run: npm build
31+
32+
- name: Visual regression tests
33+
uses: ianwalter/[email protected]
34+
with:
35+
args: npm run e2e -- --verbose
36+
env:
37+
BROWSERSTACK_ACCESS_KEY: ${{ secrets.BROWSERSTACK_KEY }}
38+
BROWSERSTACK_USERNAME: ${{ secrets.BROWSERSTACK_USER }}
39+
CI: true
40+
firefox: latest
41+
chrome: stable

.travis.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ addons:
1212
access_key: BROWSERSTACK_KEY
1313
script:
1414
- npm test -- --verbose
15-
- npm run e2e -- --verbose
15+
# - npm run e2e -- --verbose
1616
cache:
1717
directories:
1818
- node_modules

CHANGELOG-prerelease.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,7 @@
1+
## Prerelease 56 ( TBD )
2+
3+
- [](https://github.com/patternfly/patternfly-elements/commit/) fix: move visual regression testing to merge script
4+
15
## Prerelease 55 ( 2020-08-19 )
26

37
- [3ee117e](https://github.com/patternfly/patternfly-elements/commit/3ee117e8a3df17f50738b9c0fa0caa3a9af9c679) feat: pfe-autocomplete - fire custom event when dropdown is shown (#1046)

0 commit comments

Comments
 (0)