Skip to content

Commit

Permalink
[Alerting] use input element for typing (opensearch-project#872)
Browse files Browse the repository at this point in the history
When selecting delegate monitors for composite monitor creation, we are typing in the combo box div element which is throwing invalid typing element error. This PR fixes that by selecting the child input element instead.

This change is already part of opensearch-project#866 for 2.x

Issues Resolved:

opensearch-project/alerting-dashboards-plugin#728

Signed-off-by: Amardeepsingh Siglani <[email protected]>
  • Loading branch information
amsiglan committed Sep 19, 2023
1 parent 34db037 commit f06559a
Showing 1 changed file with 4 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -76,13 +76,13 @@ describe('CompositeLevelMonitor', () => {
});

// Select associated monitors
cy.get('[data-test-subj="monitors_list_0"]').type('monitorOne', {
cy.get('[id="associatedMonitorsList_0"]').type('monitorOne', {
delay: 50,
force: true,
});
cy.get('[title="monitorOne"]').click({ force: true });

cy.get('[data-test-subj="monitors_list_1"]').type('monitorTwo', {
cy.get('[id="associatedMonitorsList_1"]').type('monitorTwo', {
delay: 50,
force: true,
});
Expand Down Expand Up @@ -163,8 +163,9 @@ describe('CompositeLevelMonitor', () => {

cy.get('button').contains('Add another monitor').click({ force: true });

cy.get('[data-test-subj="monitors_list_2"]').type('monitorThree', {
cy.get('[id="associatedMonitorsList_2"]').type('monitorThree', {
delay: 50,
force: true,
});
cy.get('[title="monitorThree"]').click({ force: true });

Expand Down

0 comments on commit f06559a

Please sign in to comment.