Skip to content

Commit

Permalink
Tests: skip web server test in CI, as it often fails
Browse files Browse the repository at this point in the history
I suspect a race condition, but cannot see where it could be.

Hence, add a new npm script ‘test-ci’ for CI, which skips this test.

Also, set the timeout for AVA to 300s globally.
  • Loading branch information
rrthomas committed Sep 28, 2024
1 parent e4d36f6 commit da6d1da
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,4 +15,4 @@ jobs:
- run: git clone https://github.com/rrthomas/topiary ~/topiary && cd ~/topiary && cargo install --path topiary-cli
- run: |
npm install
npm run test-all
npm run test-ci
5 changes: 3 additions & 2 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,8 @@
"update-ohm-patch": "npm run generate-only && npm run patch-diff-ohm && diff -u src/grammar/ursa.ohm-bundle.d.ts src/grammar/ursa.ohm-bundle.d.part-patched.ts > src/grammar/ursa.ohm-bundle.d.ts.diff; npm run patch-ohm",
"compile-prelude": "npm run pre-compile-prelude && ./bin/test-run.sh compile --output=src/ark/prelude.json src/ursa/prelude.ursa && ./bin/test-run.sh --target=js compile --output=src/ark/compiler/prelude.js src/ursa/prelude.ursa",
"test": "npm run generate && npm run build && ava 'src/**/basics.test.ts' 'src/**/examples.test.ts' 'src/**/cli.test.ts' 'src/**/fsmap.test.ts'",
"test-all": "npm run generate && npm run build && ava --timeout 300s",
"test-all": "npm run generate && npm run build && ava",
"test-ci": "npm run generate && npm run build && ava --match '!Web server'",
"coverage": "NODE_ENV=coverage c8 --extension .ts npm test",
"loc": "cloc --read-lang-def=ursa.def --exclude-ext=js --force-lang='JavaScript,ohm' --not-match-f='testutil.ts|\\.test\\.ts|\\.d.*\\.ts$|prelude\\.js.*' src",
"diff-check": "git diff --exit-code",
Expand All @@ -92,7 +93,7 @@
},
"compile": "tsc"
},
"timeout": "60s",
"timeout": "300s",
"workerThreads": false
},
"pre-push": [
Expand Down

0 comments on commit da6d1da

Please sign in to comment.