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

Add screenshot of welcome page in visual test #1477

Merged
merged 3 commits into from
Oct 12, 2024

Conversation

MarvNC
Copy link
Member

@MarvNC MarvNC commented Oct 11, 2024

Resolves #386

Playwright:
8b334db7a6fd66c2406f0b9cbc107f45d4b893a7

Copy link

codspeed-hq bot commented Oct 11, 2024

CodSpeed Performance Report

Merging #1477 will degrade performances by 14.27%

Comparing MarvNC:add-welcome-page-test (a8e12aa) with master (7acf87e)

Summary

❌ 1 regressions
✅ 4 untouched benchmarks

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Benchmarks breakdown

Benchmark master MarvNC:add-welcome-page-test Change
japanese transformations (n=78) 20 ms 23.4 ms -14.27%

Copy link

github-actions bot commented Oct 11, 2024

⚠️ Visual differences introduced by this PR; please validate if they are desirable.

View Playwright Report (note: open the "playwright-report" artifact)

@MarvNC MarvNC marked this pull request as ready for review October 11, 2024 19:25
@MarvNC MarvNC requested a review from a team as a code owner October 11, 2024 19:25
djahandarie
djahandarie previously approved these changes Oct 12, 2024
@MarvNC
Copy link
Member Author

MarvNC commented Oct 12, 2024

oh
msedge_Playwright_Test_Report_and_2_more_pages_-_Personal_2024-10-11_20-54-35

any idea why this is happening when the screenshot itself succeeded?

@djahandarie
Copy link
Collaborator

@MarvNC Hmm, no idea actually, since it's not a soft except, I would expect it to terminate and not take the screenshot. Are you sure the screenshot file isn't present from the last time you ran it?

Anyways, the fix would be to choose an element that isn't hidden, I guess. Or switch to toHaveCount(1)

@MarvNC
Copy link
Member Author

MarvNC commented Oct 12, 2024

Ah I meant in the initial playwright report the first time yeah, it failed to take the screenshot after the change using that selector. Anyway changed it to use page.getByText('Welcome to Yomitan!')) which probably makes more sense anyway and it worked!

@MarvNC MarvNC added this pull request to the merge queue Oct 12, 2024
Merged via the queue into yomidevs:master with commit f02be18 Oct 12, 2024
10 of 11 checks passed
@MarvNC MarvNC deleted the add-welcome-page-test branch October 12, 2024 05:05
@Kuuuube Kuuuube added kind/meta The issue or PR is meta area/ci The issue or PR is related to continuous integration and/or continuous delivery labels Oct 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ci The issue or PR is related to continuous integration and/or continuous delivery kind/meta The issue or PR is meta
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create playwright test for welcome page
3 participants