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

Cypress - Frontend Tests Integration [WIP] #479

Closed
wants to merge 23 commits into from

Conversation

pratyush1712
Copy link
Member

@pratyush1712 pratyush1712 commented Nov 4, 2023

Summary

This Pull Request is the first step towards setting up a robust testing environment for the Web Frontend. Frontend tests are needed to make sure new features do not break any existing features on the frontend and most importantly they do not have any accessibility issues.

Setting Up

  • Installed cypress (testing framework)
  • Installed cypress-axe (testing accessibility)

Planning

  • Introduce Cypress to the codebase.
  • Add features (dummy data) for testing purposes.
  • Implement GitHub Actions Pipeline to run automated tests.
  • Implement commands to test out: the accessibility of common modals, lists, etc...
  • Implement tests for both student and admin dashboards.

General Changes

We need to use the testing database for authentication.

Authentication

  • Update the frontend testing command
    • Start the server with the testing environment
  • Add the fake accounts (student and admin) to the database

Landing Page

  • Add login functionality

@pratyush1712 pratyush1712 requested a review from a team as a code owner November 4, 2023 00:46
@dti-github-bot
Copy link
Member

dti-github-bot commented Nov 4, 2023

[diff-counting] Significant lines: 331.

- Added JSdocs comments to the encryption and decryption functions.
- Updated the useful commands section in the README.md file.
"id": 8739,
"name": "Jane",
"email": "jane@example.com"
}
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: eof should be line break

"bs": "target end-to-end models"
}
}
]
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: eof should be line break


`npm run start:frontend`: It will start the frontend server.

`npm run start`: It will start both the backend and frontend servers.
Copy link
Contributor

Choose a reason for hiding this comment

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

I might be tripping but I don't see this defined in /package.json?

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.

5 participants