Skip to content

Commit 66f1159

Browse files
hiroppyhiroppy
committed
chore: fix tests (#1997)
1 parent a4b93cf commit 66f1159

File tree

10 files changed

+135
-26
lines changed

10 files changed

+135
-26
lines changed

‎client-src/clients/BaseClient.js

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,3 @@
1-
'use strict';
2-
31
/* eslint-disable
42
no-unused-vars
53
*/

‎client-src/clients/SockJSClient.js

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,3 @@
1-
'use strict';
2-
31
/* eslint-disable
42
no-unused-vars
53
*/

‎client-src/clients/WebsocketClient.js

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,3 @@
1-
'use strict';
2-
31
const BaseClient = require('./BaseClient');
42

53
module.exports = class WebsocketClient extends BaseClient {};

‎test/__snapshots__/Server.test.js.snap

Lines changed: 116 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,116 @@
1+
// Jest Snapshot v1, https://goo.gl/fbAQLP
2+
3+
exports[`Server addEntries add hot option 1`] = `
4+
Array [
5+
Array [
6+
"client",
7+
"index.js?http:",
8+
"localhost",
9+
],
10+
Array [
11+
"node_modules",
12+
"webpack",
13+
"hot",
14+
"dev-server.js",
15+
],
16+
Array [
17+
"foo.js",
18+
],
19+
]
20+
`;
21+
22+
exports[`Server addEntries add hot option 2`] = `
23+
Array [
24+
HotModuleReplacementPlugin {
25+
"fullBuildTimeout": 200,
26+
"multiStep": undefined,
27+
"options": Object {},
28+
"requestTimeout": 10000,
29+
},
30+
]
31+
`;
32+
33+
exports[`Server addEntries add hotOnly option 1`] = `
34+
Array [
35+
Array [
36+
"client",
37+
"index.js?http:",
38+
"localhost",
39+
],
40+
Array [
41+
"node_modules",
42+
"webpack",
43+
"hot",
44+
"only-dev-server.js",
45+
],
46+
Array [
47+
"foo.js",
48+
],
49+
]
50+
`;
51+
52+
exports[`Server addEntries add hotOnly option 2`] = `
53+
Array [
54+
HotModuleReplacementPlugin {
55+
"fullBuildTimeout": 200,
56+
"multiStep": undefined,
57+
"options": Object {},
58+
"requestTimeout": 10000,
59+
},
60+
]
61+
`;
62+
63+
exports[`Server stats should works with difference stats values (contains 'hash', 'assets', 'warnings' and 'errors') 1`] = `
64+
Array [
65+
"errors",
66+
"warnings",
67+
"hash",
68+
"assetsByChunkName",
69+
"assets",
70+
"filteredAssets",
71+
]
72+
`;
73+
74+
exports[`Server stats should works with difference stats values (contains 'hash', 'assets', 'warnings' and 'errors') 2`] = `
75+
Array [
76+
"errors",
77+
"warnings",
78+
"hash",
79+
"assetsByChunkName",
80+
"assets",
81+
"filteredAssets",
82+
]
83+
`;
84+
85+
exports[`Server stats should works with difference stats values (contains 'hash', 'assets', 'warnings' and 'errors') 3`] = `
86+
Array [
87+
"errors",
88+
"warnings",
89+
"hash",
90+
"assetsByChunkName",
91+
"assets",
92+
"filteredAssets",
93+
]
94+
`;
95+
96+
exports[`Server stats should works with difference stats values (contains 'hash', 'assets', 'warnings' and 'errors') 4`] = `
97+
Array [
98+
"errors",
99+
"warnings",
100+
"hash",
101+
"assetsByChunkName",
102+
"assets",
103+
"filteredAssets",
104+
]
105+
`;
106+
107+
exports[`Server stats should works with difference stats values (contains 'hash', 'assets', 'warnings' and 'errors') 5`] = `
108+
Array [
109+
"errors",
110+
"warnings",
111+
"hash",
112+
"assetsByChunkName",
113+
"assets",
114+
"filteredAssets",
115+
]
116+
`;

‎test/client/utils/createSocketUrl.test.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ describe('createSocketUrl', () => {
2424
const {
2525
default: createSocketUrl,
2626
// eslint-disable-next-line global-require
27-
} = require('../../../../client-src/default/utils/createSocketUrl');
27+
} = require('../../../client-src/default/utils/createSocketUrl');
2828

2929
test(`should return the url when __resourceQuery is ${url}`, () => {
3030
expect(createSocketUrl(url)).toMatchSnapshot();

‎test/client/utils/getCurrentScriptSource.test.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
const {
44
default: getCurrentScriptSource,
5-
} = require('../../../../client-src/default/utils/getCurrentScriptSource');
5+
} = require('../../../client-src/default/utils/getCurrentScriptSource');
66

77
describe('getCurrentScriptSource', () => {
88
afterEach(() => {

‎test/client/utils/reloadApp.test.js

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,8 +25,7 @@ describe('reloadApp', () => {
2525
});
2626

2727
// eslint-disable-next-line global-require
28-
reloadApp = require('../../../../client-src/default/utils/reloadApp')
29-
.default;
28+
reloadApp = require('../../../client-src/default/utils/reloadApp').default;
3029
});
3130

3231
afterEach(() => {

‎test/client/utils/sendMessage.test.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44

55
const {
66
default: sendMessage,
7-
} = require('../../../../client-src/default/utils/sendMessage');
7+
} = require('../../../client-src/default/utils/sendMessage');
88

99
describe('sendMessage', () => {
1010
afterEach(() => {

‎test/server/open-option.test.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,9 @@
11
'use strict';
22

3-
jest.mock('opn');
3+
jest.mock('open');
44

55
const webpack = require('webpack');
6-
const open = require('opn');
6+
const open = require('open');
77
const Server = require('../../lib/Server');
88
const config = require('../fixtures/simple-config/webpack.config');
99
const port = require('../ports-map')['open-option'];

‎test/server/utils/runOpen.test.js

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -1,23 +1,23 @@
11
'use strict';
22

3-
const opn = require('opn');
3+
const open = require('open');
44
const runOpen = require('../../../lib/utils/runOpen');
55

6-
jest.mock('opn');
6+
jest.mock('open');
77

88
describe('runOpen util', () => {
99
afterEach(() => {
10-
opn.mockClear();
10+
open.mockClear();
1111
});
1212

1313
describe('should open browser', () => {
1414
beforeEach(() => {
15-
opn.mockImplementation(() => Promise.resolve());
15+
open.mockImplementation(() => Promise.resolve());
1616
});
1717

1818
it('on specify URL', () => {
1919
return runOpen('https://example.com', {}, console).then(() => {
20-
expect(opn.mock.calls[0]).toMatchInlineSnapshot(`
20+
expect(open.mock.calls[0]).toMatchInlineSnapshot(`
2121
Array [
2222
"https://example.com",
2323
Object {
@@ -34,7 +34,7 @@ describe('runOpen util', () => {
3434
{ openPage: '/index.html' },
3535
console
3636
).then(() => {
37-
expect(opn.mock.calls[0]).toMatchInlineSnapshot(`
37+
expect(open.mock.calls[0]).toMatchInlineSnapshot(`
3838
Array [
3939
"https://example.com/index.html",
4040
Object {
@@ -51,7 +51,7 @@ describe('runOpen util', () => {
5151
{ open: 'Google Chrome' },
5252
console
5353
).then(() => {
54-
expect(opn.mock.calls[0]).toMatchInlineSnapshot(`
54+
expect(open.mock.calls[0]).toMatchInlineSnapshot(`
5555
Array [
5656
"https://example.com",
5757
Object {
@@ -69,7 +69,7 @@ describe('runOpen util', () => {
6969
{ open: 'Google Chrome', openPage: '/index.html' },
7070
console
7171
).then(() => {
72-
expect(opn.mock.calls[0]).toMatchInlineSnapshot(`
72+
expect(open.mock.calls[0]).toMatchInlineSnapshot(`
7373
Array [
7474
"https://example.com/index.html",
7575
Object {
@@ -86,7 +86,7 @@ describe('runOpen util', () => {
8686
const logMock = { warn: jest.fn() };
8787

8888
beforeEach(() => {
89-
opn.mockImplementation(() => Promise.reject());
89+
open.mockImplementation(() => Promise.reject());
9090
});
9191

9292
afterEach(() => {
@@ -98,7 +98,7 @@ describe('runOpen util', () => {
9898
expect(logMock.warn.mock.calls[0][0]).toMatchInlineSnapshot(
9999
`"Unable to open browser. If you are running in a headless environment, please do not use the --open flag"`
100100
);
101-
expect(opn.mock.calls[0]).toMatchInlineSnapshot(`
101+
expect(open.mock.calls[0]).toMatchInlineSnapshot(`
102102
Array [
103103
"https://example.com",
104104
Object {
@@ -118,7 +118,7 @@ describe('runOpen util', () => {
118118
expect(logMock.warn.mock.calls[0][0]).toMatchInlineSnapshot(
119119
`"Unable to open browser. If you are running in a headless environment, please do not use the --open flag"`
120120
);
121-
expect(opn.mock.calls[0]).toMatchInlineSnapshot(`
121+
expect(open.mock.calls[0]).toMatchInlineSnapshot(`
122122
Array [
123123
"https://example.com/index.html",
124124
Object {
@@ -138,7 +138,7 @@ describe('runOpen util', () => {
138138
expect(logMock.warn.mock.calls[0][0]).toMatchInlineSnapshot(
139139
`"Unable to open browser: Google Chrome. If you are running in a headless environment, please do not use the --open flag"`
140140
);
141-
expect(opn.mock.calls[0]).toMatchInlineSnapshot(`
141+
expect(open.mock.calls[0]).toMatchInlineSnapshot(`
142142
Array [
143143
"https://example.com",
144144
Object {
@@ -159,7 +159,7 @@ describe('runOpen util', () => {
159159
expect(logMock.warn.mock.calls[0][0]).toMatchInlineSnapshot(
160160
`"Unable to open browser: Google Chrome. If you are running in a headless environment, please do not use the --open flag"`
161161
);
162-
expect(opn.mock.calls[0]).toMatchInlineSnapshot(`
162+
expect(open.mock.calls[0]).toMatchInlineSnapshot(`
163163
Array [
164164
"https://example.com/index.html",
165165
Object {

0 commit comments

Comments
 (0)