diff --git a/lib/tools/jsExecutionTransformer.js b/lib/tools/jsExecutionTransformer.js index 1015a09..a215341 100644 --- a/lib/tools/jsExecutionTransformer.js +++ b/lib/tools/jsExecutionTransformer.js @@ -38,7 +38,7 @@ var jsExecutionTransformer = function() { javascriptExecutionTree = JSON.parse(data.toolsResults.phantomas.offenders.javascriptExecutionTree[0]); if (javascriptExecutionTree.children) { - javascriptExecutionTree.children.forEach(function(node) { + javascriptExecutionTree.children.forEach(function(node, index) { var contextLength = (node.data.callDetails && node.data.callDetails.context) ? node.data.callDetails.context.length : null; @@ -90,6 +90,10 @@ var jsExecutionTransformer = function() { break; } } + // Fix rare bug when domComplete was never triggered + if (index === javascriptExecutionTree.children.length - 1 && !javascriptExecutionTree.data.domComplete) { + javascriptExecutionTree.data.domComplete = node.data.timestamp + 1000; + } // Transform domPaths into objects changeListOfDomPaths(node); diff --git a/test/core/weightCheckerTest.js b/test/core/weightCheckerTest.js index 208ff12..01b0ff7 100644 --- a/test/core/weightCheckerTest.js +++ b/test/core/weightCheckerTest.js @@ -76,7 +76,7 @@ describe('weightChecker', function() { newEntry.weightCheck.bodySize.should.equal(93636); newEntry.weightCheck.uncompressedSize.should.equal(newEntry.weightCheck.bodySize); newEntry.weightCheck.isCompressed.should.equal(false); - newEntry.weightCheck.headersSize.should.be.above(300).and.below(350); + newEntry.weightCheck.headersSize.should.be.above(200).and.below(400); newEntry.weightCheck.body.should.have.string('1.8.3'); done();