small fixes to tests, clearifying test names

cloudscape-design · Oct 18, 2024 · 8ab819e · 8ab819e
1 parent c20704f
commit 8ab819e
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/app-layout/__tests__/multi-layout-props.test.tsx b/src/app-layout/__tests__/multi-layout-props.test.tsx
@@ -1,7 +1,7 @@
 // Copyright Amazon.com, Inc. or its affiliates. All Rights Reserved.
 // SPDX-License-Identifier: Apache-2.0
-/* eslint-disable simple-import-sort/imports */
 import React from 'react';
+
 import {
   mergeMultiAppLayoutProps,
   SharedMultiAppLayoutProps,

diff --git a/src/app-layout/__tests__/multi-layout.test.tsx b/src/app-layout/__tests__/multi-layout.test.tsx
@@ -75,7 +75,7 @@ describeEachAppLayout({ themes: ['refresh-toolbar'], sizes: ['desktop'] }, () =>
     expect(isDrawerClosed(firstLayout.findNavigation())).toEqual(false);
   });
 
-  test('merges navigation from two instances with navigation hidden in primary', async () => {
+  test('navigationHide in primary is respected when navigation is defined when merging from two instances', async () => {
     const { firstLayout, secondLayout } = await renderAsync(
       <AppLayout
         {...defaultAppLayoutProps}
@@ -94,7 +94,7 @@ describeEachAppLayout({ themes: ['refresh-toolbar'], sizes: ['desktop'] }, () =>
     expect(secondLayout.findNavigationToggle()).toBeFalsy();
   });
 
-  test('merges tools from two instances with navigation hidden in secondary', async () => {
+  test('merges tools from two instances with where navigationHide is true in secondary', async () => {
     const { firstLayout, secondLayout } = await renderAsync(
       <AppLayout
         {...defaultAppLayoutProps}