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

[MDCT-2555] Stabilize carts e2e smoke test #139420

Merged
merged 6 commits into from
Jul 27, 2023
Merged

[MDCT-2555] Stabilize carts e2e smoke test #139420

merged 6 commits into from
Jul 27, 2023

Conversation

BearHanded
Copy link
Contributor

@BearHanded BearHanded commented Jul 26, 2023

Description

Some patient waiting for certain actions, and scope in on the test state when attempting to reset or uncertify as admin.
Cautiously reload when authenticating with cypress in case part of the state persists between users.

Related ticket(s)

MDCT-2555


How to test

Important updates


Author checklist

  • I have performed a self-review of my code
  • I have added thorough tests, if necessary
  • I have updated relevant documentation, if necessary

convert to a different template: test → val | val → prod

@BearHanded BearHanded changed the title Attempt to stabilize carts e2e smoke test [MDCT-2555] Attempt to stabilize carts e2e smoke test Jul 26, 2023
britt-mo
britt-mo previously approved these changes Jul 26, 2023
@codeclimate
Copy link

codeclimate bot commented Jul 27, 2023

Code Climate has analyzed commit 6f7b22d and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (90% is the threshold).

This pull request will bring the total coverage in the repository to 55.6% (0.0% change).

View more on Code Climate.

@BearHanded BearHanded marked this pull request as ready for review July 27, 2023 13:23
@BearHanded BearHanded changed the title [MDCT-2555] Attempt to stabilize carts e2e smoke test [MDCT-2555] Stabilize carts e2e smoke test Jul 27, 2023
Copy link
Contributor

@karla-vm karla-vm left a comment

Choose a reason for hiding this comment

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

🚢

@BearHanded BearHanded merged commit 940060a into main Jul 27, 2023
14 checks passed
@BearHanded BearHanded deleted the submit-cert-state branch July 27, 2023 13:53
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.

3 participants