codeclimate-test-reporter 0.3.0 → 0.4.0

data/Gemfile

@@ -2,3 +2,11 @@ source 'https://rubygems.org'
 
 # Specify your gem's dependencies in codeclimate-test-reporter.gemspec
 gemspec
+
+platform :ruby_19 do
+  gem "pry-debugger", group: :development
+end
+
+platform :ruby_21 do
+  gem "pry-byebug", group: :development
+end

data/README.md

@@ -9,8 +9,8 @@ Code Climate - [https://codeclimate.com](https://codeclimate.com)
 
 ## Installation
 
-This gem only works with Code Climate accounts, so if you don't have one the
-first step is to create an account at: [https://codeclimate.com](https://codeclimate.com). Then:
+This gem requires a user, but not necessarily a paid account, on Code Climate, so if you don't have one the
+first step is to signup at: [https://codeclimate.com](https://codeclimate.com). Then:
 
 1. Add this to your Gemfile:
 

data/codeclimate-test-reporter.gemspec

@@ -24,5 +24,4 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency "rspec"
   spec.add_development_dependency "artifice"
   spec.add_development_dependency "pry"
-  spec.add_development_dependency "pry-debugger"
 end

data/lib/code_climate/test_reporter.rb

@@ -15,23 +15,27 @@ module CodeClimate
     end
 
     def self.environment_variable_set?
-      environment_variable_set = !!ENV["CODECLIMATE_REPO_TOKEN"]
-      unless environment_variable_set
+      return @environment_variable_set if defined?(@environment_variable_set)
+
+      @environment_variable_set = !!ENV["CODECLIMATE_REPO_TOKEN"]
+      unless @environment_variable_set
         logger.info("Not reporting to Code Climate because ENV['CODECLIMATE_REPO_TOKEN'] is not set.")
       end
 
-      environment_variable_set
+      @environment_variable_set
     end
 
     def self.run_on_current_branch?
-      return true if configured_branch.nil?
+      return @run_on_current_branch if defined?(@run_on_current_branch)
+
+      @run_on_current_branch = true if configured_branch.nil?
+      @run_on_current_branch ||= !!(current_branch =~ /#{configured_branch}/i)
 
-      run_on_current_branch = !!(current_branch =~ /#{configured_branch}/i)
-      unless run_on_current_branch
+      unless @run_on_current_branch
         logger.info("Not reporting to Code Climate because #{configured_branch} is set as the reporting branch.")
       end
 
-      run_on_current_branch
+      @run_on_current_branch
     end
 
     def self.configured_branch

data/lib/code_climate/test_reporter/ci.rb

@@ -37,6 +37,14 @@ module CodeClimate
             build_identifier: ENV['TDDIUM_SESSION_ID'],
             worker_id:        ENV['TDDIUM_TID']
           }
+        elsif ENV['WERCKER']
+          {
+            name:             "wercker",
+            build_identifier: ENV['WERCKER_BUILD_ID'],
+            build_url:        ENV['WERCKER_BUILD_URL'],
+            branch:           ENV['WERCKER_GIT_BRANCH'],
+            commit_sha:       ENV['WERCKER_GIT_COMMIT']
+          }
         elsif ENV['CI_NAME'] =~ /codeship/i
           {
             name:             "codeship",

data/lib/code_climate/test_reporter/client.rb

@@ -53,8 +53,13 @@ module CodeClimate
         request = Net::HTTP::Post.new(uri.path)
         request["User-Agent"] = USER_AGENT
         request["Content-Type"] = "application/json"
-        request["Content-Encoding"] = "gzip"
-        request.body = compress(result.to_json)
+
+        if CodeClimate::TestReporter.configuration.gzip_request
+          request["Content-Encoding"] = "gzip"
+          request.body = compress(result.to_json)
+        else
+          request.body = result.to_json
+        end
 
         response = http.request(request)
 

data/lib/code_climate/test_reporter/configuration.rb

@@ -19,7 +19,11 @@ module CodeClimate
     end
 
     class Configuration
-      attr_accessor :branch, :logger, :profile, :path_prefix
+      attr_accessor :branch, :logger, :profile, :path_prefix, :gzip_request
+
+      def initialize
+        @gzip_request = true
+      end
 
       def logger
         @logger ||= default_logger

data/lib/code_climate/test_reporter/formatter.rb

@@ -11,6 +11,8 @@ module CodeClimate
   module TestReporter
     class Formatter
       def format(result)
+        return true unless CodeClimate::TestReporter.run?
+
         print "Coverage = #{round(result.covered_percent, 2)}%. "
 
         payload = to_payload(result)

data/lib/code_climate/test_reporter/version.rb

@@ -1,5 +1,5 @@
 module CodeClimate
   module TestReporter
-    VERSION = "0.3.0"
+    VERSION = "0.4.0"
   end
 end

data/spec/lib/formatter_spec.rb

@@ -86,33 +86,47 @@ module CodeClimate::TestReporter
     end
 
     it "sends an http request with all the coverage information" do
+      allow(CodeClimate::TestReporter).to receive(:run?).and_return(true)
+
       app = FakeCodeClimateEndpoint.new
       Artifice.activate_with(app) do
         formatter.format(simplecov_result)
       end
-      app.path_info.should == "/test_reports"
-      app.content_type.should == "application/json"
-      app.http_content_encoding.should == "gzip"
+
+      expect(app.path_info).to eq("/test_reports")
+      expect(app.content_type).to eq("application/json")
+      expect(app.http_content_encoding).to eq("gzip")
+
       uncompressed = inflate(app.request_body)
-      JSON.parse(uncompressed).should == expected_request
-      app.http_user_agent.should include("v#{CodeClimate::TestReporter::VERSION}")
+
+      expect(JSON.parse(uncompressed)).to eq(expected_request)
+      expect(app.http_user_agent).to include("v#{CodeClimate::TestReporter::VERSION}")
     end
   end
 
   describe '#short_filename' do
+    let(:formatter) { CodeClimate::TestReporter::Formatter.new }
     it 'should return the filename of the file relative to the SimpleCov root' do
-      CodeClimate::TestReporter::Formatter.new.short_filename('file1').should == 'file1'
-      CodeClimate::TestReporter::Formatter.new.short_filename("#{::SimpleCov.root}/file1").should == 'file1'
+      expect(formatter.short_filename('file1')).to eq('file1')
+      expect(formatter.short_filename("#{::SimpleCov.root}/file1")).to eq('file1')
     end
 
-    it 'should include the path prefix if set' do
-      CodeClimate::TestReporter.configure do |config|
-        config.path_prefix = 'custom'
+    context "with path prefix" do
+      before do
+        CodeClimate::TestReporter.configure do |config|
+          config.path_prefix = 'custom'
+        end
       end
-      CodeClimate::TestReporter::Formatter.new.short_filename('file1').should == 'custom/file1'
-      CodeClimate::TestReporter::Formatter.new.short_filename("#{::SimpleCov.root}/file1").should == 'custom/file1'
-      CodeClimate::TestReporter.configure do |config|
-        config.path_prefix = nil
+
+      after do
+        CodeClimate::TestReporter.configure do |config|
+          config.path_prefix = nil
+        end
+      end
+
+      it 'should include the path prefix if set' do
+        expect(formatter.short_filename('file1')).to eq('custom/file1')
+        expect(formatter.short_filename("#{::SimpleCov.root}/file1")).to eq('custom/file1')
       end
     end
   end

data/spec/lib/test_reporter_spec.rb

@@ -1,33 +1,35 @@
 require 'spec_helper'
 
 describe CodeClimate::TestReporter do
+  let(:reporter) { CodeClimate::TestReporter.dup }
 
   describe '.run_on_current_branch?' do
     it 'returns true if there is no branch configured' do
-      allow(CodeClimate::TestReporter).to receive(:configured_branch).and_return(nil)
-      expect(CodeClimate::TestReporter.run_on_current_branch?).to be_true
+      allow(reporter).to receive(:configured_branch).and_return(nil)
+      expect(reporter).to be_run_on_current_branch
     end
 
     it 'returns true if the current branch matches the configured branch' do
-      allow(CodeClimate::TestReporter).to receive(:current_branch).and_return("master\n")
-      allow(CodeClimate::TestReporter).to receive(:configured_branch).and_return(:master)
+      allow(reporter).to receive(:current_branch).and_return("master\n")
+      allow(reporter).to receive(:configured_branch).and_return(:master)
 
-      expect(CodeClimate::TestReporter.run_on_current_branch?).to be_true
+      expect(reporter).to be_run_on_current_branch
     end
 
     it 'returns false if the current branch and configured branch dont match' do
-      allow(CodeClimate::TestReporter).to receive(:current_branch).and_return("some-branch")
-      allow(CodeClimate::TestReporter).to receive(:configured_branch).and_return(:master)
+      allow(reporter).to receive(:current_branch).and_return("some-branch")
+      allow(reporter).to receive(:configured_branch).and_return(:master)
 
-      expect(CodeClimate::TestReporter.run_on_current_branch?).to be_false
+      expect(reporter).to_not be_run_on_current_branch
     end
 
     it 'logs a message if false' do
       expect_any_instance_of(Logger).to receive(:info)
-      allow(CodeClimate::TestReporter).to receive(:current_branch).and_return("another-branch")
-      allow(CodeClimate::TestReporter).to receive(:configured_branch).and_return(:master)
 
-      CodeClimate::TestReporter.run_on_current_branch?
+      allow(reporter).to receive(:current_branch).and_return("another-branch")
+      allow(reporter).to receive(:configured_branch).and_return(:master)
+
+      reporter.run_on_current_branch?
     end
   end
 

metadata

@@ -1,18 +1,20 @@
 --- !ruby/object:Gem::Specification
 name: codeclimate-test-reporter
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.4.0
+  prerelease: 
 platform: ruby
 authors:
 - Bryan Helmkamp
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-01-24 00:00:00.000000000 Z
+date: 2014-08-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: simplecov
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
@@ -23,6 +25,7 @@ dependencies:
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
@@ -33,6 +36,7 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -40,6 +44,7 @@ dependencies:
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -47,6 +52,7 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
@@ -54,6 +60,7 @@ dependencies:
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
@@ -61,6 +68,7 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
@@ -68,6 +76,7 @@ dependencies:
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
@@ -75,6 +84,7 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: artifice
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
@@ -82,6 +92,7 @@ dependencies:
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
@@ -89,6 +100,7 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: pry
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
@@ -96,20 +108,7 @@ dependencies:
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: '0'
-- !ruby/object:Gem::Dependency
-  name: pry-debugger
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
@@ -152,26 +151,27 @@ files:
 homepage: ''
 licenses:
 - MIT
-metadata: {}
 post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
+  none: false
   requirements:
   - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
+  none: false
   requirements:
   - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.2.1
+rubygems_version: 1.8.23.2
 signing_key: 
-specification_version: 4
+specification_version: 3
 summary: Uploads Ruby test coverage data to Code Climate.
 test_files:
 - spec/lib/ci_spec.rb
@@ -181,3 +181,4 @@ test_files:
 - spec/lib/payload_validator_spec.rb
 - spec/lib/test_reporter_spec.rb
 - spec/spec_helper.rb
+has_rdoc: 

checksums.yaml

@@ -1,15 +0,0 @@
----
-!binary "U0hBMQ==":
-  metadata.gz: !binary |-
-    ZGE4Y2ZjZDBmYzMyMzdhNzhkN2M1NGNhZWI5N2JkNWVmY2FjNzAxNA==
-  data.tar.gz: !binary |-
-    YTc5MjYzNTJmMGViZDdkNTQ5MzhkNjQ4MTE3ODYwMzM4OTk4NGIyMA==
-SHA512:
-  metadata.gz: !binary |-
-    MTM2ZTY3YWU4MjlmZTdlM2U2NjgyNDVlYTQ1NWI5Y2EwMmRkYTU0MjM3YTM3
-    N2I4YzJjZWZmNmM3NGNlNTkwMDQzNTFjNmZhY2M2YWUwMzQ3YTA4MTc0ZTVi
-    NGI2NjI5NmM3Yzg5MzEyNjJlNWZiYzE0NGIxZGFkNTczMDk3Mjg=
-  data.tar.gz: !binary |-
-    NDc4NzgxNWY5ZWQzMzA2NDBkYmYyMzY2MGJiNjVlYzhlZjZkNjdlMDYzZGY3
-    MzNlNDI0NWJiZWM0YTA0M2U1ZmIzYTg2NDQ1OTIzODEwYjdlMDJkZTVlMTRm
-    ZjM3YWFiNmVkZTY0ZWNjNTIyODIzNjZmNmEyNTA4YWI5Y2VlZTE=