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

chore: add github actions to add automated tests #46

Merged
merged 1 commit into from
Jan 24, 2025

Conversation

AtofStryker
Copy link
Contributor

@AtofStryker AtofStryker commented Jan 17, 2025

adds github actions to the repo to get some level of automated tests running

There are two aspects to every project:

  • Component tests - run the given component tests in the project to make sure everything is passing
  • E2E smoke tests - runs the sample app (which uses the bundler config that the given CT framework is using) and makes sure the sample app works and users can try the sample login

I noticed msw was missing from one of the angular projects so I added it. I also updated the package-lock with the correct Cypress 14 as it look like something went awry there. Each project is also configured to run on port 4200 with a start command to make the github action matrix simple

@AtofStryker AtofStryker force-pushed the chore/setup_actions branch 8 times, most recently from c6a5286 to 5306d53 Compare January 17, 2025 16:14
@AtofStryker AtofStryker self-assigned this Jan 17, 2025
@jennifer-shehane
Copy link
Member

@AtofStryker Might want to require these status checks after this merges. I wouldn't let me do it before merge.

@AtofStryker AtofStryker merged commit 807e768 into main Jan 24, 2025
10 checks passed
@AtofStryker AtofStryker deleted the chore/setup_actions branch January 24, 2025 19:44
@AtofStryker
Copy link
Contributor Author

@AtofStryker Might want to require these status checks after this merges. I wouldn't let me do it before merge.

required checks are now added and can be seen in #47

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