Skip to content

Commit 79b3890

Browse files
committed
test: Fix test case errors.
1 parent bccec47 commit 79b3890

File tree

4 files changed

+24
-25
lines changed

4 files changed

+24
-25
lines changed

package.json

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@
1414
"watch": "tsbb watch",
1515
"build": "tsbb build",
1616
"test": "tsbb test --env=jsdom",
17-
"coverage": "tsbb test --coverage",
17+
"coverage": "tsbb test --coverage --bail",
1818
"doc": "kkt build --app-src ./website",
1919
"start": "kkt start --app-src ./website",
2020
"prettier": "prettier --write '**/*.{js,jsx,tsx,ts,html,less,md,json}'"
@@ -73,31 +73,31 @@
7373
"@codemirror/lang-cpp": "0.19.1",
7474
"@codemirror/lang-html": "0.19.4",
7575
"@codemirror/lang-java": "0.19.1",
76-
"@codemirror/lang-javascript": "0.19.3",
76+
"@codemirror/lang-javascript": "0.19.4",
7777
"@codemirror/lang-json": "0.19.1",
7878
"@codemirror/lang-lezer": "0.19.1",
79-
"@codemirror/lang-markdown": "0.19.3",
79+
"@codemirror/lang-markdown": "0.19.4",
8080
"@codemirror/lang-php": "0.19.1",
8181
"@codemirror/lang-python": "0.19.2",
8282
"@codemirror/lang-rust": "0.19.1",
8383
"@codemirror/lang-sql": "0.19.4",
8484
"@codemirror/lang-xml": "0.19.2",
8585
"@codemirror/legacy-modes": "0.19.0",
8686
"@codemirror/stream-parser": "0.19.3",
87-
"@kkt/less-modules": "7.0.4",
88-
"@kkt/raw-modules": "7.0.4",
89-
"@kkt/scope-plugin-options": "7.0.4",
87+
"@kkt/less-modules": "7.0.5",
88+
"@kkt/raw-modules": "7.0.5",
89+
"@kkt/scope-plugin-options": "7.0.5",
9090
"@types/react": "17.0.38",
9191
"@types/react-dom": "17.0.11",
9292
"@types/react-test-renderer": "17.0.1",
9393
"@uiw/react-github-corners": "1.5.3",
94-
"@uiw/react-markdown-preview": "3.4.5",
94+
"@uiw/react-markdown-preview": "3.4.7",
9595
"@uiw/react-shields": "1.1.2",
9696
"@uiw/reset.css": "1.0.5",
9797
"code-example": "3.3.1",
9898
"husky": "7.0.4",
99-
"kkt": "7.0.4",
100-
"lint-staged": "12.1.4",
99+
"kkt": "7.0.5",
100+
"lint-staged": "12.1.7",
101101
"prettier": "2.5.1",
102102
"react": "17.0.2",
103103
"react-dom": "17.0.2",

src/index.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,7 @@ export interface ReactCodeMirrorRef {
7070
const ReactCodeMirror = React.forwardRef<ReactCodeMirrorRef, ReactCodeMirrorProps>((props, ref) => {
7171
const {
7272
className,
73-
value,
73+
value = '',
7474
selection,
7575
extensions = [],
7676
onChange,

src/react-app-env.d.ts

Lines changed: 0 additions & 1 deletion
This file was deleted.

test/index.test.tsx

Lines changed: 14 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,12 @@
1+
/**
2+
* @jest-environment jsdom
3+
*/
14
/* eslint-disable jest/no-conditional-expect */
25
import React, { useEffect, useRef } from 'react';
36
import renderer from 'react-test-renderer';
4-
import { render, fireEvent } from '@testing-library/react';
5-
import userEvent from '@testing-library/user-event';
6-
// import '@testing-library/jest-dom';
7+
import { render, fireEvent, screen } from '@testing-library/react';
78
// import userEvent from '@testing-library/user-event';
9+
// import '@testing-library/jest-dom';
810
import CodeMirror, { ReactCodeMirrorRef } from '../src';
911

1012
it('CodeMirror', async () => {
@@ -21,12 +23,10 @@ it('CodeMirror onChange', async () => {
2123
expect(value).toEqual('# title');
2224
return Array.isArray(value) ? value.join() : value;
2325
});
24-
const { findByRole, queryByText } = render(
25-
<CodeMirror autoFocus value="console.log('Hello world!')" onChange={handleChange} />,
26-
);
27-
const input = await findByRole('textbox');
26+
render(<CodeMirror autoFocus value="console.log('Hello world!')" onChange={handleChange} />);
27+
const input = await screen.findByRole<HTMLInputElement>('textbox'); // findByRole('textbox');
2828
fireEvent.change(input, { target: { textContent: '# title' } });
29-
const elm = queryByText('# title');
29+
const elm = screen.queryByText('# title');
3030
expect((elm as any).cmView.dom.innerHTML).toEqual('# title');
3131
});
3232

@@ -73,15 +73,15 @@ it('CodeMirror className', async () => {
7373
});
7474

7575
it('CodeMirror placeholder', async () => {
76-
const { findByText } = render(<CodeMirror placeholder="Hello World" className="test" />);
77-
const text = await findByText('Hello World');
78-
expect(text.className).toEqual('cm-placeholder');
79-
expect(text.contentEditable).toEqual('false');
76+
render(<CodeMirror placeholder="Hello World" className="test" />);
77+
const elm = screen.queryByText('Hello World');
78+
expect(elm!.style['pointerEvents']).toEqual('none');
79+
expect(elm!.className).toEqual('cm-placeholder');
8080
});
8181

8282
it('CodeMirror editable', async () => {
83-
const { getByRole } = render(<CodeMirror editable={false} className="test" />);
84-
const text = getByRole('textbox');
83+
render(<CodeMirror editable={false} className="test" />);
84+
const text = screen.getByRole('textbox');
8585
expect(text.className).toEqual('cm-content');
8686
expect(text.tagName).toEqual('DIV');
8787
});

0 commit comments

Comments
 (0)