Skip to content

[ML] Add data-test-subj to some job management elements - #35284

Merged
pheyos merged 1 commit into
elastic:masterfrom
pheyos:job_management_data_test_subj
Apr 23, 2019
Merged

pheyos merged 1 commit into
elastic:masterfrom
pheyos:job_management_data_test_subj

Conversation

@pheyos

@pheyos pheyos commented Apr 18, 2019

Copy link
Copy Markdown
Member

Summary

This PR adds a data-test-subj attribute with a descriptive value to the following elements of the ML job management page:

  • Create new job button
  • Refresh job list button
  • Job list filter bar
  • Jobs table
  • Delete job modal

This will help locate these elements during UI automation more reliably.

Checklist

Use strikethroughs to remove checklist items you don't feel are applicable to this PR.

For maintainers

@pheyos pheyos added the :ml label Apr 18, 2019
@pheyos
pheyos requested a review from a team as a code owner April 18, 2019 09:45
@pheyos pheyos self-assigned this Apr 18, 2019
@elasticmachine

Copy link
Copy Markdown
Contributor

Pinging @elastic/ml-ui

@elasticmachine

Copy link
Copy Markdown
Contributor

💚 Build Succeeded

@walterra walterra left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jgowdyelastic jgowdyelastic left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@peteharverson peteharverson left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@pheyos
pheyos merged commit e43fb2f into elastic:master Apr 23, 2019
@pheyos
pheyos deleted the job_management_data_test_subj branch April 23, 2019 09:08
pheyos added a commit to pheyos/kibana that referenced this pull request Apr 23, 2019
pheyos added a commit to pheyos/kibana that referenced this pull request Apr 23, 2019
pheyos added a commit that referenced this pull request Apr 23, 2019
)

Backports the following commits to 7.0:
- [ML] Add data-test-subj to some job management elements (#35284)
pheyos added a commit that referenced this pull request Apr 23, 2019
)

Backports the following commits to 7.x:
- [ML] Add data-test-subj to some job management elements (#35284)
@peteharverson peteharverson added the release_note:skip Skip the PR/issue when compiling release notes label May 22, 2019
patrykkopycinski pushed a commit to patrykkopycinski/kibana that referenced this pull request May 6, 2026
This PR adds a `data-test-subj` attribute with a descriptive value to the following elements of the ML job management page:
- Create new job button
- Refresh job list button
- Job list filter bar
- Jobs table
- Delete job modal

This will help locate these elements during UI automation more reliably.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:ml release_note:skip Skip the PR/issue when compiling release notes v7.0.1 v7.2.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants