Skip to content

Commit 1adf003

Browse files
committed
Updated test names per coderabbit
1 parent 45d986b commit 1adf003

File tree

3 files changed

+2
-9
lines changed

3 files changed

+2
-9
lines changed

packages/react-core/src/components/Page/__tests__/Page.test.tsx

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -407,13 +407,6 @@ describe('Page', () => {
407407
expect(screen.getByTestId('page')).not.toHaveClass(styles.modifiers.dock);
408408
});
409409

410-
test(`Renders with ${styles.pageDockMain} wrapper when variant is docked`, () => {
411-
render(<Page masthead={<>Masthead</>} data-testid="page"></Page>);
412-
413-
const pageDockMain = screen.getByText('Masthead').closest(`.${styles.pageDockMain}`);
414-
expect(pageDockMain).not.toBeInTheDocument();
415-
});
416-
417410
test(`Renders with ${styles.pageDockMain} wrapper when variant is docked`, () => {
418411
render(<Page variant="docked" masthead={<>Masthead</>} data-testid="page"></Page>);
419412

packages/react-core/src/components/Page/__tests__/PageGroup.test.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,7 @@ test(`Renders with ${styles.modifiers.plain} class when isPlain is true`, () =>
9696
expect(screen.getByText('test')).toHaveClass(styles.modifiers.plain);
9797
});
9898

99-
test(`Renders with ${styles.modifiers.noPlainOnGlass} class when isPlain is true`, () => {
99+
test(`Renders with ${styles.modifiers.noPlainOnGlass} class when isNoPlainOnGlass is true`, () => {
100100
render(<PageGroup isNoPlainOnGlass>test</PageGroup>);
101101

102102
expect(screen.getByText('test')).toHaveClass(styles.modifiers.noPlainOnGlass);

packages/react-core/src/components/Page/__tests__/PageSection.test.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -190,7 +190,7 @@ test(`Renders with ${styles.modifiers.plain} class when isPlain is true`, () =>
190190
expect(screen.getByText('test')).toHaveClass(styles.modifiers.plain);
191191
});
192192

193-
test(`Renders with ${styles.modifiers.noPlainOnGlass} class when isPlain is true`, () => {
193+
test(`Renders with ${styles.modifiers.noPlainOnGlass} class when isNoPlainOnGlass is true`, () => {
194194
render(
195195
<PageSection hasBodyWrapper={false} isNoPlainOnGlass>
196196
test

0 commit comments

Comments
 (0)