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

[Snapshot testing - Part 2] Filing app tests #1072

Open
wants to merge 27 commits into
base: main
Choose a base branch
from
Open

[Snapshot testing - Part 2] Filing app tests #1072

wants to merge 27 commits into from

Conversation

meissadia
Copy link
Collaborator

@meissadia meissadia commented Dec 6, 2024 •
edited
Loading

Closes #1062

Changes

  • Adds Snapshot testing for Filing details - Point of contact page
  • Adds Snapshot testing for Filing step routing
  • Adds Snapshot testing for Filing - Sign and submit
  • Adds Snapshot testing for Filing - Upload file
  • Adds Snapshot testing for Filing - Form alerts
  • Adds Snapshot testing for Filing - Unavailable APIs

How to test this PR

  1. yarn test:e2e:snapshot
  2. Run filing-app tests

Screenshots

Screenshot 2024-12-12 at 10 19 08 AM

@meissadia
feat: [Playwright] Support running of snapshot tests via environment …
2e13c5f 
…variable (SBL_ENABLE_PLAYWRIGHT_SNAPSHOT_TESTING)
@meissadia
task: [Playwright] Centralize check for running/skipping snapshot tes…
9353393 
…ts based on ENV
@meissadia
task: [Snapshot test] Complete user profile form
86716bb
@meissadia
feat: [Playwright] Support running of snapshot tests via environment …
035db97 
…variable (SBL_ENABLE_PLAYWRIGHT_SNAPSHOT_TESTING)
@meissadia
task: [Playwright] Centralize check for running/skipping snapshot tes…
22cd9bc 
…ts based on ENV
@meissadia
task: [Snapshot test] Complete user profile form
4481f39
@meissadia
Merge branch 'main' into 1062-snapshot__complete-user-profile
60bf075
@meissadia
task: Update Vite environment with new ENV variable for snapshot testing
5b9118c
@meissadia
task: Snapshot testing - adjust default maxDiffPixels
5fb6a2a
@meissadia
task: Snapshot testing - Use a more generalized masking approach (add…
ef07a8d 
… .snapshot-ignore to the element) to simplify the process
@meissadia
Merge branch 'main' into 1062-snapshot__complete-user-profile
f861537
@meissadia
task: [Snapshot test] Filing details
2e7206b
@meissadia meissadia changed the title 1062 snapshot filing details [Snapshot testing] Filing details - Point of contact Dec 6, 2024
@meissadia meissadia mentioned this pull request Dec 6, 2024
Merged
@meissadia meissadia marked this pull request as ready for review December 9, 2024 17:31
@meissadia meissadia enabled auto-merge (squash) December 9, 2024 17:32
@meissadia meissadia changed the title [Snapshot testing] Filing details - Point of contact [Snapshot testing - Part 2] Filing details - Point of contact Dec 9, 2024
@meissadia meissadia changed the title [Snapshot testing - Part 2] Filing details - Point of contact [Snapshot testing - Part 2] Filing app Dec 11, 2024
@meissadia meissadia changed the title [Snapshot testing - Part 2] Filing app [Snapshot testing - Part 2] Filing app tests Dec 11, 2024
This was referenced Dec 11, 2024
Open
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@billhimmelsbach billhimmelsbach Awaiting requested review from billhimmelsbach billhimmelsbach is a code owner

@tanner-ricks tanner-ricks Awaiting requested review from tanner-ricks tanner-ricks is a code owner

At least 1 approving review is required to merge this pull request.

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

Successfully merging this pull request may close these issues.

[e2e] Add snapshot testing
1 participant
@meissadia