Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create node.js.yml for npm test CI. #771

Merged
merged 1 commit into from
Aug 24, 2024

Conversation

jjsil
Copy link
Collaborator

@jjsil jjsil commented Aug 24, 2024

  • It checks out the code, sets up node, builds and runs the tests.
  • Run tests in ChromeHeadless and do a single run.
  • Run automatically after pushes and pull-requests in master and develop.

Needs to be merged after #770 to make sure the tests are green.
Resolves #740.

It checks out the code, sets up node, builds and runs the tests.
Run tests in ChromeHeadless and do a single run.
@jjsil jjsil requested a review from graphefruit August 24, 2024 11:45
@jjsil jjsil self-assigned this Aug 24, 2024
Copy link

@graphefruit
Copy link
Owner

Have pulled the dev repo and executed
npm test -- --browsers=ChromeHeadless --watch=false
->

 SUCCESS (0 secs / 2.609 secs)
Chrome Headless 127.0.6533.122 (Mac OS 10.15.7): Executed 161 of 162 (skipped 1)Chrome Headless 127.0.6533.122 (Mac OS 10.15.7): Executed 161 of 162 (skipped 1) SUCCESS (2.248 secs / 2.643 secs)
TOTAL: 161 SUCCESS
TOTAL: 161 SUCCESS

Looks good

@graphefruit graphefruit marked this pull request as ready for review August 24, 2024 18:27
@graphefruit graphefruit merged commit b6187ae into graphefruit:develop Aug 24, 2024
2 of 3 checks passed
@jjsil jjsil deleted the infra/test-github-action branch August 25, 2024 01:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants