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.

Sign up for GitHub

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

Jump to bottom

80959_TestCases #66

Merged
merged 7 commits into from
Oct 17, 2022
Merged

80959_TestCases #66

merged 7 commits into from
Oct 17, 2022

Conversation

manishasharma5757
Copy link
Contributor

ADO-81785

Description

List of proposed changes:

  • Test case added to validate Security settings page heading in English
  • Test case added to validate Security settings page heading in French
  • Test case to validate French button click goes to /fr/security
  • Test case added to validate that user can select 'Security Settings' option from Menu dropdown
  • Test case added to validate the breadcrumbs are present on security settings page

What to test for/How to test

Validate Security settings page heading and breadcrumbs using cypress tests

Additional Notes

@manishasharma5757 manishasharma5757 requested a review from a team as a code owner October 13, 2022 22:05
@github-actions
Copy link

Integration Deployment 🚀 - Build Status)
Default Tests Workflow Status
Jest Coverage Report
Cypress Coverage Report

DaceyTom2
DaceyTom2 approved these changes Oct 17, 2022
View reviewed changes
Copy link
Contributor

@DaceyTom2 DaceyTom2 left a comment

Choose a reason for hiding this comment

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

LGTM, covers existing page

@nibivi77 nibivi77 self-requested a review October 17, 2022 14:18
nibivi77
nibivi77 approved these changes Oct 17, 2022
View reviewed changes
Copy link
Contributor

@nibivi77 nibivi77 left a comment

Choose a reason for hiding this comment

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

👍

@manishasharma5757 manishasharma5757 merged commit cad5cea into dev Oct 17, 2022
@manishasharma5757 manishasharma5757 deleted the 80959_E2E_TCs branch October 17, 2022 15:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nibivi77 nibivi77 nibivi77 approved these changes

@DaceyTom2 DaceyTom2 DaceyTom2 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@manishasharma5757 @DaceyTom2 @nibivi77