codeclimate-test-reporter 0.1.1 → 0.2.0

data/README.md

@@ -30,6 +30,23 @@ Code Climate account if you are in the test coverage private beta.
 
 Please contact hello@codeclimate.com if you need any assistance setting this up.
 
+## Configuration
+
+Certain behaviors of the test reporter can be configured. See the `Configuration`
+class for more details. For example, you can change the logging level to not
+print info messages:
+
+*Note that the configuration block must come before TestReporter.start.*
+
+```ruby
+CodeClimate::TestReporter.configure do |config|
+  config.logger.level = Logger::WARN
+end
+
+CodeClimate::TestReporter.start
+```
+
+
 ## Help! Your gem is raising a ...
 
 ### VCR::Errors::UnhandledHTTPRequestError

data/lib/code_climate/test_reporter.rb

@@ -6,14 +6,45 @@ module CodeClimate
         require "simplecov"
         ::SimpleCov.add_filter 'vendor'
         ::SimpleCov.formatter = Formatter
-        ::SimpleCov.start("test_frameworks")
-      else
-        puts("Not reporting to Code Climate because ENV['CODECLIMATE_REPO_TOKEN'] is not set.")
+        ::SimpleCov.start(configuration.profile)
       end
     end
 
     def self.run?
-      !!ENV["CODECLIMATE_REPO_TOKEN"]
+      environment_variable_set? && run_on_current_branch?
     end
+
+    def self.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
+    end
+
+    def self.run_on_current_branch?
+      return true if configured_branch.nil?
+
+      run_on_current_branch = !!(current_branch =~ /#{configured_branch}/i)
+      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
+    end
+
+    def self.configured_branch
+      configuration.branch
+    end
+
+    def self.current_branch
+      Git.branch_from_git_or_ci
+    end
+
+    def self.logger
+      CodeClimate::TestReporter.configuration.logger
+    end
+
   end
 end

data/lib/code_climate/test_reporter/ci.rb

@@ -0,0 +1,55 @@
+module CodeClimate
+  module TestReporter
+    class Ci
+
+      def self.service_data
+        if ENV['TRAVIS']
+          {
+            name:             "travis-ci",
+            branch:           ENV['TRAVIS_BRANCH'],
+            build_identifier: ENV['TRAVIS_JOB_ID'],
+            pull_request:     ENV['TRAVIS_PULL_REQUEST']
+          }
+        elsif ENV['CIRCLECI']
+          {
+            name:             "circlci",
+            build_identifier: ENV['CIRCLE_BUILD_NUM'],
+            branch:           ENV['CIRCLE_BRANCH'],
+            commit_sha:       ENV['CIRCLE_SHA1']
+          }
+        elsif ENV['SEMAPHORE']
+          {
+            name:             "semaphore",
+            branch:           ENV['BRANCH_NAME'],
+            build_identifier: ENV['SEMAPHORE_BUILD_NUMBER']
+          }
+        elsif ENV['JENKINS_URL']
+          {
+            name:             "jenkins",
+            build_identifier: ENV['BUILD_NUMBER'],
+            build_url:        ENV['BUILD_URL'],
+            branch:           ENV['GIT_BRANCH'],
+            commit_sha:       ENV['GIT_COMMIT']
+          }
+        elsif ENV['TDDIUM']
+          {
+            name:             "tddium",
+            build_identifier: ENV['TDDIUM_SESSION_ID'],
+            worker_id:        ENV['TDDIUM_TID']
+          }
+        elsif ENV['CI_NAME'] =~ /codeship/i
+          {
+            name:             "codeship",
+            build_identifier: ENV['CI_BUILD_NUMBER'],
+            build_url:        ENV['CI_BUILD_URL'],
+            branch:           ENV['CI_BRANCH'],
+            commit_sha:       ENV['CI_COMMIT_ID'],
+          }
+        else
+          {}
+        end
+      end
+
+    end
+  end
+end

data/lib/code_climate/test_reporter/configuration.rb

@@ -0,0 +1,43 @@
+require 'logger'
+
+module CodeClimate
+  module TestReporter
+    @@configuration = nil
+
+    def self.configure
+      @@configuration = Configuration.new
+
+      if block_given?
+        yield configuration
+      end
+
+      configuration
+    end
+
+    def self.configuration
+      @@configuration || configure
+    end
+
+    class Configuration
+      attr_accessor :branch, :logger, :profile
+
+      def logger
+        @logger ||= default_logger
+      end
+
+      def profile
+        @profile ||= "test_frameworks"
+      end
+
+      private
+
+      def default_logger
+        log = Logger.new($stderr)
+        log.level = Logger::INFO
+
+        log
+      end
+    end
+
+  end
+end

data/lib/code_climate/test_reporter/formatter.rb

@@ -19,8 +19,7 @@ module CodeClimate
           File.open(file_path, "w") { |file| file.write(payload.to_json) }
         else
           client = Client.new
-          computed_branch = compute_branch(payload)
-          print "Sending report to #{client.host} for branch #{computed_branch}... "
+          print "Sending report to #{client.host} for branch #{Git.branch_from_git_or_ci}... "
           client.post_results(payload)
         end
 
@@ -64,11 +63,7 @@ module CodeClimate
           covered_strength: round(result.covered_strength, 2),
           line_counts:      totals,
           partial:          partial?,
-          git: {
-            head:         `git log -1 --pretty=format:'%H'`,
-            committed_at: committed_at,
-            branch:       git_branch,
-          },
+          git: Git.info,
           environment: {
             test_framework: result.command_name.downcase,
             pwd:            Dir.pwd,
@@ -80,46 +75,6 @@ module CodeClimate
         }
       end
 
-      def ci_service_data
-        if ENV['TRAVIS']
-          {
-            name:             "travis-ci",
-            branch:           ENV['TRAVIS_BRANCH'],
-            build_identifier: ENV['TRAVIS_JOB_ID'],
-            pull_request:     ENV['TRAVIS_PULL_REQUEST']
-          }
-        elsif ENV['CIRCLECI']
-          {
-            name:             "circlci",
-            build_identifier: ENV['CIRCLE_BUILD_NUM'],
-            branch:           ENV['CIRCLE_BRANCH'],
-            commit_sha:       ENV['CIRCLE_SHA1']
-          }
-        elsif ENV['SEMAPHORE']
-          {
-            name:             "semaphore",
-            branch:           ENV['BRANCH_NAME'],
-            build_identifier: ENV['SEMAPHORE_BUILD_NUMBER']
-          }
-        elsif ENV['JENKINS_URL']
-          {
-            name:             "jenkins",
-            build_identifier: ENV['BUILD_NUMBER'],
-            build_url:        ENV['BUILD_URL'],
-            branch:           ENV['GIT_BRANCH'],
-            commit_sha:       ENV['GIT_COMMIT']
-          }
-        elsif ENV['TDDIUM']
-          {
-            name:             "tddium",
-            build_identifier: ENV['TDDIUM_SESSION_ID'],
-            worker_id:        ENV['TDDIUM_TID']
-          }
-        else
-          {}
-        end
-      end
-
       def calculate_blob_id(path)
         content = File.open(path, "rb") {|f| f.read }
         header = "blob #{content.length}\0"
@@ -132,39 +87,21 @@ module CodeClimate
         filename.gsub(::SimpleCov.root, '.').gsub(/^\.\//, '')
       end
 
-      def committed_at
-        committed_at = `git log -1 --pretty=format:'%ct'`
-        committed_at.to_i.zero? ? nil : committed_at.to_i
-      end
-
-      def git_branch
-        branch = `git branch`.split("\n").delete_if { |i| i[0] != "*" }
-        branch = [branch].flatten.first
-        branch ? branch.gsub("* ","") : nil
-      end
-
       def tddium?
         ci_service_data && ci_service_data[:name] == "tddium"
       end
 
-      def compute_branch(payload)
-        git_branch = payload[:git][:branch]
-        ci_branch = payload[:ci_service][:branch]
-
-        if ci_branch.to_s.strip.size > 0
-          ci_branch.sub(/^origin\//, "")
-        elsif git_branch.to_s.strip.size > 0 && !git_branch.to_s.strip.start_with?("(")
-          git_branch.sub(/^origin\//, "")
-        else
-          "master"
-        end
-      end
-
       # Convert to Float before rounding.
       # Fixes [#7] possible segmentation fault when calling #round on a Rational
       def round(numeric, precision)
         Float(numeric).round(precision)
       end
+
+      private
+
+      def ci_service_data
+        @ci_service_data ||= Ci.service_data
+      end
     end
   end
 end

data/lib/code_climate/test_reporter/git.rb

@@ -0,0 +1,42 @@
+module CodeClimate
+  module TestReporter
+    class Git
+
+      class << self
+        def info
+          {
+            head:         `git log -1 --pretty=format:'%H'`,
+            committed_at: committed_at,
+            branch:       branch_from_git,
+          }
+        end
+
+        def branch_from_git_or_ci
+          git_branch = branch_from_git
+          ci_branch = Ci.service_data[:branch]
+
+          if ci_branch.to_s.strip.size > 0
+            ci_branch.sub(/^origin\//, "")
+          elsif git_branch.to_s.strip.size > 0 && !git_branch.to_s.strip.start_with?("(")
+            git_branch.sub(/^origin\//, "")
+          else
+            "master"
+          end
+        end
+
+        private
+
+        def committed_at
+          committed_at = `git log -1 --pretty=format:'%ct'`
+          committed_at.to_i.zero? ? nil : committed_at.to_i
+        end
+
+        def branch_from_git
+          branch = `git branch`.split("\n").delete_if { |i| i[0] != "*" }
+          branch = [branch].flatten.first
+          branch ? branch.gsub("* ","") : nil
+        end
+      end
+    end
+  end
+end

data/lib/code_climate/test_reporter/version.rb

@@ -1,5 +1,5 @@
 module CodeClimate
   module TestReporter
-    VERSION = "0.1.1"
+    VERSION = "0.2.0"
   end
 end

data/lib/codeclimate-test-reporter.rb

@@ -2,4 +2,7 @@ require "code_climate/test_reporter"
 require "code_climate/test_reporter/version"
 require "code_climate/test_reporter/client"
 require "code_climate/test_reporter/formatter"
+require "code_climate/test_reporter/configuration"
+require "code_climate/test_reporter/git"
+require "code_climate/test_reporter/ci"
 

data/spec/lib/ci_spec.rb

@@ -0,0 +1,31 @@
+require 'spec_helper'
+
+module CodeClimate::TestReporter
+  describe Ci do
+
+    describe '.service_data' do
+      before :each do
+        ENV['SEMAPHORE'] = 'yes?'
+        ENV['BRANCH_NAME'] = 'master'
+        ENV['SEMAPHORE_BUILD_NUMBER'] = '1234'
+      end
+
+      after :each do
+        ENV.delete('SEMAPHORE')
+        ENV.delete('BRANCH_NAME')
+        ENV.delete('SEMAPHORE_BUILD_NUMBER')
+      end
+
+      it 'returns a hash of CI environment info' do
+        expected_semaphore_hash = {
+          name: 'semaphore',
+          branch: 'master',
+          build_identifier: '1234'
+        }
+
+        expect(Ci.service_data).to include expected_semaphore_hash
+      end
+    end
+
+  end
+end

data/spec/lib/configuration_spec.rb

@@ -0,0 +1,52 @@
+require 'spec_helper'
+require 'logger'
+
+module CodeClimate::TestReporter
+  describe Configuration do
+    describe 'none given' do
+      before do
+        CodeClimate::TestReporter.configure
+      end
+
+      it 'provides defaults' do
+        expect(CodeClimate::TestReporter.configuration.branch).to be_nil
+        expect(CodeClimate::TestReporter.configuration.logger).to be_instance_of Logger
+        expect(CodeClimate::TestReporter.configuration.logger.level).to eq Logger::INFO
+        expect(CodeClimate::TestReporter.configuration.profile).to eq('test_frameworks')
+      end
+    end
+
+    describe 'with config block' do
+      after do
+        CodeClimate::TestReporter.configure
+      end
+
+      it 'stores logger' do
+        logger = Logger.new($stderr)
+
+        CodeClimate::TestReporter.configure do |config|
+          logger.level = Logger::DEBUG
+          config.logger = logger
+        end
+
+        expect(CodeClimate::TestReporter.configuration.logger).to eq logger
+      end
+
+      it 'stores branch' do
+        CodeClimate::TestReporter.configure do |config|
+          config.branch = :master
+        end
+
+        expect(CodeClimate::TestReporter.configuration.branch).to eq :master
+      end
+
+      it 'stores profile' do
+        CodeClimate::TestReporter.configure do |config|
+          config.profile = 'custom'
+        end
+
+        expect(CodeClimate::TestReporter.configuration.profile).to eq('custom')
+      end
+    end
+  end
+end

data/spec/lib/formatter_spec.rb

@@ -61,7 +61,7 @@ module CodeClimate::TestReporter
             "pwd" => Dir.pwd,
             "rails_root" => nil,
             "simplecov_root" => Dir.pwd,
-            "gem_version" => "0.0.11"
+            "gem_version" => VERSION
           },
         "ci_service" => {}
       }

data/spec/lib/git_spec.rb

@@ -0,0 +1,39 @@
+require 'spec_helper'
+
+module CodeClimate::TestReporter
+  describe Git do
+    describe '.info' do
+      it 'returns a hash with git information.' do
+        expected_git_hash = {
+          head: `git log -1 --pretty=format:'%H'`,
+          committed_at: `git log -1 --pretty=format:'%ct'`.to_i,
+          branch: Git.send(:branch_from_git)
+        }
+
+        expect(Git.info).to include expected_git_hash
+      end
+    end
+
+    describe 'branch_from_git_or_ci' do
+      it 'returns the branch from ci' do
+        allow(Ci).to receive(:service_data).and_return({branch: 'ci-branch'})
+
+        expect(Git.branch_from_git_or_ci).to eq 'ci-branch'
+      end
+
+      it 'returns the branch from git if there is no ci branch' do
+        allow(Ci).to receive(:service_data).and_return({})
+
+        expect(Git.branch_from_git_or_ci).to eq Git.send(:branch_from_git)
+      end
+
+      it 'returns master otherwise' do
+        allow(Ci).to receive(:service_data).and_return({})
+        allow(Git).to receive(:branch_from_git).and_return(nil)
+
+        expect(Git.branch_from_git_or_ci).to eq 'master'
+      end
+    end
+
+  end
+end

data/spec/lib/test_reporter_spec.rb

@@ -0,0 +1,34 @@
+require 'spec_helper'
+
+describe CodeClimate::TestReporter do
+
+  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
+    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)
+
+      expect(CodeClimate::TestReporter.run_on_current_branch?).to be_true
+    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)
+
+      expect(CodeClimate::TestReporter.run_on_current_branch?).to be_false
+    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?
+    end
+  end
+
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: codeclimate-test-reporter
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.2.0
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-09-22 00:00:00.000000000 Z
+date: 2013-11-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: simplecov
@@ -148,12 +148,19 @@ files:
 - codeclimate-test-reporter.gemspec
 - config/cacert.pem
 - lib/code_climate/test_reporter.rb
+- lib/code_climate/test_reporter/ci.rb
 - lib/code_climate/test_reporter/client.rb
+- lib/code_climate/test_reporter/configuration.rb
 - lib/code_climate/test_reporter/exception_message.rb
 - lib/code_climate/test_reporter/formatter.rb
+- lib/code_climate/test_reporter/git.rb
 - lib/code_climate/test_reporter/version.rb
 - lib/codeclimate-test-reporter.rb
+- spec/lib/ci_spec.rb
+- spec/lib/configuration_spec.rb
 - spec/lib/formatter_spec.rb
+- spec/lib/git_spec.rb
+- spec/lib/test_reporter_spec.rb
 - spec/spec_helper.rb
 homepage: ''
 licenses:
@@ -181,6 +188,9 @@ signing_key:
 specification_version: 3
 summary: Uploads Ruby test coverage data to Code Climate.
 test_files:
+- spec/lib/ci_spec.rb
+- spec/lib/configuration_spec.rb
 - spec/lib/formatter_spec.rb
+- spec/lib/git_spec.rb
+- spec/lib/test_reporter_spec.rb
 - spec/spec_helper.rb
-has_rdoc: