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.

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

Playwright Snapshots blog #295

Merged
merged 35 commits into from
Feb 12, 2025

Conversation

mnyamunda-scottlogic
Copy link
Contributor

@mnyamunda-scottlogic mnyamunda-scottlogic commented Jan 24, 2025

Please add a direct link to your post here: https://mnyamunda-scottlogic.github.io/blog/2025/01/22/playwright-visual-testing.html

Have you (please tick each box to show completion):

  • Added your blog post to a single category?
  • Added a brief summary for your post? Summaries should be roughly two sentences in length and give potential readers a good idea of the contents of your post.
  • Checked that the build passes?
  • Checked your spelling (you can use npm install followed by npx mdspell "**/{FILE_NAME}.md" --en-gb -a -n -x -t if that's your thing)
  • Ensured that your author profile contains a profile image, and a brief description of yourself? (make it more interesting than just your job title!)
  • Optimised any images in your post? They should be less than 100KBytes as a general guide.

Posts are reviewed / approved by your Regional Tech Lead.

Copy link
Member

@chriswilty chriswilty left a comment

Choose a reason for hiding this comment

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

A few comments. Many of the code blocks are messed up, don't know why...

_posts/2025-01-09-playwright-visual-testing.markdown Outdated Show resolved Hide resolved
_posts/2025-01-09-playwright-visual-testing.markdown Outdated Show resolved Hide resolved

Shown above is the UI consisting of six different charts that show forecasted and in-situ pollutant measurements. The first big problem is that the data is non-static. We solve this by mocking data via Playwright. This way, our regression focuses on the functionality and various transformations that the data undergoes. The second problem is how do we know that the actual data is being visualised correctly? This is where Playwright snapshots come in.

## Playwright Snapshotting Explained
Copy link
Member

Choose a reason for hiding this comment

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

I can't find any reference to the term Snapshotting, maybe best leave it as Snapshot Testing.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sounds good to me

_posts/2025-01-09-playwright-visual-testing.markdown Outdated Show resolved Hide resolved
_posts/2025-01-09-playwright-visual-testing.markdown Outdated Show resolved Hide resolved
_posts/2025-01-09-playwright-visual-testing.markdown Outdated Show resolved Hide resolved
_posts/2025-01-09-playwright-visual-testing.markdown Outdated Show resolved Hide resolved
_posts/2025-01-09-playwright-visual-testing.markdown Outdated Show resolved Hide resolved
_posts/2025-01-09-playwright-visual-testing.markdown Outdated Show resolved Hide resolved
_posts/2025-01-09-playwright-visual-testing.markdown Outdated Show resolved Hide resolved
@chriswilty
Copy link
Member

This is looking really good 👌

Copy link
Member

@chriswilty chriswilty left a comment

Choose a reason for hiding this comment

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

Just a couple of typos I spotted this time around, otherwise all looks great

_posts/2025-01-09-playwright-visual-testing.markdown Outdated Show resolved Hide resolved
_posts/2025-01-09-playwright-visual-testing.markdown Outdated Show resolved Hide resolved
@chriswilty chriswilty merged commit 359f27d into ScottLogic:gh-pages Feb 12, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants