Skip to content

Commit 550eb3c

Browse files
authored
chore: reorganize app layout feature flags (#3739)
1 parent 8767da0 commit 550eb3c

File tree

4 files changed

+5
-5
lines changed

4 files changed

+5
-5
lines changed

src/app-layout/__integ__/global-breadcrumbs.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ describe('classic', () => {
5858
'does not react to the feature flag even if it is enabled',
5959
setupTest(
6060
{
61-
url: `#/light/app-layout/global-breadcrumbs/?${new URLSearchParams({ visualRefresh: 'false', appLayoutWidget: 'true', hasOwnBreadcrumbs: 'true' }).toString()}`,
61+
url: `#/light/app-layout/global-breadcrumbs/?${new URLSearchParams({ visualRefresh: 'false', appLayoutToolbar: 'true', hasOwnBreadcrumbs: 'true' }).toString()}`,
6262
},
6363
async page => {
6464
await page.toggleExtraBreadcrumb();

src/app-layout/__integ__/utils.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,7 @@ export const setupTest = (
7474
const page = new AppLayoutDrawersPage(browser);
7575
const params = new URLSearchParams({
7676
visualRefresh: `${theme !== 'classic'}`,
77-
appLayoutWidget: `${theme === 'refresh-toolbar'}`,
77+
appLayoutToolbar: `${theme === 'refresh-toolbar'}`,
7878
...(splitPanelPosition ? { splitPanelPosition } : {}),
7979
}).toString();
8080
await browser.url(`#/light/app-layout/with-drawers?${params}`);

src/app-layout/__tests__/trigger-button.test.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -98,7 +98,7 @@ const renderVisualRefreshToolbarTriggerButton = (
9898
return { wrapper, rerender, getByTestId, getByText, container };
9999
};
100100

101-
describe('Visual refresh trigger-button (not in appLayoutWidget toolbar)', () => {
101+
describe('Visual refresh trigger-button (not in appLayoutToolbar toolbar)', () => {
102102
beforeEach(() => jest.clearAllMocks());
103103

104104
describe.each([true, false])('Toolbar trigger-button with isMobile=%s', isMobile => {

src/app-layout/__tests__/utils.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -59,12 +59,12 @@ export function describeEachAppLayout(
5959
beforeEach(() => {
6060
globalWithFlags[forceMobileModeSymbol] = size === 'mobile';
6161
globalWithFlags[Symbol.for('awsui-visual-refresh-flag')] = () => theme !== 'classic';
62-
setGlobalFlag('appLayoutWidget', theme === 'refresh-toolbar');
62+
setGlobalFlag('appLayoutToolbar', theme === 'refresh-toolbar');
6363
});
6464
afterEach(() => {
6565
delete globalWithFlags[forceMobileModeSymbol];
6666
delete globalWithFlags[Symbol.for('awsui-visual-refresh-flag')];
67-
setGlobalFlag('appLayoutWidget', undefined);
67+
setGlobalFlag('appLayoutToolbar', undefined);
6868
clearVisualRefreshState();
6969
});
7070
test('mocks applied correctly', () => {

0 commit comments

Comments
 (0)