From 629729756f35a70e91c161741bf7ace4461d4c3d Mon Sep 17 00:00:00 2001 From: Mauricio Gonzalez <mgonzalez.btig@softwarecraftcr.com> Date: Sun, 27 Jun 2021 20:54:49 -0600 Subject: [PATCH] Fix js test --- app/assets/javascripts/main.js | 1 - spec/frontend/nav/components/responsive_app_spec.js | 10 ++++------ 2 files changed, 4 insertions(+), 7 deletions(-) diff --git a/app/assets/javascripts/main.js b/app/assets/javascripts/main.js index 7a4963dd8134b..1aaefcaa13b7a 100644 --- a/app/assets/javascripts/main.js +++ b/app/assets/javascripts/main.js @@ -35,7 +35,6 @@ import GlFieldErrors from './gl_field_errors'; import initUserPopovers from './user_popovers'; import initBroadcastNotifications from './broadcast_notification'; import { initTopNav } from './nav'; -import navEventHub, { EVENT_RESPONSIVE_TOGGLE } from './nav/event_hub'; import 'ee_else_ce/main_ee'; diff --git a/spec/frontend/nav/components/responsive_app_spec.js b/spec/frontend/nav/components/responsive_app_spec.js index e1b443745e389..fbf4d54baa259 100644 --- a/spec/frontend/nav/components/responsive_app_spec.js +++ b/spec/frontend/nav/components/responsive_app_spec.js @@ -3,7 +3,6 @@ import ResponsiveApp from '~/nav/components/responsive_app.vue'; import ResponsiveHeader from '~/nav/components/responsive_header.vue'; import ResponsiveHome from '~/nav/components/responsive_home.vue'; import TopNavContainerView from '~/nav/components/top_nav_container_view.vue'; -import eventHub, { EVENT_RESPONSIVE_TOGGLE } from '~/nav/event_hub'; import { resetMenuItemsActive } from '~/nav/utils/reset_menu_items_active'; import KeepAliveSlots from '~/vue_shared/components/keep_alive_slots.vue'; import { TEST_NAV_DATA } from '../mock_data'; @@ -26,8 +25,7 @@ describe('~/nav/components/responsive_app.vue', () => { }, }); }; - const triggerResponsiveToggle = () => eventHub.$emit(EVENT_RESPONSIVE_TOGGLE); - + const triggerResponsiveToggle = () => document.body.classList.toggle('top-nav-responsive-open'); const findHome = () => wrapper.findComponent(ResponsiveHome); const findMobileOverlay = () => wrapper.find('[data-testid="mobile-overlay"]'); const findSubviewHeader = () => wrapper.findComponent(ResponsiveHeader); @@ -68,7 +66,8 @@ describe('~/nav/components/responsive_app.vue', () => { ({ bodyHtml, expectation }) => { document.body.innerHTML = bodyHtml; - triggerResponsiveToggle(); + if (bodyHtml === HTML_HEADER_WITH_MENU_EXPANDED) + triggerResponsiveToggle(); expect(hasBodyResponsiveOpen()).toBe(expectation); }, @@ -99,6 +98,7 @@ describe('~/nav/components/responsive_app.vue', () => { describe('with menu expanded in body', () => { beforeEach(() => { document.body.innerHTML = HTML_HEADER_WITH_MENU_EXPANDED; + triggerResponsiveToggle(); createComponent(); }); @@ -167,8 +167,6 @@ describe('~/nav/components/responsive_app.vue', () => { }); it('responsive toggle event does nothing', () => { - triggerResponsiveToggle(); - expect(hasBodyResponsiveOpen()).toBe(false); }); }); -- GitLab