coveralls 0.8.3 → 0.8.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: a846818377296cbdc1e2e3a02e95b78bda07a89e
-  data.tar.gz: 85ef772d52ce6d32237249a2dbeffae914973409
+  metadata.gz: 766530ea38e74bb4d1a38671945a41f572aa063b
+  data.tar.gz: 51352ecb98eb89ba741000a4efd3accb9c946f23
 SHA512:
-  metadata.gz: e16b49628b4e0bcff035e460129dc1825b065faa01eda452707470034e36d07f805a9a6b56ba0ec7875a6198f79e8732041681349907ac2a77211c23dc2f2981
-  data.tar.gz: 0cb60c9f52e2fd31b1ed4bcf8e22dbc3b09cbf1a4e6850c93ba558c88e560a479a39e1e7348a7d0bc0bfed984c543dbad5278f1ece66984965c50bc55c666ea4
+  metadata.gz: b41e904e1aca79f3179b8aaf45fbcc7ac501e11a414819f6034031cf6aef57c8738958b9f5d30b9e1980d1f06c434864792651ca8806c6d0c813ff392af4fcb6
+  data.tar.gz: 0012c11f145c1309224430ca2263d78b6b2c58f7ad2b8415356b1c9f9fb1cb66ec8624411e295b49f0209cb00161b2c6beb154fe82df6718970ab7c11af8fb05

data/Gemfile

@@ -24,3 +24,7 @@ platform :rbx do
   gem 'json'
   gem 'rubinius-developer_tools'
 end
+
+group :test do
+  gem 'pry'
+end

data/lib/coveralls/simplecov.rb

@@ -26,15 +26,7 @@ module Coveralls
         true
       end
 
-      def format(result)
-
-        unless Coveralls.should_run?
-          if Coveralls.noisy?
-            display_result result
-          end
-          return
-        end
-
+      def get_source_files(result)
         # Gather the source files.
         source_files = []
         result.files.each do |file|
@@ -47,29 +39,33 @@ module Coveralls
           properties[:name] = short_filename(file.filename)
 
           # Get the coverage
-          properties[:coverage] = file.coverage
-
-          # Get the group
-          # puts "result groups: #{result.groups}"
-          # result.groups.each do |group_name, grouped_files|
-          #   puts "Checking #{grouped_files.map(&:filename)} for #{file.filename}"
-          #   if grouped_files.map(&:filename).include?(file.filename)
-          #     properties[:group] = group_name
-          #     break
-          #   end
-          # end
+          properties[:coverage] = file.coverage.dup
+
+          # Skip nocov lines
+          file.lines.each_with_index do |line, i|
+            properties[:coverage][i] = nil if line.skipped?
+          end
 
           source_files << properties
         end
+        source_files
+      end
+
+      def format(result)
 
-        # Log which files are being submitted.
-        # puts "Submitting #{source_files.length} file#{source_files.length == 1 ? '' : 's'}:"
-        # source_files.each do |f|
-        #   puts "  => #{f[:name]}"
-        # end
+        unless Coveralls.should_run?
+          if Coveralls.noisy?
+            display_result result
+          end
+          return
+        end
 
         # Post to Coveralls.
-        API.post_json "jobs", {:source_files => source_files, :test_framework => result.command_name.downcase, :run_at => result.created_at}
+        API.post_json "jobs", 
+          :source_files => get_source_files(result),
+          :test_framework => result.command_name.downcase,
+          :run_at => result.created_at
+
         Coveralls::Output.puts output_message result
 
         true

data/lib/coveralls/version.rb

@@ -1,3 +1,3 @@
 module Coveralls
- VERSION = "0.8.3"
+ VERSION = "0.8.5"
 end

data/spec/coveralls/fixtures/app/controllers/sample.rb

@@ -3,8 +3,10 @@ class Foo
   def initialize
     @foo = 'baz'
   end
-  
+
+  # :nocov:
   def bar
     @foo
   end
-end
+  # :nocov:
+end

data/spec/coveralls/fixtures/sample.rb

@@ -4,7 +4,9 @@ class Foo
     @foo = 'baz'
   end
 
+  # :nocov:
   def bar
     @foo
   end
+  # :nocov:
 end

data/spec/coveralls/simplecov_spec.rb

@@ -19,7 +19,7 @@ describe Coveralls::SimpleCov::Formatter do
       source_fixture( 'app/models/house.rb' )        => [nil, nil, nil, nil, nil, nil, nil, nil, nil, nil],
       source_fixture( 'app/models/airplane.rb' )     => [0, 0, 0, 0, 0],
       source_fixture( 'app/models/dog.rb' )          => [1, 1, 1, 1, 1],
-      source_fixture( 'app/controllers/sample.rb' )  => [nil, 1, 1, 1, nil, nil, 0, 0, nil, nil]
+      source_fixture( 'app/controllers/sample.rb' )  => [nil, 1, 1, 1, nil, 0, 1, 1, nil, nil]
     })
   }
 
@@ -69,5 +69,13 @@ describe Coveralls::SimpleCov::Formatter do
       end
     end
 
+    context "#get_source_files" do
+      let(:source_files) { Coveralls::SimpleCov::Formatter.new.get_source_files(result) }
+      it "nils the skipped lines" do
+        source_file = source_files.first
+        source_file[:coverage].should_not eq result.files.first.coverage
+        source_file[:coverage].should eq [nil, 1, 1, 1, nil, 0, 1, nil, nil, nil]
+      end
+    end
   end
 end

data/spec/spec_helper.rb

@@ -2,6 +2,8 @@ require 'simplecov'
 require 'webmock'
 require 'vcr'
 
+require 'pry' if RUBY_VERSION > "1.8.7"
+
 class InceptionFormatter
   def format(result)
     Coveralls::SimpleCov::Formatter.new.format(result)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: coveralls
 version: !ruby/object:Gem::Version
-  version: 0.8.3
+  version: 0.8.5
 platform: ruby
 authors:
 - Nick Merwin
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-10-02 00:00:00.000000000 Z
+date: 2015-11-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: json