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

[PROPOSAL] - HPC-9658 Add cypress tests #479

Open
wants to merge 8 commits into
base: HPC-8546
Choose a base branch
from
Open

Conversation

Onitoxan
Copy link
Contributor

@Onitoxan Onitoxan commented Nov 27, 2024

Nature of PR

  • Bug-fix
  • Hot-fix
  • Feature
  • 🟢 Testing
  • Rewrite
  • CI

Description

Initial implementation of tests for FTS Admin. These test consist of mocked data and mocked endpoints, so we are focusing and isolating frontend behaviors. This is a decision in order to have deterministic tests. Therefore these tests would not really be E2E tests.

How to test changes

For testing, you can run the test in headed mode by running :

npm run e2e hpc-ftsadmin-e2e -- --watch

All the test that are written should not fail.

TODO:

These test can be really improved and implement way more use cases, but I think as a first approach is best to review these small tests and decide if we want to invest more time into building them

@Onitoxan Onitoxan added the ready for review All comments have been addressed, and the Pull Request is ready for review label Nov 27, 2024
@Onitoxan Onitoxan requested a review from a team as a code owner November 27, 2024 14:12
Copy link

@unocha-hpc unocha-hpc left a comment

Choose a reason for hiding this comment

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

Checks have passed and this pull request is ready for manual review

@Onitoxan Onitoxan force-pushed the HPC-8546 branch 3 times, most recently from 7597db7 to 5eef990 Compare December 4, 2024 10:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready for review All comments have been addressed, and the Pull Request is ready for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants