Skip to content

Commit

Permalink
tests(smokehouse): adjust byte efficiency CPU multiplier (#4809)
Browse files Browse the repository at this point in the history
  • Loading branch information
patrickhulce authored and paulirish committed Mar 19, 2018
1 parent 0206170 commit 941b014
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -136,6 +136,8 @@ class UnusedBytes extends Audit {
*/
static createAuditResult(result, graph) {
const simulatorOptions = PredictivePerf.computeRTTAndServerResponseTime(graph);
// TODO: calibrate multipliers, see https://github.com/GoogleChrome/lighthouse/issues/820
Object.assign(simulatorOptions, {cpuTaskMultiplier: 1, layoutTaskMultiplier: 1});
const simulator = new LoadSimulator(graph, simulatorOptions);

const debugString = result.debugString;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ describe('Byte efficiency base audit', () => {

graph = new NetworkNode(networkRecord);
// add a CPU node to force improvement to TTI
graph.addDependent(new CPUNode({tid: 1, ts: 0, dur: 50 * 1000}));
graph.addDependent(new CPUNode({tid: 1, ts: 0, dur: 100 * 1000}));
});

const baseHeadings = [
Expand Down Expand Up @@ -188,7 +188,7 @@ describe('Byte efficiency base audit', () => {
graph
);

assert.equal(result.rawValue, 420);
assert.equal(result.rawValue, 300);
});
});
});

0 comments on commit 941b014

Please sign in to comment.