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

E2E Tests: Intermittent failure in sidebar resizing test #21177

Closed
aduth opened this issue Mar 26, 2020 · 0 comments · Fixed by #21180
Closed

E2E Tests: Intermittent failure in sidebar resizing test #21177

aduth opened this issue Mar 26, 2020 · 0 comments · Fixed by #21180
Assignees
Labels
[Status] In Progress Tracking issues with work in progress [Type] Bug An existing feature does not function as intended

Comments

@aduth
Copy link
Member

aduth commented Mar 26, 2020

Previously: #20982 (comment)

Examples:

Failure message:

FAIL packages/e2e-tests/specs/editor/various/sidebar.test.js (16.039s)
  Sidebar
  ● Sidebar › should reopen sidebar the sidebar when resizing from mobile to desktop if the sidebar was closed automatically
    expect(received).toHaveLength(expected)
    Expected length: 1
    Received length: 0
    Received array:  []
      89 | 
      90 | 		const sidebarsDesktop = await page.$$( SIDEBAR_SELECTOR );
    > 91 | 		expect( sidebarsDesktop ).toHaveLength( 1 );
         | 		                          ^
      92 | 	} );
      93 | 
      94 | 	it( 'should preserve tab order while changing active tab', async () => {
      at Object.toHaveLength (specs/editor/various/sidebar.test.js:91:29)
          at runMicrotasks (<anonymous>)

Steps to Reproduce:

It is fairly reliably reproduced when using CPU throttling in end-to-end tests, and not without, indicating some issue of CPU timing.

THROTTLE_CPU=4 npm run test-e2e packages/e2e-tests/specs/editor/various/sidebar.test.js

Related history:

@aduth aduth added [Type] Bug An existing feature does not function as intended [Type] Automated Testing Testing infrastructure changes impacting the execution of end-to-end (E2E) and/or unit tests. labels Mar 26, 2020
@github-actions github-actions bot added the [Status] In Progress Tracking issues with work in progress label Mar 26, 2020
@aduth aduth removed the [Type] Automated Testing Testing infrastructure changes impacting the execution of end-to-end (E2E) and/or unit tests. label Apr 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Status] In Progress Tracking issues with work in progress [Type] Bug An existing feature does not function as intended
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant