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.

Sign up for GitHub

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

Jump to bottom

Add CI for running tests #567

Closed
murny opened this issue Apr 25, 2024 · 3 comments
Closed

Add CI for running tests #567

murny opened this issue Apr 25, 2024 · 3 comments
Assignees
@murny

Comments

@murny
Copy link
Collaborator

murny commented Apr 25, 2024

We have no CI running on this project currently. It would be nice to have a simple test suite, which would just spin up the CMS and do some smoke tests, just to give us a bit of confidence going forward as we start working in this repo.
@murny murny self-assigned this Apr 25, 2024
@pgwillia
Copy link
Member

related to #522

@murny murny mentioned this issue May 28, 2024
Merged
@murny
Copy link
Collaborator Author

murny commented May 28, 2024 •
edited
Loading

Next steps:

  • #567 add test suite #573 adds a simple CI and tests ✅
  • Lets add more tests to cover "Staff" pages and maybe Admin login
  • Add rubocop

murny added a commit that referenced this issue May 28, 2024
@murny
Merge pull request #573 from ualbertalib/#567-add-test-suite
e6aeaba 
#567 add test suite
@murny
Copy link
Collaborator Author

murny commented Jul 15, 2024

Think we can close this.

We now have a decent coverage of smoke tests running in CI (tests for covering Admin, CMS, and profile code). CI is also checking for rubocop linter.

@murny murny closed this as completed Jul 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@murny murny

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@pgwillia @murny