Skip to content

Commit 45c0e9b

Browse files
committed
resolve todo items
1 parent c50a200 commit 45c0e9b

File tree

2 files changed

+1
-2
lines changed

2 files changed

+1
-2
lines changed

src/app-layout/__tests__/common.test.tsx

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -239,7 +239,6 @@ describeEachAppLayout(({ theme, size }) => {
239239
expect(findOpenElement(wrapper)).toBeFalsy();
240240
});
241241

242-
// TODO: is async focus management a breaking change?
243242
test('Moves focus between open and close buttons', async () => {
244243
// use content type with initial closed state for all drawers
245244
const { wrapper } = renderComponent(<AppLayout contentType="form" />);

src/app-layout/__tests__/mobile.test.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -365,7 +365,7 @@ describeEachAppLayout({ sizes: ['mobile'] }, ({ theme }) => {
365365
test('closes navigation via ref', async () => {
366366
let ref: AppLayoutProps.Ref | null = null;
367367
const { wrapper } = renderComponent(<AppLayout ref={newRef => (ref = newRef)} />);
368-
// TODO: is this a breaking change?
368+
369369
await act(() => Promise.resolve());
370370
wrapper.findNavigationToggle().click();
371371
await waitFor(() => {

0 commit comments

Comments
 (0)