mirror
/
sitespeed.io
mirror of https://github.com/sitespeedio/sitespeed.io.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #752 from jzoldak/per-rule-budget 

Add ability to budget on a per rule basis
This commit is contained in:
Peter Hedenskog 2015-09-26 20:54:41 +02:00
parent fd47c07080 bf99608eda
commit a28a7a27c3
1 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
lib/tests/testRenderer.js
View File

@ -266,6 +266,8 @@ TestRenderer.prototype._yslow = function(url, pageData) {
for (var i = 0; i < rule.length; i++) {
var score = rules[rule[i]].score;
var scoreLimit = this.budget.rules[rule[i]] ? this.budget.rules[rule[i]] : defaultLimit;
var result = {};
// is this skippable?
if (self.config.skipTest) {
@ -277,7 +279,7 @@ TestRenderer.prototype._yslow = function(url, pageData) {
result.description = 'Skipping ' + rule[i] + ' score ' + score;
result.value = score;
result.type = 'rule';
result.limit = defaultLimit;
result.limit = scoreLimit;
results.push(result);
continue;
}
@ -285,12 +287,12 @@ TestRenderer.prototype._yslow = function(url, pageData) {
result.title = rule[i] + ' :' + ruleDictionary[rule[i]].name;
result.url = url;
result.id = rule[i];
result.isOk = score > defaultLimit;
result.isOk = score > scoreLimit;
result.description = 'The ' + rule[i] + ' has the score ' + score;
result.value = score;
result.components = rules[rule[i]].components;
result.type = 'rule';
result.limit = defaultLimit;
result.limit = scoreLimit;
results.push(result);
}
}