Skip to content

feat: Playback Validation using screenshot#519

Merged
Abhishk123 merged 9 commits intodevfrom
playbackValidation
Apr 21, 2026
Merged

feat: Playback Validation using screenshot#519
Abhishk123 merged 9 commits intodevfrom
playbackValidation

Conversation

@Abhishk123
Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings March 18, 2026 13:02
@github-actions
Copy link
Copy Markdown

Invalid PR title. It must start with feat: or fix: or BREAKING CHANGE: or feat(): or fix(): or BREAKING CHANGE():. Please see our Contributing-Guide.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR extends the Cypress screenshot capture flow to support “playback” validation by tagging screenshots with a type and capturing a screenshot during the “3rd party app is …” validation step.

Changes:

  • Capture a screenshot for the 3rd party '<app>' app is '<validation>' validation step, tagging it as PLAYBACK.
  • Extend UTILS.captureScreenshot() to accept an additional screenshotType parameter and include it in the request params.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
cypress/support/step_definitions/validations.js Captures a screenshot (typed as playback) before running the validation object flow.
cypress/support/cypress-support/src/utils.js Updates screenshot utility API to accept and forward screenshotType in the screenshot request.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread cypress/support/cypress-support/src/utils.js
@Abhishk123 Abhishk123 changed the title Playback Validation using screenshot feat: Playback Validation using screenshot Mar 18, 2026
NeeradaN
NeeradaN previously approved these changes Mar 25, 2026
Copilot AI review requested due to automatic review settings March 27, 2026 10:35
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds support for categorizing screenshots taken during Cypress validation steps, enabling playback-specific screenshot validation flows.

Changes:

  • Pass a playback screenshot type when capturing a screenshot for the “3rd party app is …” validation step.
  • Extend UTILS.captureScreenshot() to accept an optional screenshotType and include it in the screenshot request payload.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
cypress/support/step_definitions/validations.js Captures a screenshot with OCR processing enabled and labels it as playback for the relevant validation step.
cypress/support/cypress-support/src/utils.js Updates the screenshot utility to accept and forward an optional screenshotType field in the platform request.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread cypress/support/cypress-support/src/utils.js Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings March 27, 2026 11:25
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread cypress/support/cypress-support/src/utils.js Outdated
Copilot AI review requested due to automatic review settings March 30, 2026 07:50
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread cypress/support/step_definitions/fireboltCalls.js Outdated
Copilot AI review requested due to automatic review settings April 2, 2026 12:17
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread cypress/support/step_definitions/fireboltCalls.js
Comment thread cypress/support/cypress-commands/commands.js
Comment thread cypress/support/cypress-commands/commands.js
NeeradaN
NeeradaN previously approved these changes Apr 8, 2026
@Abhishk123 Abhishk123 merged commit 529f428 into dev Apr 21, 2026
7 checks passed
@Abhishk123 Abhishk123 deleted the playbackValidation branch April 21, 2026 04:42
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 21, 2026
@jbigel
Copy link
Copy Markdown
Contributor

jbigel commented Apr 21, 2026

🎉 This PR is included in version 1.23.0-dev.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants