letitcrash 0.1.1 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: bfef0c17578eda715477e78176c528533db28768
-  data.tar.gz: d5369bdbf841a5b19c9cb815ba140c66fade3a96
+  metadata.gz: 65797c26e15f11f329c5053d132d6170caf78a4a
+  data.tar.gz: e0c656b78fce7ebf4671de2d43c5d50ace958363
 SHA512:
-  metadata.gz: a4d9887ecb30c06aec1f6fcdcdcac2faec4de1cd2f9069837e03ab3a164262d3cf80e8ec0ca36ffcc95f5bdc957b43fa44b51cf30d37545449199ffa28b353a6
-  data.tar.gz: c145d2c202330ad4332425476b022a0f22e125b7432cc317a36a12faae41673e76ccd216b8fd5e5097b034b97dec79f046092d7d26b0706c4471c5424299ff8a
+  metadata.gz: 23c2b10ed26864ffc5726912c1ae2f7afd0e0a5a52f2e744cd5aa2181fc69b062266e96aee3d7e22af1280131266c5b49f0eed060073d6427a3d6b451bfc7904
+  data.tar.gz: 48f537b477fe4786a959d3ebf8ed3f4bb51328254c3003024031281840d44bba6d4a7481ab5244c4f6dbb7f2372421a9f0e5777e55b4e70acd033fe011e7d3e8

data/lib/letitcrash/builders/file_builder.rb

@@ -48,6 +48,7 @@ module LetItCrash
       end
 
       def coverage
+        return 1 if relevant_lines.empty?
         relevant_lines.values.count(&:positive?) / relevant_lines.count.to_f
       end
 

data/lib/letitcrash/reporters/upload.rb

@@ -21,13 +21,27 @@ module LetItCrash
       end
 
       def report(input)
+        blockers_off
         RestClient.post(endpoint, input.to_json, headers)
+        blockers_on
       end
 
       private
 
       attr_reader :endpoint, :token
 
+      # This method reeks of :reek:UtilityFunction.
+      def blockers_off
+        WebMock.disable! if defined?(WebMock)
+        VCR.turn_off!(ignore_cassettes: true) if defined?(VCR)
+      end
+
+      # This method reeks of :reek:UtilityFunction.
+      def blockers_on
+        WebMock.enable! if defined?(WebMock)
+        VCR.turn_on! if defined?(VCR)
+      end
+
       def headers
         {
           content_type: :json,

data/lib/letitcrash/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module LetItCrash
-  VERSION = '0.1.1'
+  VERSION = '0.1.2'
 end # module LetItCrash

data/spec/builders/file_builder_spec.rb

@@ -7,30 +7,33 @@ module LetItCrash
     RSpec.describe FileBuilder do
       describe '.build' do
         let(:filename) { 'spec/fixtures/source.file' }
+        let(:lines) do
+          [
+            instance_double(
+              SimpleCov::SourceFile::Line,
+              line_number: 1,
+              skipped?: false,
+              coverage: 0,
+            ),
+            instance_double(
+              SimpleCov::SourceFile::Line,
+              line_number: 2,
+              skipped?: false,
+              coverage: 0,
+            ),
+            instance_double(
+              SimpleCov::SourceFile::Line,
+              line_number: 3,
+              skipped?: false,
+              coverage: 1,
+            )
+          ]
+        end
         let(:file) do
           instance_double(
             SimpleCov::SourceFile,
             filename: filename,
-            lines: [
-              instance_double(
-                SimpleCov::SourceFile::Line,
-                line_number: 1,
-                skipped?: false,
-                coverage: 0,
-              ),
-              instance_double(
-                SimpleCov::SourceFile::Line,
-                line_number: 2,
-                skipped?: false,
-                coverage: 0,
-              ),
-              instance_double(
-                SimpleCov::SourceFile::Line,
-                line_number: 3,
-                skipped?: false,
-                coverage: 1,
-              )
-            ],
+            lines: lines,
           )
         end
         let(:result) { described_class.build(file: file, rewriter: rewriter) }
@@ -43,10 +46,10 @@ module LetItCrash
           it { expect(result.digest).to start_with '73054' }
 
           context 'with lines' do
-            let(:lines) { result.lines }
+            let(:result_lines) { result.lines }
 
-            it { expect(lines.count).to eq 1 }
-            it { expect(lines[3]).to eq 1 }
+            it { expect(result_lines.count).to eq 1 }
+            it { expect(result_lines[3]).to eq 1 }
           end # context 'with lines'
 
           context 'with path' do
@@ -62,6 +65,13 @@ module LetItCrash
 
           it { expect(result.path.rewritten).to start_with 'bacon/spec' }
         end # context 'with rewriter'
+
+        context 'with no relevant lines' do
+          let(:rewriter) { nil }
+          let(:lines)    { [] }
+
+          it { expect(result.coverage).to eq 1 }
+        end # context 'with no relevant lines'
       end # describe '.build'
     end # RSpec.describe FileBuilder
   end # module Builders

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: letitcrash
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
 platform: ruby
 authors:
 - Marcin Wyszynski
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-06-23 00:00:00.000000000 Z
+date: 2017-07-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: google-protobuf