Merge pull request #2 from wuchengwei/master

fix bug in calculating total coverage
arian · Jun 26, 2012 · 11b297c · 11b297c
2 parents 78d13e8 + 8490a0a
commit 11b297c
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/lib/reporters/HTMLReporter.js b/lib/reporters/HTMLReporter.js
@@ -35,13 +35,13 @@ var HTMLReporter = prime({
 		for (var file in this.object){
 			var fileReporter = new HTMLFileReporter(this.object[file]);
 
+			var fileReport = fileReporter.report();
+			var percentage = fileReporter.pass / fileReporter.total * 100;
+
 			this.error += fileReporter.error;
 			this.pass  += fileReporter.pass;
 			this.total += fileReporter.total;
 
-			var fileReport = fileReporter.report();
-			var percentage = fileReporter.pass / fileReporter.total * 100;
-
 			result += '<h1>' + file + ' (' + Math.round(percentage) + '%)</h1>\n\n';
 			result += '<pre>' + fileReport + '</pre>';
 		}

diff --git a/lib/reporters/Reporter.js b/lib/reporters/Reporter.js
@@ -19,13 +19,13 @@ var Reporter = prime({
 
 			var fileReporter = new FileReporter(this.object[file], '<<<', '>>>');
 
+			var fileReport = fileReporter.report();
+			var percentage = fileReporter.pass / fileReporter.total * 100;
+
 			this.error += fileReporter.error;
 			this.pass  += fileReporter.pass;
 			this.total += fileReporter.total;
 
-			var fileReport = fileReporter.report();
-			var percentage = fileReporter.pass / fileReporter.total * 100;
-
 			result += '+++  ' + result + ' (' + Math.round(percentage) + ') +++ \n\n';
 			result += fileReport;
 			result += '\n\n\n\n';