Skip to content

Commit 83edd88

Browse files
hiroppyhiroppy
committed
chore: fix tests (#1997)
1 parent f222ec5 commit 83edd88

File tree

3 files changed

+131
-15
lines changed

3 files changed

+131
-15
lines changed

‎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/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 {
@@ -122,7 +122,7 @@ describe('runOpen util', () => {
122122
const logMock = { warn: jest.fn() };
123123

124124
beforeEach(() => {
125-
opn.mockImplementation(() => Promise.reject());
125+
open.mockImplementation(() => Promise.reject());
126126
});
127127

128128
afterEach(() => {
@@ -134,7 +134,7 @@ describe('runOpen util', () => {
134134
expect(logMock.warn.mock.calls[0][0]).toMatchInlineSnapshot(
135135
`"Unable to open browser. If you are running in a headless environment, please do not use the --open flag"`
136136
);
137-
expect(opn.mock.calls[0]).toMatchInlineSnapshot(`
137+
expect(open.mock.calls[0]).toMatchInlineSnapshot(`
138138
Array [
139139
"https://example.com",
140140
Object {
@@ -154,7 +154,7 @@ describe('runOpen util', () => {
154154
expect(logMock.warn.mock.calls[0][0]).toMatchInlineSnapshot(
155155
`"Unable to open browser. If you are running in a headless environment, please do not use the --open flag"`
156156
);
157-
expect(opn.mock.calls[0]).toMatchInlineSnapshot(`
157+
expect(open.mock.calls[0]).toMatchInlineSnapshot(`
158158
Array [
159159
"https://example.com/index.html",
160160
Object {
@@ -174,7 +174,7 @@ describe('runOpen util', () => {
174174
expect(logMock.warn.mock.calls[0][0]).toMatchInlineSnapshot(
175175
`"Unable to open browser: Google Chrome. If you are running in a headless environment, please do not use the --open flag"`
176176
);
177-
expect(opn.mock.calls[0]).toMatchInlineSnapshot(`
177+
expect(open.mock.calls[0]).toMatchInlineSnapshot(`
178178
Array [
179179
"https://example.com",
180180
Object {
@@ -195,7 +195,7 @@ describe('runOpen util', () => {
195195
expect(logMock.warn.mock.calls[0][0]).toMatchInlineSnapshot(
196196
`"Unable to open browser: Google Chrome. If you are running in a headless environment, please do not use the --open flag"`
197197
);
198-
expect(opn.mock.calls[0]).toMatchInlineSnapshot(`
198+
expect(open.mock.calls[0]).toMatchInlineSnapshot(`
199199
Array [
200200
"https://example.com/index.html",
201201
Object {

0 commit comments

Comments
 (0)