-
Notifications
You must be signed in to change notification settings - Fork 219
Fix: E2E flaky test related to Stock status Form Token Field #7839
Conversation
The release ZIP for this PR is accessible via:
|
TypeScript Errors ReportFiles with errors: 439 🎉 🎉 This PR does not introduce new TS errors. |
Size Change: 0 B Total Size: 1 MB ℹ️ View Unchanged
|
Script Dependencies ReportThere is no changed script dependency between this branch and trunk. This comment was automatically generated by the |
Note that in 3ba8e28, I migrate refactoring improvements from #7719 so we can merge those changes earlier. cc @sunyatasattva |
…g on the front end
The release ZIP for this PR is accessible via:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have paired on this with Tung. The code seems sound! The tests passed for me on the 3rd try.
This PR has been marked as If deemed still relevant, the pr can be kept active by ensuring it's up to date with the main branch and removing the stale label. |
Given that some changes in this PR are overridden/updated by #7749, I'm closing this one. |
Since #7684, the tests interacting with Form Token Field have become a new source of flaky tests. This PR fix those tests to make them more reliable.
This PR also enables the
Editor preview shows correct products that has enabled stock statuses
test as we merged #7682.Testing
Automated Tests