Skip to content

Commit

Permalink
Merge pull request #150 from nhsuk/remove-build-action-from-test
Browse files Browse the repository at this point in the history
Remove build action from test script and document
  • Loading branch information
will-dyson-nhsd authored Jan 12, 2023
2 parents 66ea3ac + dfbb3d5 commit ffbc061
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 1 deletion.
3 changes: 3 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -96,6 +96,7 @@ expiry date of the cookie.

- `VERSION` the current version as defined in package.json

<a name="compiling"></a>
## Compiling

This project uses [Webpack](https://webpack.js.org/) and [Babel](https://babeljs.io/)
Expand Down Expand Up @@ -143,6 +144,8 @@ LOG_TO_SPLUNK=true npm run build:production

## Tests

The tests require the javascript to be compiled. See the [Compiling](#compiling) section above.

To run the [Jest](https://jestjs.io/en/) tests

```sh
Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
"build:production": "webpack --mode=production",
"lint": "eslint .",
"start": "npm run watch & npm run testserver",
"test": "npm run build && start-server-and-test testserver http://localhost:8080 test-all",
"test": "start-server-and-test testserver http://localhost:8080 test-all",
"test-all": "npm-run-all --continue-on-error test:*",
"test:unit": "jest --runInBand --coverage=true",
"test:integration": "jest -c tests/integration-tests/jest.config.js --coverage=true",
Expand Down

0 comments on commit ffbc061

Please sign in to comment.