-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
Change expected data to match change in table UI #11495
Conversation
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 on green
@@ -109,7 +109,7 @@ export default function ({ getService, getPageObjects }) { | |||
|
|||
it('should show correct data, ordered by Term', function () { | |||
|
|||
const expectedChartData = ['png 1,373', 'php 445', 'jpg 9,109', 'gif 918', 'css 2,159']; | |||
const expectedChartData = ['png', '1,373', 'php', '445', 'jpg', '9,109', 'gif', '918', 'css 2,159']; |
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.
Missed one
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.
yep, I see that in the failed test. Checked in now
Looks like this is passing now @LeeDr ! |
@LeeDr I added version labels for 6, 5.5, and 5.4.1 FYI. |
* Change expected data to match change in table UI * fix more expected results for data UI change * Revert one previous change and fix one other test * fix last issues * missed splitting the last array element * missed splitting the last array element
This needs a backport to 5.5 as well |
* Change expected data to match change in table UI * fix more expected results for data UI change * Revert one previous change and fix one other test * fix last issues * missed splitting the last array element * missed splitting the last array element
* Change expected data to match change in table UI * fix more expected results for data UI change * Revert one previous change and fix one other test * fix last issues * missed splitting the last array element * missed splitting the last array element
Fix test that I think was broken by #11452