Skip to content

Commit 2d4f3a9

Browse files
committed
chore: fix tests
1 parent a07861e commit 2d4f3a9

File tree

4 files changed

+41
-39
lines changed

4 files changed

+41
-39
lines changed

apps/site/components/Common/Tabs/__tests__/index.test.mjs

Lines changed: 23 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -5,37 +5,35 @@ import userEvent from '@testing-library/user-event';
55
import Link from '../../../Link';
66
import Tabs from '../index';
77

8-
describe('Tabs', () => {
9-
const Sut = ({ addons }) => {
10-
const tabs = [
11-
{ key: 'package', label: 'Package Manager' },
12-
{ key: 'prebuilt', label: 'Prebuilt Installer' },
13-
{ key: 'source', label: 'Source Code' },
14-
];
15-
16-
return (
17-
<Tabs tabs={tabs} defaultValue="package" addons={addons}>
18-
<TabsPrimitive.Content value="package">
19-
Package Manager
20-
</TabsPrimitive.Content>
21-
<TabsPrimitive.Content value="prebuilt">
22-
Prebuilt Installer
23-
</TabsPrimitive.Content>
24-
<TabsPrimitive.Content value="source">
25-
Source Code
26-
</TabsPrimitive.Content>
27-
</Tabs>
28-
);
29-
};
8+
const Sut = ({ addons }) => {
9+
const tabs = [
10+
{ key: 'package', label: 'Package Manager' },
11+
{ key: 'prebuilt', label: 'Prebuilt Installer' },
12+
{ key: 'source', label: 'Source Code' },
13+
];
14+
15+
return (
16+
<Tabs tabs={tabs} defaultValue="package" addons={addons}>
17+
<TabsPrimitive.Content value="package">
18+
Package Manager
19+
</TabsPrimitive.Content>
20+
<TabsPrimitive.Content value="prebuilt">
21+
Prebuilt Installer
22+
</TabsPrimitive.Content>
23+
<TabsPrimitive.Content value="source">Source Code</TabsPrimitive.Content>
24+
</Tabs>
25+
);
26+
};
3027

28+
describe('Tabs', () => {
3129
it('should render the correct number of tabs', () => {
32-
render(<Sut />);
30+
act(() => render(<Sut />));
3331

3432
expect(screen.getAllByRole('tab')).toHaveLength(3);
3533
});
3634

3735
it('should render the correct tab content when clicked', async () => {
38-
render(<Sut />);
36+
act(() => render(<Sut />));
3937

4038
expect(screen.getByRole('tabpanel')).toHaveTextContent('Package Manager');
4139

@@ -47,7 +45,7 @@ describe('Tabs', () => {
4745
});
4846

4947
it('should render the given addons', async () => {
50-
render(<Sut addons={<Link href="/">addon</Link>} />);
48+
act(() => render(<Sut addons={<Link href="/">addon</Link>} />));
5149

5250
expect(screen.getByRole('link', { name: 'addon' })).toBeInTheDocument();
5351
});

apps/site/jest.setup.mjs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1 +1,3 @@
1+
globalThis.IS_REACT_ACT_ENVIRONMENT = true;
2+
13
import '@testing-library/jest-dom';
Lines changed: 15 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { render } from '@testing-library/react';
1+
import { act, render } from '@testing-library/react';
22

33
import { MatterProvider, MatterContext } from '@/providers/matterProvider';
44

@@ -9,21 +9,23 @@ const mockContext = {
99
readingTime: { text: '', minutes: 0, time: 0, words: 0 },
1010
filename: '',
1111
os: 'LOADING',
12-
architecture: '',
13-
bitness: '',
12+
architecture: expect.any(String),
13+
bitness: expect.any(String),
1414
};
1515

1616
describe('MatterProvider', () => {
17-
it('renders the provider with the provided context value', () => {
18-
render(
19-
<MatterProvider os="LOADING">
20-
<MatterContext.Consumer>
21-
{value => {
22-
expect(value).toEqual(mockContext);
23-
return null;
24-
}}
25-
</MatterContext.Consumer>
26-
</MatterProvider>
17+
it('renders the provider with the provided context value', async () => {
18+
await act(async () =>
19+
render(
20+
<MatterProvider os="LOADING">
21+
<MatterContext.Consumer>
22+
{value => {
23+
expect(value).toEqual(mockContext);
24+
return null;
25+
}}
26+
</MatterContext.Consumer>
27+
</MatterProvider>
28+
)
2729
);
2830
});
2931
});

apps/site/providers/__tests__/themeProvider.test.mjs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ describe('ThemeProvider', () => {
2323
storageKey: 'theme',
2424
children: <div>Mock Component</div>,
2525
},
26-
expect.any(Object)
26+
undefined
2727
);
2828
expect(container.firstChild).toHaveTextContent('Mock Component');
2929
});

0 commit comments

Comments
 (0)