Skip to content

Commit

Permalink
another option for testing on Windows
Browse files Browse the repository at this point in the history
  • Loading branch information
joneugster committed Aug 24, 2024
1 parent 6905b2f commit 995370b
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 11 deletions.
21 changes: 11 additions & 10 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -33,16 +33,14 @@ jobs:
name: Linux
- os: macos-latest
name: macOS
# currently only one Windows test
- os: windows-latest
name: Windows
browser: electron
# - os: windows-latest
# name: Windows
# browser: edge
# exclude:
# - os: windows-latest
# browser: webkit
- os: windows-latest
name: Windows
browser: edge
exclude:
- os: windows-latest
browser: webkit

name: ${{ matrix.name }} - ${{ matrix.browser }}
runs-on: ${{ matrix.os }}
Expand Down Expand Up @@ -76,7 +74,6 @@ jobs:
- uses: actions/setup-node@v4
- run: npm install
- run: npm run setup_demo
if: matrix.os != 'windows-latest'
- name: Run tests
if: matrix.os != 'windows-latest'
uses: cypress-io/github-action@v6
Expand All @@ -85,5 +82,9 @@ jobs:
wait-on: 'http://localhost:5173'
browser: ${{ matrix.browser }}
- name: Run test (Windows)
run: npm test
run: |
npx concurrently --hide 0 --kill-others \
"npm start" \
"npx wait-on http://localhost:5173 && \
cypress run --browser ${{ matrix.browser }}" -n server,cypress -s command-cypress
if: matrix.os == 'windows-latest'
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -202,7 +202,7 @@ You can run

```
npm install
npm run test
npm test
```

for the automated cypress tests.
Expand Down

0 comments on commit 995370b

Please sign in to comment.