From 40887ef42e1e8d9e2fcbfc1bb65ed97ab7b64405 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 | 16 ++++++---------- 1 file changed, 6 insertions(+), 10 deletions(-) diff --git a/cypress/integration/ui/menu.spec.js b/cypress/integration/ui/menu.spec.js index 7b7d534837fa..0f6a064d32c8 100644 --- a/cypress/integration/ui/menu.spec.js +++ b/cypress/integration/ui/menu.spec.js @@ -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", () => {