Skip to content

Commit 9c0c078

Browse files
authored
Expose LHR to modules consuming cli/run.ts (#2654)
* driveby: launcher: silent logging by default
1 parent b0df777 commit 9c0c078

File tree

3 files changed

+16
-4
lines changed

3 files changed

+16
-4
lines changed

chrome-launcher/chrome-launcher.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,7 @@ export class Launcher {
8787
this.rimraf = moduleOverrides.rimraf || rimraf;
8888
this.spawn = moduleOverrides.spawn || spawn;
8989

90-
log.setLevel(defaults(this.opts.logLevel, 'info'));
90+
log.setLevel(defaults(this.opts.logLevel, 'silent'));
9191

9292
// choose the first one (default)
9393
this.startingUrl = defaults(this.opts.startingUrl, 'about:blank');

lighthouse-cli/run.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -138,7 +138,9 @@ export async function runLighthouse(
138138
await performanceXServer.hostExperiment({url, flags, config}, results);
139139
}
140140

141-
return await launchedChrome.kill();
141+
await launchedChrome.kill();
142+
143+
return results;
142144
} catch (err) {
143145
if (typeof launchedChrome !== 'undefined') {
144146
await launchedChrome!.kill();

lighthouse-cli/test/cli/run-test.js

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,11 +25,21 @@ describe('CLI run', function() {
2525
const url = 'chrome://version';
2626
const filename = path.join(process.cwd(), 'run.ts.results.json');
2727
const flags = getFlags(`--output=json --output-path=${filename} ${url}`);
28-
return run.runLighthouse(url, flags, fastConfig).then(_ => {
28+
return run.runLighthouse(url, flags, fastConfig).then(passedResults => {
2929
assert.ok(fs.existsSync(filename));
3030
const results = JSON.parse(fs.readFileSync(filename, 'utf-8'));
3131
assert.equal(results.audits.viewport.rawValue, false);
32+
33+
// passed results match saved results
34+
assert.strictEqual(results.generatedTime, passedResults.generatedTime);
35+
assert.strictEqual(results.url, passedResults.url);
36+
assert.strictEqual(results.audits.viewport.rawValue, passedResults.audits.viewport.rawValue);
37+
assert.strictEqual(
38+
Object.keys(results.audits).length,
39+
Object.keys(passedResults.audits).length);
40+
assert.deepStrictEqual(results.timing, passedResults.timing);
41+
3242
fs.unlinkSync(filename);
3343
});
34-
});
44+
}).timeout(60000);
3545
});

0 commit comments

Comments
 (0)