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

Fix failing cypress tests #774

Merged
merged 1 commit into from
Mar 9, 2024

Conversation

sir-radar
Copy link
Contributor

@sir-radar sir-radar commented Mar 7, 2024

Describe your changes

Changes/Improvements to cypress test code to cater for new changes causing pipeline failures

Issue ticket number and link

Closed #773

Motivation and Context

Resolve pipeline failure.

How Has This Been Tested?

Run npm run cypress:run

@sir-radar
Copy link
Contributor Author

@ijemmao please take a look

Copy link
Collaborator

@ijemmao ijemmao left a comment

Choose a reason for hiding this comment

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

@sir-radar can you include an image of the frontend Cypress tests locally passing on your machine in your PR description?

@sir-radar
Copy link
Contributor Author

@ijemmao I noticed another issue making some Cypress tests fail, I have created issue #776 to resolve it.

Copy link
Collaborator

@ijemmao ijemmao left a comment

Choose a reason for hiding this comment

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

great! we can follow up in your new issue ticket - amazing work

@ijemmao ijemmao merged commit 8c08203 into nkowaokwu:master Mar 9, 2024
3 checks passed
ijemmao pushed a commit that referenced this pull request Mar 9, 2024
## [1.63.2](v1.63.1...v1.63.2) (2024-03-09)

### Bug Fixes

* make failing cypress tests pass ([#774](#774)) ([8c08203](8c08203))
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.

Failing cypress tests
2 participants