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

[FE][apispec-view] Screenshot tests restoration #2

Open
wants to merge 7 commits into
base: develop
Choose a base branch
from

Conversation

vOrigins
Copy link
Contributor

@vOrigins vOrigins commented Mar 1, 2025

No description provided.

@vOrigins vOrigins self-assigned this Mar 1, 2025
@vOrigins vOrigins changed the title [FE][apispec-view] screenshot tests restoration [FE][apispec-view] Screenshot tests restoration Mar 1, 2025
@vOrigins vOrigins requested review from JayLim2 and CountRedClaw March 4, 2025 08:16
@vOrigins vOrigins marked this pull request as ready for review March 4, 2025 08:16
@@ -18,6 +18,7 @@
"@netcracker/qubership-apihub-api-diff": "1.0.4",
"@netcracker/qubership-apihub-api-doc-viewer": "2.0.3",
"@netcracker/qubership-apihub-http-spec": "5.5.7",
"@netcracker/qubership-apihub-jest-chrome-in-docker-environment": "dev",
Copy link
Collaborator

Choose a reason for hiding this comment

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

don't we want to release new version of this package before merging ADV into develop?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

As discussed with @b41ex, we will leave the dev version for now.

Copy link
Collaborator

Choose a reason for hiding this comment

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

screenshot tests failed on these screenshots, right? because I don't see any difference on changed screenshot

Copy link
Contributor Author

Choose a reason for hiding this comment

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

A few pixels difference due to Chrome version upgrade from 121 to 125.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In review
Development

Successfully merging this pull request may close these issues.

[FE] Configure screenshot tests for open source
3 participants