Skip to content

Commit e490afc

Browse files
committed
Add some basic tests
1 parent 12b97ed commit e490afc

File tree

2 files changed

+27
-0
lines changed

2 files changed

+27
-0
lines changed

Diff for: src/components/tests/terminal-picker.tsx

+26
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,26 @@
1+
import { render, screen, fireEvent } from '@testing-library/react';
2+
import { TerminalPicker } from 'src/components/terminal-picker';
3+
4+
describe( 'TerminalPicker', () => {
5+
const mockOnChange = jest.fn();
6+
7+
beforeEach( () => {
8+
jest.clearAllMocks();
9+
} );
10+
11+
it( 'renders correctly with initial props', () => {
12+
render( <TerminalPicker value="terminal" onChange={ mockOnChange } /> );
13+
14+
expect( screen.getByText( 'Shell' ) ).toBeVisible();
15+
expect( screen.getByRole( 'combobox' ) ).toBeVisible();
16+
} );
17+
18+
it( 'calls onChange when selecting a different terminal', async () => {
19+
render( <TerminalPicker value="terminal" onChange={ mockOnChange } /> );
20+
21+
const select = screen.getByRole( 'combobox' );
22+
fireEvent.change( select, { target: { value: 'iterm' } } );
23+
24+
expect( mockOnChange ).toHaveBeenCalledWith( 'iterm', expect.anything() );
25+
} );
26+
} );

Diff for: src/components/tests/user-settings.test.tsx

+1
Original file line numberDiff line numberDiff line change
@@ -87,6 +87,7 @@ describe( 'UserSettings', () => {
8787
fireEvent.click( screen.getByText( 'Preferences' ) );
8888
expect( screen.getByText( 'Preferences' ) ).toHaveAttribute( 'aria-selected', 'true' );
8989
expect( screen.getByText( 'Language' ) ).toBeVisible();
90+
expect( screen.getByText( 'Shell' ) ).toBeVisible();
9091

9192
// Switch to Usage tab
9293
fireEvent.click( screen.getByText( 'Usage' ) );

0 commit comments

Comments
 (0)