-
Notifications
You must be signed in to change notification settings - Fork 18
Conversation
@@ -37,6 +37,7 @@ | |||
"@babel/preset-typescript": "^7.3.3", | |||
"@elastic/eslint-config-kibana": "link:../../packages/eslint-config-kibana", | |||
"@elastic/eslint-import-resolver-kibana": "link:../../packages/kbn-eslint-import-resolver-kibana", | |||
"@elastic/eui": "^22.3.1", |
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.
Do we find out why need to use an old version of EUI? Kibana 7.9 is using "26.3.3"
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.
@ylwu-amzn I'm still researching, adding this allows most of UT to pass.
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.
@ylwu-amzn the upgrade from 22.3.1 to 26.3.3 is breaking unit tests for this plugin.
Without downgrading we have about 16/48 (1 skipped) test suites passing.
With downgrading we have about 30/48 (1 skipped) test suites passing.
I am working on other 17 tests which are breaking because of 2 components:
- chrome.getUiSettingsClient() - fixing this now has 43/48 (1 skipped) test suites passing.
- chrome.breadcrumbs.set()
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.
@saratvemulapalli so this PR doesn't fix all tests?
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.
No I am still working on fixing the others.
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.
All tests can pass now after fixing the chrome function mock issue ?
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.
All the tests are passing. 5 of them which are not, I created an issue to track them. #288
Thanks for adding. Can you also bump the |
Thats a good point. Sure, adding them. |
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.
LGTM. Thanks for the change.
* Adding support for Kibana 7.9.0 * Fixing unit tests and adding release notes * Updating Github workflows to work with ODFE 1.10.0 and Kibana 7.9.0
Unit Test Summary:
Test Suites: 1 skipped, 47 passed, 47 of 48 total
Tests: 9 skipped, 215 passed, 224 total
5 new tests which are breaking are skipped and will be taken care with issue: #288
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.