Skip to content

Commit cd8fc1a

Browse files
committed
update unit test
1 parent f486954 commit cd8fc1a

1 file changed

Lines changed: 8 additions & 19 deletions

File tree

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

Lines changed: 8 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -129,33 +129,22 @@ test(`Renders without class ${styles.modifiers.noPlain} by default`, () => {
129129
expect(screen.getByText('Test')).not.toHaveClass(styles.modifiers.noPlain);
130130
});
131131

132-
test(`Renders without class ${styles.modifiers.noPlain} when isPlain is undefined`, () => {
133-
render(<Accordion isPlain={undefined}>Test</Accordion>);
132+
test(`Renders with class ${styles.modifiers.noPlain} when isNoPlainOnGlass`, () => {
133+
render(<Accordion isNoPlainOnGlass>Test</Accordion>);
134134

135-
expect(screen.getByText('Test')).not.toHaveClass(styles.modifiers.noPlain);
135+
expect(screen.getByText('Test')).toHaveClass(styles.modifiers.noPlain);
136136
});
137137

138-
test(`Renders without class ${styles.modifiers.noPlain} when isPlain=false and glass theme is not applied`, () => {
139-
render(<Accordion isPlain={false}>Test</Accordion>);
140-
141-
expect(screen.getByText('Test')).not.toHaveClass(styles.modifiers.noPlain);
142-
});
143-
144-
test(`Renders with class ${styles.modifiers.noPlain} when isPlain=false and glass theme is applied`, () => {
145-
document.documentElement.classList.add('pf-v6-theme-glass');
146-
try {
147-
render(<Accordion isPlain={false}>Test</Accordion>);
138+
test(`Renders without class ${styles.modifiers.plain} by default`, () => {
139+
render(<Accordion>Test</Accordion>);
148140

149-
expect(screen.getByText('Test')).toHaveClass(styles.modifiers.noPlain);
150-
} finally {
151-
document.documentElement.classList.remove('pf-v6-theme-glass');
152-
}
141+
expect(screen.getByText('Test')).not.toHaveClass(styles.modifiers.plain);
153142
});
154143

155-
test(`Renders without class ${styles.modifiers.noPlain} when isPlain=true`, () => {
144+
test(`Renders with class ${styles.modifiers.plain} when isPlain`, () => {
156145
render(<Accordion isPlain>Test</Accordion>);
157146

158-
expect(screen.getByText('Test')).not.toHaveClass(styles.modifiers.noPlain);
147+
expect(screen.getByText('Test')).toHaveClass(styles.modifiers.plain);
159148
});
160149

161150
test('Renders without pf-m-display-lg by default', () => {

0 commit comments

Comments
 (0)