From fb18588e98327aa49a523f511de13904d3dbd60e Mon Sep 17 00:00:00 2001 From: Nick LaMuro Date: Thu, 9 Jul 2020 20:33:55 -0500 Subject: [PATCH] [menu.spec.js] Fix bad menu navigations 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: https://github.com/ManageIQ/manageiq-ui-classic/pull/6963 --- cypress/integration/ui/menu.spec.js | 18 ++++++++++-------- 1 file changed, 10 insertions(+), 8 deletions(-) diff --git a/cypress/integration/ui/menu.spec.js b/cypress/integration/ui/menu.spec.js index 7b7d534837fa..f4f65476bf8e 100644 --- a/cypress/integration/ui/menu.spec.js +++ b/cypress/integration/ui/menu.spec.js @@ -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", () => {