Skip to content

Commit 0937109

Browse files
committed
Refactor useCurrentColorScheme test
1 parent 0ebf220 commit 0937109

File tree

1 file changed

+6
-9
lines changed

1 file changed

+6
-9
lines changed

packages/mui-system/src/cssVars/useCurrentColorScheme.test.js

Lines changed: 6 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -85,26 +85,23 @@ describe('useCurrentColorScheme', () => {
8585
});
8686

8787
it('trigger a re-render for a multi color schemes', () => {
88+
let effectRunCount = 0;
8889
function Data() {
8990
const { mode } = useCurrentColorScheme({
9091
supportedColorSchemes: ['light', 'dark'],
9192
defaultLightColorScheme: 'light',
9293
defaultDarkColorScheme: 'dark',
9394
});
94-
const count = React.useRef(0);
9595
React.useEffect(() => {
96-
count.current += 1;
96+
effectRunCount += 1;
9797
});
98-
return (
99-
<div>
100-
{mode}:{count.current}
101-
</div>
102-
);
98+
return <div>{mode}</div>;
10399
}
104-
const expectedCount = reactMajor >= 19 ? 1 : 2;
100+
105101
const { container } = render(<Data />);
106102

107-
expect(container.firstChild.textContent).to.equal(`light:${expectedCount}`);
103+
expect(container.firstChild.textContent).to.equal('light');
104+
expect(effectRunCount).to.equal(2);
108105
});
109106

110107
it('[noSsr] does not trigger a re-render', () => {

0 commit comments

Comments
 (0)