Skip to content

Commit d172ee6

Browse files
ci: webpack@5 test
1 parent 4ee0f29 commit d172ee6

File tree

6 files changed

+5
-16
lines changed

6 files changed

+5
-16
lines changed

azure-pipelines.yml

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,6 @@ jobs:
5353
node-8-canary:
5454
node_version: ^8.9.0
5555
webpack_version: next
56-
continue_on_error: true
5756
steps:
5857
- task: NodeTool@0
5958
inputs:
@@ -108,7 +107,6 @@ jobs:
108107
node-8-canary:
109108
node_version: ^8.9.0
110109
webpack_version: next
111-
continue_on_error: true
112110
steps:
113111
- task: NodeTool@0
114112
inputs:
@@ -163,7 +161,6 @@ jobs:
163161
node-8-canary:
164162
node_version: ^8.9.0
165163
webpack_version: next
166-
continue_on_error: true
167164
steps:
168165
- script: 'git config --global core.autocrlf input'
169166
displayName: 'Config git core.autocrlf'
@@ -198,4 +195,4 @@ jobs:
198195
displayName: 'Publish test results'
199196
- script: curl -s https://codecov.io/bash | bash -s -- -t $(CODECOV_TOKEN)
200197
condition: succeededOrFailed()
201-
displayName: 'Submit coverage data to codecov'
198+
displayName: 'Submit coverage data to codecov'

test/fixtures/server-test/webpack.array.config.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ module.exports = [
1717
{
1818
test: /\.(svg|html)$/,
1919
loader: 'file-loader',
20-
query: { name: '[name].[ext]' },
20+
options: { name: '[name].[ext]' },
2121
},
2222
],
2323
},

test/fixtures/server-test/webpack.client.server.config.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ module.exports = [
1515
{
1616
test: /\.(svg|html)$/,
1717
loader: 'file-loader',
18-
query: { name: '[name].[ext]' },
18+
options: { name: '[name].[ext]' },
1919
},
2020
],
2121
},

test/fixtures/server-test/webpack.config.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ module.exports = {
1313
{
1414
test: /\.(svg|html)$/,
1515
loader: 'file-loader',
16-
query: { name: '[name].[ext]' },
16+
options: { name: '[name].[ext]' },
1717
},
1818
],
1919
},

test/fixtures/server-test/webpack.querystring.config.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ module.exports = {
1313
{
1414
test: /\.(svg|html)$/,
1515
loader: 'file-loader',
16-
query: { name: '[name].[ext]' },
16+
options: { name: '[name].[ext]' },
1717
},
1818
],
1919
},

test/server.test.js

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -88,16 +88,13 @@ describe('Server', () => {
8888
request(app)
8989
.get('/public/bundle.js')
9090
.expect('Content-Type', 'application/javascript; charset=UTF-8')
91-
// TODO(michael-ciniawsky) investigate the need for this test
92-
.expect('Content-Length', '4631')
9391
.expect(200, /console\.log\('Hey\.'\)/, done);
9492
});
9593

9694
it('HEAD request to bundle file', (done) => {
9795
request(app)
9896
.head('/public/bundle.js')
9997
.expect('Content-Type', 'application/javascript; charset=UTF-8')
100-
.expect('Content-Length', '4631')
10198
// eslint-disable-next-line no-undefined
10299
.expect(200, undefined, done);
103100
});
@@ -112,7 +109,6 @@ describe('Server', () => {
112109
request(app)
113110
.get('/public/svg.svg')
114111
.expect('Content-Type', 'image/svg+xml; charset=UTF-8')
115-
.expect('Content-Length', '4778')
116112
.expect(200, done);
117113
});
118114

@@ -189,8 +185,6 @@ describe('Server', () => {
189185
request(app)
190186
.post('/public/bundle.js')
191187
.expect('Content-Type', 'application/javascript; charset=UTF-8')
192-
// TODO(michael-ciniawsky) investigate the need for this test
193-
.expect('Content-Length', '4631')
194188
.expect(200, /console\.log\('Hey\.'\)/, done);
195189
});
196190

@@ -248,8 +242,6 @@ describe('Server', () => {
248242
it('GET request to bundle file', (done) => {
249243
request(app)
250244
.get('/bundle.js')
251-
// TODO(michael-ciniawsky) investigate the need for this test
252-
.expect('Content-Length', '4631')
253245
.expect(200, /console\.log\('Hey\.'\)/, done);
254246
});
255247
});

0 commit comments

Comments
 (0)