Skip to content

Cypress integration migrated to 11+#116

Merged
peterwilsoncc merged 2 commits intodevelopfrom
fix/114
Jan 11, 2023
Merged

Cypress integration migrated to 11+#116
peterwilsoncc merged 2 commits intodevelopfrom
fix/114

Conversation

@jayedul
Copy link
Contributor

@jayedul jayedul commented Dec 7, 2022

Description of the Change

Cypress integration migrated to 11+

Closes #114

How to test the Change

  • Install NPM dependencies
  • Run npm run cypress:run to test without GUI
  • Run npm run cypress:open to test with GUI
  • Check if tests pass successfully

Changelog Entry

Changed - Cypress integration migrated to 11+

@jayedul jayedul requested a review from dkotter December 7, 2022 09:01
@jayedul jayedul self-assigned this Dec 7, 2022
@jeffpaul
Copy link
Member

@jayedul is there any additional work here or can this PR move out of draft and come up for review?

@jeffpaul jeffpaul added this to the 1.5.0 milestone Dec 22, 2022
@jayedul jayedul marked this pull request as ready for review January 10, 2023 08:52
@jayedul
Copy link
Contributor Author

jayedul commented Jan 10, 2023

@jayedul is there any additional work here or can this PR move out of draft and come up for review?

There was a merge conflict since develop branch updated later. Now it has been resolved and marked as ready for review.

Copy link
Contributor

@peterwilsoncc peterwilsoncc left a comment

Choose a reason for hiding this comment

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

LGTM, thanks 👍

Testing notes:

  • npm i as expected
  • npm env:start as expected
  • npm cypress:run as expected
  • npm cypress:open as expected (ran each spec individually)

@peterwilsoncc peterwilsoncc merged commit 5463930 into develop Jan 11, 2023
@peterwilsoncc peterwilsoncc deleted the fix/114 branch January 11, 2023 02:28
@dkotter dkotter modified the milestones: 1.5.0, 1.4.2 Jan 13, 2023
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.

Migrate to Cypress 11

4 participants

Comments