-
Notifications
You must be signed in to change notification settings - Fork 8.1k
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
[ML] File data viz fix fields from mappings #77251
[ML] File data viz fix fields from mappings #77251
Conversation
Pinging @elastic/ml-ui (:ml) |
@elasticmachine merge upstream |
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
async assertNumberOfFieldCards(number: number) { | ||
const cards = await testSubjects.findAll('mlPageFileDataVisFieldDataCard'); | ||
if (cards.length !== number) { | ||
throw new Error(`expected ${number} field cards to exist, but found ${cards.length}`); |
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.
While this works, I'd suggest to use expect
which is used for most of our assertions, something like
expect(cards.length).to.eql(number, `expected ${number} field cards to exist, but found ${cards.length}`);
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.
changed in 088c544
💚 Build SucceededBuild metricsasync chunks size
History
To update your PR or re-run it, just comment with: |
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.
Functional test changes LGTM
* [ML] File data viz fix fields from mappings * adding card count test * updating test Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
* [ML] File data viz fix fields from mappings * adding card count test * updating test Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com> Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
Removes
properties
as a field in the UI which was introduced by accident as part of the changes in elastic/elasticsearch#62158 and #77035Fixes #77220