Skip to content

Commit

Permalink
[REMOVEME][search_box.spec.js] Fix failing specs
Browse files Browse the repository at this point in the history
For both specs, they suffered from the menu changes
ManageIQ/manageiq-ui-classic PR 6963, so those have been addressed to
use some form of an equivalent.

For the first spec ("is present"), it also had a random "div click" in
there that either doesn't make sense any more, is entirely unnecessary,
or there is a disconnect between the author of this commit (me) and the
original author of the spec.  Regardless, commenting out the line in
question allows the spec to pass.
  • Loading branch information
NickLaMuro committed Jul 17, 2020
1 parent c900d41 commit c6c64df
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions cypress/integration/ui/searchbox.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,13 +4,14 @@ describe('Search box', () => {
});

it("is present", () => {
cy.menu("Configuration");
cy.menu("Overview", "Dashboard");
cy.menu("Compute", "Infrastructure", "Virtual Machines");
cy.get('div[class=panel-heading]').first().click();
cy.search_box();
});

it("is not present", () => {
cy.menu("Overview");
cy.menu("Overview", "Dashboard");
cy.no_search_box();
});
});

0 comments on commit c6c64df

Please sign in to comment.