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

Add CI workflows (testing & linting) #29

Merged
merged 28 commits into from
Nov 6, 2024
Merged

Conversation

xenosf
Copy link

@xenosf xenosf commented Nov 5, 2024

Add CI checks using GitHub Actions. These checks run for every PR and push to the main branch. All tests except basic linting and system testing run on windows, mac, ubuntu.

Checks:

  • Linting: Uses eslint to perform basic static checks
    • NOTE I had to modify some of the existing code to comply with the linting output. May need to fix other code to pass tests before merging this PR
  • Client: Automated snapshot/unit testing using Jest
  • Server: Automated unit testing using Jest for each service
  • System: Automated system/integration testing using Selenium WebDriver on Chrome and Firefox

Not all unit/integration/system test cases have been written yet, this PR mainly sets up the CI workflow

@xenosf xenosf changed the title Add testing Add CI action for linting Nov 5, 2024
@xenosf xenosf changed the title Add CI action for linting Add CI actions (testing & linting) Nov 5, 2024
@xenosf xenosf changed the title Add CI actions (testing & linting) Add CI workflows (testing & linting) Nov 5, 2024
@xenosf xenosf marked this pull request as ready for review November 5, 2024 21:49
@xenosf xenosf mentioned this pull request Nov 6, 2024
@xenosf xenosf merged commit f9364c5 into CS3219-AY2425S1:main Nov 6, 2024
21 checks passed
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.

1 participant