Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

BUG: Fix coverage calculation for empty files #2

Merged
merged 1 commit into from Feb 1, 2016
Jump to file or symbol
Failed to load files and symbols.
+27 −5
Split
@@ -54,4 +54,10 @@
function r=get_coverage_ratio(obj)
executable=get_lines_executable(obj);
- r=sum(get_lines_executed(obj) & executable) / sum(executable);
+ numerator = sum(get_lines_executed(obj) & executable);
+ denominator = sum(executable);
+ if denominator==0
+ r=1;
+ else
+ r=numerator/denominator;
+ end
@@ -66,6 +66,10 @@ function write_to_file(fn,s)
denominator=denominator+sum(able);
end
- coverage=numerator/denominator;
+ if denominator==0
+ coverage=1;
+ else
+ coverage=numerator/denominator;
+ end
@@ -96,7 +96,11 @@ function write_index_html(output_fn, mfiles, mfile_node_fns)
executed(~executable)=false;
- coverage=100*sum(executed)/sum(executable);
+ if sum(executable)==0
+ coverage=100;
+ else
+ coverage=100*sum(executed)/sum(executable);
+ end
stat=[numel(executable),...
sum(executable),...
@@ -111,7 +111,11 @@ function write_index_html(output_fn, mfiles, mfile_node_fns)
executed(~executable)=false;
- coverage=100*sum(executed)/sum(executable);
+ if sum(executable)==0
+ coverage=100;
+ else
+ coverage=100*sum(executed)/sum(executable);
+ end
stat=[numel(executable),...
sum(executable),...
@@ -83,6 +83,10 @@ function write_to_file(fn,s)
denominator=denominator+sum(executable);
end
- coverage=numerator/denominator;
+ if denominator==0
+ coverage=1;
+ else
+ coverage=numerator/denominator;
+ end