Skip to content

Test Automation#44

Open
Kelvin205 wants to merge 4 commits into
notprowler:mainfrom
Kelvin205:test-automation
Open

Test Automation#44
Kelvin205 wants to merge 4 commits into
notprowler:mainfrom
Kelvin205:test-automation

Conversation

@Kelvin205
Copy link
Copy Markdown
Collaborator

Title

Test Automation

Summary

This is an assignment do not merge.
This PR includes unit tests for utility functions, integration tests for the signup page, and an end-to-end test using Cypress. Test automation is handled through a GitHub Actions workflow triggered on pull requests.

Note: The Cypress E2E test currently fails due to configuration issues in package.json or other files.
However, the unit tests and integration test are legit. Problems encountered getting cypress to work due to incorrect test setup.
Github actions is wonky if docker is not installed. The act tool told me docker needs to be installed. I also included some keys in Github secrets.

Changes

-Added unit tests for utils.ts
-Added integration test for SignupPage.tsx
-Created spec.cy.ts Cypress E2E test
-Configured .github/workflows/test.yml for automated testing

Additional Information

  • GitHub Actions was tested locally using act before committing.
  • If Docker is not installed it gives error.

Checklist

  • Tested (unit and integration)
  • Error Handling works
  • No console errors or warnings (except known Cypress failure)
  • Relevant documentation/comments are included
  • Code follows the project’s coding standards

run: |
cd frontend
npm run test -- --run # Add --run flag to run tests once
npm run test -- --run
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This step fails and the action never get to the next step!

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