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

[Test] Login Frontend #54

Merged
merged 32 commits into from
Aug 31, 2023
Merged

[Test] Login Frontend #54

merged 32 commits into from
Aug 31, 2023

Conversation

UranusLin
Copy link
Collaborator

Summary

Completed the setup of the Testing Library and merged the Playwright settings branch. Added several DOM test cases to enhance our testing coverage.

Linked Issue

#36

Details

  • Set up the Testing Library for frontend testing.
  • Merged the Playwright settings branch.
  • Introduced new DOM test cases.

Impacted Areas

  • Frontend Testing setup
  • Playwright settings

Tests

  • Add new DOM test cases.
  • Testing was performed on the local development environment.
  • All tests ran successfully without issues.

Possible Impacts

The changes made primarily focus on testing, so there shouldn't be any impact on the actual functionality of the application. However, there could be potential overlaps or conflicts with other testing tools or configurations, if any.

Verification Steps

  1. Pull the latest changes from this branch.
  2. Run the test suite using the command yarn test, yarn test:e2e, yarn test:dom.
  3. Ensure all tests pass without any errors.
  4. Review the new DOM test cases to ensure they cover the desired functionality.

Todo

  • Add more DOM test cases.
  • Add new Playwright test case.

Checklist

  • All new and existing tests pass

@UranusLin UranusLin marked this pull request as ready for review August 28, 2023 16:23
@ryanycw ryanycw changed the base branch from release/integrate_0713 to main August 29, 2023 11:06
@github-actions github-actions bot added documentation Improvements or additions to documentation automation CI/CD related frontend test test improvements labels Aug 29, 2023
Copy link
Collaborator

@badukwei badukwei left a comment

Choose a reason for hiding this comment

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

It seems to me that this pr is fine. But we have to create more tests for the details.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Should we create tests for the functions in the context and make it safer?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes, we need add more test about dom and state change

@UranusLin UranusLin merged commit 4f975b5 into main Aug 31, 2023
3 of 5 checks passed
@ryanycw ryanycw mentioned this pull request Aug 31, 2023
4 tasks
@ryanycw ryanycw deleted the feature/Add-frontend-test branch September 10, 2023 15:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automation CI/CD related documentation Improvements or additions to documentation frontend test test improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants