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

test(item-sliding): update snapshots #29906

Draft
wants to merge 3 commits into
base: ROU-11149-item
Choose a base branch
from

Conversation

thetaPC
Copy link
Contributor

@thetaPC thetaPC commented Sep 27, 2024

Issue number: internal


What is the current behavior?

Playwright was not importing the correct ionic bundle file, which has caused some snapshots to not be correct. This is apparent when a test uses setContent. There are several components that are needing their snapshots to be re-generated. In order to not have a PR with an immense amount of snapshots, several PRs will be created to split off the work to make it easier on reviewers.

The snapshots for item sliding are not accurate. This can be verified by comparing it to the test pages, which do import the right bundle file.

Since the ion-item was updated, then the snapshots for ion-item-sliding would also need to be updated due to ion-item-sliding using ion-item.

What is the new behavior?

  • Updated snapshots

Does this introduce a breaking change?

  • Yes
  • No

Other information

Base branch
Item branch

Copy link

vercel bot commented Sep 27, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
ionic-framework ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 27, 2024 10:39pm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package: core @ionic/core package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants