fix tests

hey-car · May 25, 2023 · e2eb105 · e2eb105
1 parent 62436a7
commit e2eb105
Show file tree

Hide file tree

Showing 8 changed files with 12 additions and 7 deletions.
diff --git a/packages/header/src/__tests__/Header.screenshots.test.tsx b/packages/header/src/__tests__/Header.screenshots.test.tsx
@@ -182,7 +182,7 @@ describe('Header screenshots', () => {
       test('header open nav item w/ 1 subnav group', async () => {
         await screenshotMatchClick(
           mainPageURL,
-          '#root button[aria-label="Navigation menu"], nav[data-nav-type="burger-menu"] #nav-item-Carreviews',
+          '#root button[aria-label="Navigation menu"], nav[data-nav-type="burger-menu"] #burger-nav-item-Carreviews',
           {
             animations: 'disabled',
           },
@@ -195,7 +195,7 @@ describe('Header screenshots', () => {
       test('header open nav item w/ 2 subnav groups', async () => {
         await screenshotMatchClick(
           mainPageURL,
-          '#root button[aria-label="Navigation menu"], nav[data-nav-type="burger-menu"] #nav-item-CarsbyLocation',
+          '#root button[aria-label="Navigation menu"], nav[data-nav-type="burger-menu"] #burger-nav-item-CarsbyLocation',
           {
             animations: 'disabled',
           },
@@ -208,7 +208,7 @@ describe('Header screenshots', () => {
       test('header open nav item w/ 3 subnav groups', async () => {
         await screenshotMatchClick(
           mainPageURL,
-          '#root button[aria-label="Navigation menu"], nav[data-nav-type="burger-menu"] #nav-item-NewsGuides',
+          '#root button[aria-label="Navigation menu"], nav[data-nav-type="burger-menu"] #burger-nav-item-NewsGuides',
           {
             animations: 'disabled',
           },

diff --git a/...er screenshots mobile nav items header open nav item w/ 1 subnav group-snap.png b/...er screenshots mobile nav items header open nav item w/ 1 subnav group-snap.png
diff --git a/...r screenshots mobile nav items header open nav item w/ 2 subnav groups-snap.png b/...r screenshots mobile nav items header open nav item w/ 2 subnav groups-snap.png
diff --git a/...r screenshots mobile nav items header open nav item w/ 3 subnav groups-snap.png b/...r screenshots mobile nav items header open nav item w/ 3 subnav groups-snap.png
diff --git a/...Header screenshots tablet-l header items All header items visual state-snap.png b/...Header screenshots tablet-l header items All header items visual state-snap.png
diff --git a/packages/header/src/__tests__/headerItemHelpers.test.tsx b/packages/header/src/__tests__/headerItemHelpers.test.tsx
@@ -144,7 +144,7 @@ describe('headerItemHelpers', () => {
         hasHeaderItems(
           { label: 'search', hide: false, Component: <a /> },
           { label: 'favs', hide: undefined },
-          { currentLang: 'en-EN', hide: false },
+          { currentLang: 'en-GB', hide: false },
           { label: 'account' },
           { label: 'call', hide: false },
         );
@@ -161,7 +161,7 @@ describe('headerItemHelpers', () => {
         hasHeaderItems(
           { label: 'search', hide: true, Component: <a /> },
           undefined,
-          { currentLang: 'en-EN', hide: true },
+          { currentLang: 'en-GB', hide: true },
           undefined,
           undefined,
         );

diff --git a/packages/header/src/__tests__/navigationHelpers.test.tsx b/packages/header/src/__tests__/navigationHelpers.test.tsx
@@ -161,7 +161,7 @@ describe('navigationHelpers', () => {
       },
     ];
     const langItemConfig = {
-      currentLang: 'en-EN',
+      currentLang: 'en-GB',
       options: LANG_OPTIONS,
       hide: true,
     };
@@ -189,7 +189,7 @@ describe('navigationHelpers', () => {
         ...testNav,
         {
           isBurgerMenuOnly: true,
-          label: 'Select Language',
+          label: 'English',
           subNavGroups: [
             {
               heading: DEFAULT_LOCALE.langListHeading,

diff --git a/packages/header/src/styles/navigationDropdown.module.css b/packages/header/src/styles/navigationDropdown.module.css
@@ -14,6 +14,11 @@
   color: #fff;
   transition: color 0.3s ease, background 0.3s ease;
 
+  & > span {
+    width: 100%;
+    white-space: nowrap;
+  }
+
   &.isActive,
   &.isCurrentPage,
   &:hover,