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

"previousnextlink › should navigate in the builds history by using the previous next links" e2e test fails on firefox sometimes #7409

Closed
p12tic opened this issue Jan 27, 2024 · 1 comment
Labels

Comments

@p12tic
Copy link
Member

p12tic commented Jan 27, 2024

See e.g. here https://buildbot.buildbot.net/#/builders/127/builds/162, https://buildbot.buildbot.net/#/builders/127/builds/160:

[19/51] [firefox] › buildsnavigation.spec.ts:28:7 › previousnextlink › should navigate in the builds history by using the previous next links
1) [firefox] › buildsnavigation.spec.ts:28:7 › previousnextlink › should navigate in the builds history by using the previous next links
Error: build count has been incremented
expect.poll.toBeTruthy: Target closed
at pages/builder.ts:108
106 | }, {
107 | message: "build count has been incremented",
> 108 | }).toBeTruthy();
| ^
109 | }
110 |
111 | static async waitGoToBuild(page: Page, expectedBuildNumber: number) {
at Function.waitBuildFinished (/buildbot/buildbot-job/build/smokes-react/tests/pages/builder.ts:108:8)
at /buildbot/buildbot-job/build/smokes-react/tests/buildsnavigation.spec.ts:41:23
[1A�[2K 5) [firefox] › buildsnavigation.spec.ts:28:7 › previousnextlink › should navigate in the builds history by using the previous next links
Error: locator.click: Target closed
=========================== logs ===========================
waiting for getByRole('link', { name: 'runtests', exact: true })
============================================================
at pages/builder.ts:28
26 | static async goto(page: Page, builder: string) {
27 | await BuilderPage.gotoBuildersList(page);
> 28 | await page.getByRole('link', {name: builder, exact: true}).click();
| ^
29 | await page.waitForURL(/\/#\/builders\/[0-9]+$/);
30 | await BasePage.waitUntilFinishedLoading(page);
31 | }
at Function.goto (/buildbot/buildbot-job/build/smokes-react/tests/pages/builder.ts:28:64)
at Function.gotoForce (/buildbot/buildbot-job/build/smokes-react/tests/pages/builder.ts:34:5)
at /buildbot/buildbot-job/build/smokes-react/tests/buildsnavigation.spec.ts:38:5
@p12tic p12tic added the bug label Jan 27, 2024
@mokibit
Copy link
Collaborator

mokibit commented Apr 28, 2024

@p12tic,
this issue was fixed by #7535.

@p12tic p12tic closed this as completed Apr 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants