Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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] Add AD job validation UI tests #179358
[ML] Add AD job validation UI tests #179358
Changes from 5 commits
4f9a588
215d757
49c3901
057da93
2667c93
d5bc5a5
433aad8
21c1f50
f1bb227
30a59c6
affd426
9dc3b15
f5cd43b
6602b29
ef73e43
e1e75d1
bdd577b
57ed9df
b33540c
b1385ce
5a8d112
4e73965
d0eab64
28b0228
a859e3d
0fca96c
3664a94
347173a
01be0f4
2c17a27
d57bc59
0973dce
9ea07a3
db7ff28
e3d5652
1bfe1b3
4fcc3b9
2204171
1f9611a
6c22755
2af4109
1937d40
97995c1
18d214c
3d2720a
c30644a
bee8109
aabfc04
894de3d
43d2b26
9ac5448
e0ad466
7ec5b38
82afd94
969bf50
dfa29d5
bf81eee
c5237ca
66488da
a7a75e3
ea3e6dc
38a829b
ac47e74
ecbd487
268e403
60cb125
b8c45d7
ab3b025
afa90d7
e53e9d9
a1fe7e4
0f62a46
d8edc00
9d7a2a2
a0e5557
4be34ab
820e96c
0355263
dd28646
1ee9dee
d4bea26
528d151
31f5136
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
https://eui.elastic.co/#/forms/date-picker supports
data-test-subj
, hence you do not need a wrapper.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.
I tried it, following the docs per our zoom, but it did not work. I'll try again though!
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.
@darnautov perhaps a longer timeout?
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.
FYI adding a
div
wrapper also braked styles.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.
@wayneseymour you set a
data-test-subj
prop in the wrong place. It should be forEuiDatePickerRange
, notendDateControl
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.
Ah ok, so many dates lol. Thanks @darnautov 🙉
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.
yeah I dont think either of those work, so I've switched it to a span
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 more span!
1f9611a
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.
I reckon you should not create a dedicated test suite for that, and go through all the wizard steps again.
We already have multiple tests for job wizards. Simply add a test suite to assert job validation.
For instance in the multi-metric wizards tests:
Add your assertion after these lines
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.
@darnautov One of the reasons I created this is not have a nested suite.
I'll see if I can fit these in elsewhere without a nested suite.
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.
We have a similar code snippet copy-pasted in so many places. Can you please create reusable methods in
x-pack/test/functional/services/ml/common_ui.ts
?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.
wayneseymour@6602b29