Skip to content

Commit 9dbb0a5

Browse files
authoredDec 16, 2020
tests(smoke): restore dbw_tester exception assertions (#11836)
1 parent a6738e0 commit 9dbb0a5

File tree

3 files changed

+39
-49
lines changed

3 files changed

+39
-49
lines changed
 

‎lighthouse-cli/test/cli/__snapshots__/index-test.js.snap

+1
Original file line numberDiff line numberDiff line change
@@ -1454,6 +1454,7 @@ Object {
14541454
Object {
14551455
"blankPage": "about:blank",
14561456
"blockedUrlPatterns": Array [
1457+
"*.css",
14571458
"*.jpg",
14581459
"*.jpeg",
14591460
"*.png",

‎lighthouse-cli/test/smokehouse/test-definitions/dobetterweb/dbw-expectations.js

+37-47
Original file line numberDiff line numberDiff line change
@@ -210,53 +210,43 @@ const expectations = [
210210
'errors-in-console': {
211211
score: 0,
212212
details: {
213-
items: {
214-
length: '7 +/- 1',
215-
// COMPAT: In 89.0.4351.0 we observed one additional runtime error and it's origin is currently unknown
216-
// TODO: Investigate and resolve it's presence. https://github.com/GoogleChrome/lighthouse/issues/11803
217-
// {
218-
// source: 'exception',
219-
// description: 'TypeError: the given value is not a Promise',
220-
// url: 'http://localhost:10200/dobetterweb/third_party/aggressive-promise-polyfill.js',
221-
// },
222-
},
223-
// items: [
224-
// {
225-
// source: 'other',
226-
// description: 'Application Cache Error event: Manifest fetch failed (404) http://localhost:10200/dobetterweb/clock.appcache',
227-
// url: 'http://localhost:10200/dobetterweb/dbw_tester.html',
228-
// },
229-
// {
230-
// source: 'exception',
231-
// description: /^Error: A distinctive error\s+at http:\/\/localhost:10200\/dobetterweb\/dbw_tester.html:\d+:\d+$/,
232-
// url: 'http://localhost:10200/dobetterweb/dbw_tester.html',
233-
// },
234-
// {
235-
// source: 'network',
236-
// description: 'Failed to load resource: the server responded with a status of 404 (Not Found)',
237-
// url: 'http://localhost:10200/dobetterweb/unknown404.css?delay=200',
238-
// },
239-
// {
240-
// source: 'network',
241-
// description: 'Failed to load resource: the server responded with a status of 404 (Not Found)',
242-
// url: 'http://localhost:10200/dobetterweb/fcp-delayer.js?delay=5000',
243-
// },
244-
// {
245-
// source: 'network',
246-
// description: 'Failed to load resource: the server responded with a status of 404 (Not Found)',
247-
// url: 'http://localhost:10200/favicon.ico',
248-
// },
249-
// {
250-
// source: 'network',
251-
// description: 'Failed to load resource: the server responded with a status of 404 (Not Found)',
252-
// url: 'http://localhost:10200/dobetterweb/unknown404.css?delay=200',
253-
// },
254-
// {
255-
// source: 'console.error',
256-
// description: 'Error! Error!',
257-
// url: 'http://localhost:10200/dobetterweb/dbw_tester.html',
258-
// },
259-
// ],
213+
items: [
214+
{
215+
source: 'other',
216+
description: 'Application Cache Error event: Manifest fetch failed (404) http://localhost:10200/dobetterweb/clock.appcache',
217+
url: 'http://localhost:10200/dobetterweb/dbw_tester.html',
218+
},
219+
{
220+
source: 'exception',
221+
description: /^Error: A distinctive error\s+at http:\/\/localhost:10200\/dobetterweb\/dbw_tester.html:\d+:\d+$/,
222+
url: 'http://localhost:10200/dobetterweb/dbw_tester.html',
223+
},
224+
{
225+
source: 'console.error',
226+
description: 'Error! Error!',
227+
url: 'http://localhost:10200/dobetterweb/dbw_tester.html',
228+
},
229+
{
230+
source: 'network',
231+
description: 'Failed to load resource: the server responded with a status of 404 (Not Found)',
232+
url: 'http://localhost:10200/dobetterweb/unknown404.css?delay=200',
233+
},
234+
{
235+
source: 'network',
236+
description: 'Failed to load resource: the server responded with a status of 404 (Not Found)',
237+
url: 'http://localhost:10200/dobetterweb/fcp-delayer.js?delay=5000',
238+
},
239+
{
240+
source: 'network',
241+
description: 'Failed to load resource: the server responded with a status of 404 (Not Found)',
242+
url: 'http://localhost:10200/favicon.ico',
243+
},
244+
{
245+
source: 'network',
246+
description: 'Failed to load resource: the server responded with a status of 404 (Not Found)',
247+
url: 'http://localhost:10200/dobetterweb/unknown404.css?delay=200',
248+
},
249+
],
260250
},
261251
},
262252
'is-on-https': {

‎lighthouse-core/config/default-config.js

+1-2
Original file line numberDiff line numberDiff line change
@@ -175,8 +175,7 @@ const defaultConfig = {
175175
passName: 'redirectPass',
176176
loadFailureMode: 'warn',
177177
// Speed up the redirect pass by blocking stylesheets, fonts, and images
178-
// TODO: restore blocked *.css when https://github.com/GoogleChrome/lighthouse/issues/11803 is resolved.
179-
blockedUrlPatterns: ['*.jpg', '*.jpeg', '*.png', '*.gif', '*.svg', '*.ttf', '*.woff', '*.woff2'],
178+
blockedUrlPatterns: ['*.css', '*.jpg', '*.jpeg', '*.png', '*.gif', '*.svg', '*.ttf', '*.woff', '*.woff2'],
180179
gatherers: [
181180
'http-redirect',
182181
],

0 commit comments

Comments
 (0)
Please sign in to comment.