fix: Minor review comments

cloudscape-design · Sep 10, 2024 · af49dab · af49dab
1 parent 0dbe0d7
commit af49dab
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/src/app-layout/__tests__/runtime-drawers-layout.test.tsx b/src/app-layout/__tests__/runtime-drawers-layout.test.tsx
@@ -2,7 +2,7 @@
 // SPDX-License-Identifier: Apache-2.0
 /* eslint simple-import-sort/imports: 0 */
 import React from 'react';
-import { act, render } from '@testing-library/react';
+import { act, render, waitFor } from '@testing-library/react';
 import { describeEachAppLayout, getGlobalDrawersTestUtils, testDrawer } from './utils';
 import AppLayout from '../../../lib/components/app-layout';
 import { awsuiPlugins, awsuiPluginsInternal } from '../../../lib/components/internal/plugins/api';
@@ -107,9 +107,9 @@ describe('toolbar mode only features', () => {
 
       wrapper.findDrawerTriggerById('local-drawer')!.click();
 
-      await delay();
-
-      expect(globalDrawersWrapper.findActiveDrawers()!.length).toBe(2);
+      await waitFor(() => {
+        expect(globalDrawersWrapper.findActiveDrawers()!.length).toBe(2);
+      });
       expect(globalDrawersWrapper.findActiveDrawers()[0].getElement()).toHaveTextContent('local-drawer');
       expect(globalDrawersWrapper.findActiveDrawers()[1].getElement()).toHaveTextContent('global drawer content 2');
     });

diff --git a/src/app-layout/__tests__/runtime-drawers.test.tsx b/src/app-layout/__tests__/runtime-drawers.test.tsx
@@ -697,7 +697,7 @@ describe('toolbar mode only features', () => {
 
       await delay();
 
-      expect(wrapper.findDrawersTriggers()!.length).toBe(4);
+      expect(wrapper.findDrawersTriggers().length).toBe(4);
       expect(wrapper.find(`.${toolbarStyles['group-divider']}`)!.getElement()).toBeInTheDocument();
       expect(globalDrawersWrapper.findActiveDrawers()!.length).toBe(3);
       expect(globalDrawersWrapper.findDrawerById('local-drawer')!.getElement()).toHaveTextContent(