From d4ed972f6d8667697d40b59cde8ae3e16a992ad0 Mon Sep 17 00:00:00 2001 From: Nick LaMuro Date: Thu, 9 Jul 2020 20:33:55 -0500 Subject: [PATCH] [REMOVEME][menu.spec.js] Fix bad menu navigations 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: 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", () => {