Skip to content

Commit b27bf48

Browse files
committed
Merge pull request #104 from codeclimate/skipped-lines
Ignore skipped lines
2 parents bf1584b + d72cb2a commit b27bf48

File tree

2 files changed

+8
-2
lines changed

2 files changed

+8
-2
lines changed

lib/code_climate/test_reporter/formatter.rb

+5
Original file line numberDiff line numberDiff line change
@@ -56,6 +56,11 @@ def to_payload(result)
5656
totals[:covered] += file.covered_lines.count
5757
totals[:missed] += file.missed_lines.count
5858

59+
# Set coverage for all skipped lines to nil
60+
file.skipped_lines.each do |skipped_line|
61+
file.coverage[skipped_line.line_number - 1] = nil
62+
end
63+
5964
{
6065
name: short_filename(file.filename),
6166
blob_id: CalculateBlob.new(file.filename).blob_id,

spec/lib/formatter_spec.rb

+3-2
Original file line numberDiff line numberDiff line change
@@ -18,8 +18,9 @@ module CodeClimate::TestReporter
1818
:lines => [double, double, double],
1919
:covered_lines => [double, double],
2020
:missed_lines => [double],
21+
:skipped_lines => [double(:line_number => 5), double(:line_number => 6)],
2122
:filename => project_file,
22-
:coverage => [0,3,2,nil],
23+
:coverage => [0,3,2,nil,1,0],
2324
:covered_percent => 33.2,
2425
:covered_strength => 2
2526
)
@@ -45,7 +46,7 @@ module CodeClimate::TestReporter
4546
{
4647
"name" => project_file,
4748
"blob_id" => "e69de29bb2d1d6434b8b29ae775ad8c2e48c5391",
48-
"coverage" => "[0,3,2,null]",
49+
"coverage" => "[0,3,2,null,null,null]",
4950
"covered_percent" => 33.2,
5051
"covered_strength" => 2.0,
5152
"line_counts" => {"total"=>3, "covered"=>2, "missed"=>1}

0 commit comments

Comments
 (0)