Skip to content

Commit a0d2bc6

Browse files
authored
fix: Make sure clearVisualRefreshState helper removes the class too (#118)
1 parent 9b48c5e commit a0d2bc6

File tree

2 files changed

+7
-2
lines changed

2 files changed

+7
-2
lines changed

src/internal/visual-mode/__tests__/use-runtime-visual-refresh.test.tsx

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,8 +18,10 @@ describe('useVisualRefresh', () => {
1818
return <div data-testid="current-mode">{isRefresh.toString()}</div>;
1919
}
2020

21-
beforeEach(() => clearVisualRefreshState());
22-
afterEach(() => document.body.classList.remove('awsui-visual-refresh'));
21+
afterEach(() => {
22+
clearVisualRefreshState();
23+
expect(document.querySelector('.awsui-visual-refresh')).toBeFalsy();
24+
});
2325
afterEach(() => {
2426
clearMessageCache();
2527
jest.restoreAllMocks();

src/internal/visual-mode/index.ts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -107,6 +107,9 @@ let visualRefreshState: undefined | boolean = undefined;
107107
// for testing
108108
export function clearVisualRefreshState() {
109109
visualRefreshState = undefined;
110+
if (typeof document !== 'undefined') {
111+
document.body.classList.remove('awsui-visual-refresh');
112+
}
110113
}
111114

112115
function detectVisualRefreshClassName() {

0 commit comments

Comments
 (0)