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

chore: adds and fixes e2e for content history #20098

Merged
merged 14 commits into from
May 27, 2024

Conversation

remidej
Copy link
Contributor

@remidej remidej commented Apr 11, 2024

What does it do?

  • Adds e2e tests to ensure that Content History handles missing relations properly.
  • Fixes issues with the design system upgrade: links opening in a new tab, links not having the right color
  • Various e2e test fixes

Related issue(s)/PR(s)

@remidej remidej added the source: core:content-manager Source is core/content-manager package label Apr 11, 2024
@remidej remidej requested a review from markkaylor April 11, 2024 17:46
@remidej remidej self-assigned this Apr 11, 2024
@remidej remidej changed the title chore: add e2e to content history for relations handling chore: add e2e tests to content history for relations handling Apr 11, 2024
Copy link

vercel bot commented Apr 11, 2024

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

Name Status Preview Comments Updated (UTC)
contributor-docs ❌ Failed (Inspect) May 24, 2024 3:06pm

Copy link
Contributor

@markkaylor markkaylor left a comment

Choose a reason for hiding this comment

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

Tests look good to me and are passing locally. But some of the other tests are now failing. Maybe related to the flaky tests already mentioned... 🤔

@remidej
Copy link
Contributor Author

remidej commented Apr 16, 2024

Yes I'm planning to come back to it after the beta and investigate the timeouts

@remidej remidej changed the title chore: add e2e tests to content history for relations handling chore: adds and fixes e2e for content history May 24, 2024
Comment on lines +44 to +48
const closeSuccessNotification = async (page: Page) => {
const successNotification = page.getByRole('alert', { name: /success/i });
if (await successNotification.isVisible()) {
await successNotification.getByRole('button', { name: /close/i }).click();
}
Copy link
Contributor

Choose a reason for hiding this comment

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

Not positive but it looks like this util might do the same thing, or is meant to do the same thing...

/**
* Look for an element containing text, and then click a sibling close button
*/
export const findAndClose = async (
page: Page,
text: string,
role: string = 'status',
closeLabel: string = 'Close'
) => {

@remidej remidej merged commit 79490c1 into v5/main May 27, 2024
24 of 29 checks passed
@remidej remidej deleted the chore/history-relations-e2e branch May 27, 2024 15:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
source: core:content-manager Source is core/content-manager package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants