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

Fixed Flaky Cypress Test in the User_Manage File #6583

Merged
merged 6 commits into from
Nov 10, 2023
Merged

Conversation

nihal467
Copy link
Member

@nihal467 nihal467 commented Nov 10, 2023

WHAT

🤖 Generated by Copilot at 352f326

The pull request enhances the test code for user management by using a class for user creation, adding more steps and assertions to a test case, and waiting for the users API response before navigating to the profile page. The changes affect the files cypress/e2e/users_spec/user_manage.cy.ts and cypress/pageobject/Users/ManageUserPage.ts.

Proposed Changes

  • Fixes #issue?
  • Change 1
  • Change 2
  • More?

@coronasafe/care-fe-code-reviewers @coronasafe/code-reviewers

Merge Checklist

  • Add specs that demonstrate bug / test a new feature.
  • Update product documentation.
  • Ensure that UI text is kept in I18n files.
  • Prep screenshot or demo video for changelog entry, and attach it to issue.
  • Request for Peer Reviews
  • Completion of QA

HOW

🤖 Generated by Copilot at 352f326

  • Import the UserCreationPage class to access user creation and verification methods (link)
  • Create an instance of the UserCreationPage class and assign it to the userCreationPage variable (link)
  • Add steps and assertions to the test case for adding a skill to a user in user_manage.cy.ts (link)
  • Modify the navigateToProfile method in ManageUserPage.ts to intercept and wait for the users API request and assert that the response status code is 200 (link)

@nihal467 nihal467 requested a review from a team November 10, 2023 09:27
@nihal467 nihal467 requested a review from a team as a code owner November 10, 2023 09:27
@nihal467 nihal467 requested a review from gigincg November 10, 2023 09:27
Copy link

netlify bot commented Nov 10, 2023

Deploy Preview for care-egov-staging ready!

Name Link
🔨 Latest commit 814697f
🔍 Latest deploy log https://app.netlify.com/sites/care-egov-staging/deploys/654e03bda559c90008e2cfd6
😎 Deploy Preview https://deploy-preview-6583--care-egov-staging.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

vercel bot commented Nov 10, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
care-storybook ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 10, 2023 10:20am

Copy link

cypress bot commented Nov 10, 2023

Passing run #1565 ↗︎

0 85 0 0 Flakiness 0

Details:

Fixed Flaky Cypress Test in the User_Manage File
Project: CARE Commit: 814697f8cd
Status: Passed Duration: 03:10 💡
Started: Nov 10, 2023 10:30 AM Ended: Nov 10, 2023 10:33 AM

Review all test suite changes for PR #6583 ↗︎

@khavinshankar khavinshankar merged commit 9f36488 into develop Nov 10, 2023
56 of 58 checks passed
@khavinshankar khavinshankar deleted the fix-flaky-user branch November 10, 2023 10:36
Copy link

@nihal467 We truly appreciate your efforts. Thank you for taking the time to contribute; this is a very valuable contribution to us 🥇. We always welcome your contribution 🙂, so feel free to contribute to anything anytime, and never lose that spirit of innovation 🙌.

Copy link
Member

@rithviknishad rithviknishad left a comment

Choose a reason for hiding this comment

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

@nihal467 you've changed the package.json, but have not committed the package-lock changes.

@nihal467 nihal467 restored the fix-flaky-user branch November 11, 2023 06:17
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