Skip to content

Content auto-validation implemented #942

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
Open
wants to merge 26 commits into
base: master
Choose a base branch
from
Open

Content auto-validation implemented #942

wants to merge 26 commits into from

Conversation

@Gautham059
Copy link
Contributor

@Gautham059 Gautham059 commented Nov 12, 2025 •
edited
Loading

Jira link

https://tools.hmcts.net/jira/browse/HDPI-2675

Change description

Implemented page content auto-vallidation for all the pages. Stages involved -

  1. Auto URL change detection
  2. Finding the appropriate file based on URL to filename mapping
  3. Iterate through all the elemnents in the file
  4. Validate the elements using their respective page locators.
  5. Report any missing elements in Allure reports. individually for each test.
  6. Fail the test if any mismtaches found.

Testing done

Failed test example:
image

Passed test example:
image

image

@Gautham059 Gautham059 added the enable_e2e_test Currently used to test the common e2e test call label Nov 12, 2025
@github-actions

This comment was marked as spam.

Sign in to view
@github-actions

This comment was marked as spam.

Sign in to view
@github-actions

This comment was marked as spam.

Sign in to view
@hmcts hmcts deleted a comment from github-actions bot Nov 12, 2025
@github-actions
Copy link

github-actions bot commented Nov 12, 2025 •
edited
Loading

CCD diff summary

👉 Full report: https://github.com/hmcts/pcs-api/actions/runs/19371702550?check_suite_focus=true

No change

@hmcts-jenkins-j-to-z hmcts-jenkins-j-to-z bot requested a deployment to preview November 13, 2025 19:59 Abandoned
@Gautham059 Gautham059 marked this pull request as ready for review November 14, 2025 09:06
@Gautham059 Gautham059 requested review from a team as code owners November 14, 2025 09:06
srinijg
srinijg reviewed Nov 14, 2025
View reviewed changes
SameenaHMCTS
SameenaHMCTS approved these changes Nov 14, 2025
View reviewed changes
Copy link
Contributor

@SameenaHMCTS SameenaHMCTS left a comment

Choose a reason for hiding this comment

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

Thank you

seymour-quality
seymour-quality approved these changes Nov 14, 2025
View reviewed changes
Copy link
Contributor

@seymour-quality seymour-quality left a comment

Choose a reason for hiding this comment

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

lgtm

srinijg
srinijg approved these changes Nov 14, 2025
View reviewed changes
Copy link
Contributor

@srinijg srinijg left a comment

Choose a reason for hiding this comment

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

looks good to me

@Gautham059 Gautham059 changed the title Content auto-validation added Content auto-validation implemented Nov 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@seymour-quality seymour-quality seymour-quality approved these changes

@SameenaHMCTS SameenaHMCTS SameenaHMCTS approved these changes

@lovelin-07 lovelin-07 lovelin-07 approved these changes

@srinijg srinijg srinijg approved these changes

Assignees

No one assigned

Labels

enable_e2e_test Currently used to test the common e2e test call enable_keep_helm ns:pcs pr-values:ccd prd:pcs rel:pcs-api-pr-942

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@Gautham059 @seymour-quality @SameenaHMCTS @lovelin-07 @srinijg