Skip to content

Commit

Permalink
[menu.spec.js] Fix bad menu navigations
Browse files Browse the repository at this point in the history
Clicking top level menu items is no longer supported with the Carbon
styles, so fixing the specs to better reflect the current functionality,
or commenting out places where it isn't valid.

Most likely broken because of these changes:

ManageIQ#6963
  • Loading branch information
NickLaMuro committed Jul 17, 2020
1 parent 9d7bee7 commit 95de4e6
Showing 1 changed file with 6 additions and 10 deletions.
16 changes: 6 additions & 10 deletions cypress/integration/ui/menu.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -14,24 +14,20 @@ describe('Menu', () => {
expect(menu[2].items[1].items[3].title).to.equal('Virtual Machines');
});

cy.menu('Overview')
cy.menu('Overview', 'Dashboard')
.get('widget-wrapper');

cy.menu('Services')
cy.menu('Overview', 'Reports')
.expect_explorer_title('All Saved Reports');

cy.menu('Services', 'My Services')
.expect_explorer_title('Active Services');

cy.menu('Compute')
cy.menu('Compute', 'Clouds', 'Providers')
.expect_show_list_title('Cloud Providers');

cy.menu('Overview', 'Reports')
.expect_explorer_title('All Saved Reports');

cy.menu('Compute', 'Infrastructure', 'Virtual Machines')
.expect_explorer_title('All VMs & Templates');

// test it remembers the last Overview > * screen used
cy.menu('Overview')
.expect_explorer_title('All Saved Reports');
});

it.skip("all menu items lead to non-error screens", () => {
Expand Down

0 comments on commit 95de4e6

Please sign in to comment.