Skip to content

Adding in testing for screenshots#16

Merged
peterpeterparker merged 4 commits intojunobuild:mainfrom
biocodeit:main
Oct 30, 2025
Merged

Adding in testing for screenshots#16
peterpeterparker merged 4 commits intojunobuild:mainfrom
biocodeit:main

Conversation

@biocodeit
Copy link
Copy Markdown
Contributor

I have written tests for checking in mail deposited and deposit-failed.

@biocodeit
Copy link
Copy Markdown
Contributor Author

@peterpeterparker how is first screenshot capture configed, so it has to run npm run test:snapshot, then next it can check against the generated snapshots.

Comment thread tests/homePage.spec.ts Outdated
Comment thread tests/homePage.spec.ts Outdated
Comment thread tests/homePage.spec.ts Outdated
Comment thread tests/homePage.spec.ts Outdated
Comment thread tests/homePage.spec.ts Outdated
Comment thread tests/homePage.spec.ts Outdated
Comment thread tests/homePage.spec.ts Outdated
Copy link
Copy Markdown
Contributor

@peterpeterparker peterpeterparker left a comment

Choose a reason for hiding this comment

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

Can you remove the screenshot from the PR? We can generate those with the GitHub Action, I'll start it when ready.

Copy link
Copy Markdown
Contributor

@peterpeterparker peterpeterparker left a comment

Choose a reason for hiding this comment

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

Can you also revert the change regarding the dummy.spec? It's cleaner to remove it in another PR as we also have to remove its related template.

Copy link
Copy Markdown
Contributor

@peterpeterparker peterpeterparker 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 enough for me. I'll fix and improve it after merge.

Thanks for your contribution!

@peterpeterparker peterpeterparker merged commit 88f178e into junobuild:main Oct 30, 2025
1 of 2 checks passed
@peterpeterparker peterpeterparker linked an issue Oct 30, 2025 that may be closed by this pull request
@biocodeit
Copy link
Copy Markdown
Contributor Author

@peterpeterparker Thanks for allowing me to contribute. :)

@peterpeterparker
Copy link
Copy Markdown
Contributor

Follow-up #17, #18, #19 and #20

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Screenshots testing with Playwright

2 participants