Skip to content

Commit

Permalink
[REMOVEME][menu.spec.js] Fix bad menu navigations
Browse files Browse the repository at this point in the history
NOTE:  Pulling this in from another branch, but is already part of a
separate PR.  Just doing to help make CI "less red"

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:

#6963
  • Loading branch information
NickLaMuro committed Jul 15, 2020
1 parent b261774 commit d4ed972
Showing 1 changed file with 10 additions and 8 deletions.
18 changes: 10 additions & 8 deletions cypress/integration/ui/menu.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -14,24 +14,26 @@ 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');

// FIXME: This isn't valid functionality for the menu at this time.
//
// test it remembers the last Overview > * screen used
cy.menu('Overview')
.expect_explorer_title('All Saved Reports');
// 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 d4ed972

Please sign in to comment.