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

Flaky test: EuiGlobalToastList lifetime elapse #1999

Closed
thompsongl opened this issue Jun 5, 2019 · 1 comment · Fixed by #2253
Closed

Flaky test: EuiGlobalToastList lifetime elapse #1999

thompsongl opened this issue Jun 5, 2019 · 1 comment · Fixed by #2253
Labels
testing Issues or PRs that only affect tests - will not need changelog entries

Comments

@thompsongl
Copy link
Contributor

Had one-off CI failures here a couple times this week. Noting for possible fix in the future

e.g., https://kibana-ci.elastic.co/job/elastic-eui-pull-request/11/console

09:23:02 Summary of all failing tests
09:23:02 FAIL src/components/toast/global_toast_list.test.js
09:23:02   ● EuiGlobalToastList › props › dismissToast › is called when the toast lifetime elapses
09:23:02 
09:23:02     expect(received).toBe(expected) // Object.is equality
09:23:02 
09:23:02     Expected: true
09:23:02     Received: false
09:23:02 
09:23:02       87 |             toasts={[
09:23:02       88 |               {
09:23:02     > 89 |                 'data-test-subj': 'b',
09:23:02          |                                       ^
09:23:02       90 |                 id: 'b',
09:23:02       91 |               },
09:23:02       92 |             ]}
09:23:02 
09:23:02       at src/components/toast/global_toast_list.test.js:89:42
09:23:02       at Timeout.callback [as _onTimeout] (node_modules/jsdom/lib/jsdom/browser/Window.js:633:19)
09:23:02 
09:23:02   ● EuiGlobalToastList › props › dismissToast › is called when the toast lifetime elapses
09:23:02 
09:23:02     Error: Uncaught [Error: expect(received).toBe(expected) // Object.is equality
09:23:02 
09:23:02     Expected: true
09:23:02     Received: false]
09:23:02 
09:23:02       2 | // failed prop types check.
09:23:02       3 | console.error = message => {
09:23:02     > 4 |   throw new Error(message);
09:23:02         |         ^
09:23:02       5 | };
09:23:02       6 | 
09:23:02 
09:23:02       at reportException (node_modules/jsdom/lib/jsdom/living/helpers/runtime-script-errors.js:66:24)
09:23:02       at Timeout.callback [as _onTimeout] (node_modules/jsdom/lib/jsdom/browser/Window.js:635:7)
09:23:02       at BufferedConsole.Object.<anonymous>.console.error (scripts/jest/setup/throw_on_console_error.js:4:9)
09:23:02       at VirtualConsole.on.e (node_modules/jsdom/lib/jsdom/virtual-console.js:29:45)
09:23:02       at reportException (node_modules/jsdom/lib/jsdom/living/helpers/runtime-script-errors.js:70:28)
09:23:02       at Timeout.callback [as _onTimeout] (node_modules/jsdom/lib/jsdom/browser/Window.js:635:7)
@thompsongl thompsongl added the testing Issues or PRs that only affect tests - will not need changelog entries label Jun 5, 2019
@thompsongl
Copy link
Contributor Author

Definitely flaky. Just had it fail locally.

@thompsongl thompsongl changed the title Possible flaky test: EuiGlobalToastList lifetime elapse Flaky test: EuiGlobalToastList lifetime elapse Jun 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
testing Issues or PRs that only affect tests - will not need changelog entries
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant