coverband-service-client 0.0.6 → 0.0.11

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2bbda63a9e7eacfee67d03b5e0b08f9a4a2ef31d4eea06fcb3901730b81fad95
-  data.tar.gz: bab021109fc8d6a785f273c35a7e39c9da4cf78f65d3a2b3a52105d2c9c00333
+  metadata.gz: a0d326e9f18a89a72e6a251e92b1a2c4d99a44288c1e695570327675017e5328
+  data.tar.gz: 83efe304d0dac5797b63ad4dea23516df89ab211a0aa5809554c9b3fce5dfa82
 SHA512:
-  metadata.gz: c9b9fa656753664c2e6dd37670274d9bd9231dea30c071c8735c406f234649e9c1758804da33239889ad6e7636c4652ac8e8d6c851d2eb87b74bb62683d40cc8
-  data.tar.gz: 52f461ebbfdbd5b379c7e8a48661c89aada386cca21e85f60aee3b075bb9291b6a9397503063117c33b850c933562339d57dd5910727f6713240fe57ce84e12e
+  metadata.gz: 3588eb3a0fc1c70afa925b61feaf92ee95b7169476e0008abac2ee83413a7ad0a02726876fda490089fb2008d17861f71cebd297dc78744de049fc62a82c2022
+  data.tar.gz: ee29788c3d877f9cd0079615c30da437490cc3a4c6fb74ed7a0454d44b405ef041965c648b4e3d5162f775c13d8661165c578fa57993de090d39be253f34a139

data/Gemfile.lock

@@ -1,17 +1,17 @@
 PATH
   remote: .
   specs:
-    coverband-service-client (0.0.6)
+    coverband-service-client (0.0.11)
       coverband (~> 4.2.4)
 
 GEM
   remote: https://rubygems.org/
   specs:
-    coverband (4.2.4)
+    coverband (4.2.5)
       redis
     minitest (5.14.0)
     rake (13.0.1)
-    redis (4.1.3)
+    redis (4.1.4)
 
 PLATFORMS
   ruby

data/lib/coverband-service-client.rb

@@ -1,18 +1,31 @@
 # frozen_string_literal: true
 
+COVERBAND_ORIGINAL_START = ENV['COVERBAND_DISABLE_AUTO_START']
+ENV['COVERBAND_DISABLE_AUTO_START'] = 'true'
+require 'coverband'
 require 'coverband/service/client/version'
 require 'securerandom'
 
-COVERBAND_ENV = ENV['RACK_ENV'] || ENV['RAILS_ENV'] || (defined?(Rails) ? Rails.env : 'unknown')
-COVERBAND_SERVICE_URL = ENV['COVERBAND_URL'] ||
-  ((COVERBAND_ENV == 'development') ? 'http://127.0.0.1:3456' : 'https://coverband-service.herokuapp.com')
-COVERBAND_TIMEOUT = (COVERBAND_ENV == 'development') ? 5 : 1
-
 module Coverband
+  COVERBAND_ENV = ENV['RACK_ENV'] || ENV['RAILS_ENV'] || (defined?(Rails) ? Rails.env : 'unknown')
+  COVERBAND_SERVICE_URL = ENV['COVERBAND_URL'] || 'https://coverband.io'
+  COVERBAND_TIMEOUT = (COVERBAND_ENV == 'development') ? 5 : 1
+  COVERBAND_ENABLE_DEV_MODE = ENV['COVERBAND_ENABLE_DEV_MODE'] || false
+  COVERBAND_ENABLE_TEST_MODE = ENV['COVERBAND_ENABLE_TEST_MODE'] || false
+  COVERBAND_PROCESS_TYPE = ENV['PROCESS_TYPE'] || 'unknown'
+  COVERBAND_REPORT_PERIOD = (ENV['COVERBAND_REPORT_PERIOD'] || 600).to_i
+
+  def self.service_disabled_dev_test_env?
+    (COVERBAND_ENV == 'test' && !COVERBAND_ENABLE_TEST_MODE) ||
+      (COVERBAND_ENV == 'development' && !COVERBAND_ENABLE_DEV_MODE)
+  end
 
-  if COVERBAND_ENV == 'test' && !ENV['COVERBAND_ENABLE_TEST_MODE']
+  if service_disabled_dev_test_env?
     def self.report_coverage
-      # for now disable coverband reporting in test env by default
+      # for now disable coverband reporting in test & dev env by default
+      if Coverband.configuration.verbose
+        puts "Coverband: disabled for #{COVERBAND_ENV}, set COVERBAND_ENABLE_DEV_MODE or COVERBAND_ENABLE_TEST_MODE to enable" if Coverband.configuration.verbose || COVERBAND_ENABLE_DEV_MODE
+      end
     end
   end
 
@@ -30,10 +43,14 @@ module Coverband
         def initialize(coverband_url, opts = {})
           super()
           @coverband_url = coverband_url
-          @process_type = opts.fetch(:process_type) { 'unknown' }
+          @process_type = opts.fetch(:process_type) { COVERBAND_PROCESS_TYPE }
           @runtime_env = opts.fetch(:runtime_env) { COVERBAND_ENV }
         end
 
+        def logger
+          Coverband.configuration.logger
+        end
+
         def clear!
           # TBD
         end
@@ -47,24 +64,25 @@ module Coverband
           0
         end
 
-        # TODO: no longer get by type just get both reports in a single request
+        def api_key
+          ENV['COVERBAND_API_KEY'] || Coverband.configuration.api_key
+        end
+
         def coverage(local_type = nil, opts = {})
           local_type ||= opts.key?(:override_type) ? opts[:override_type] : type
-          uri = URI("#{coverband_url}/api/coverage/#{ENV['COVERBAND_ID']}?type=#{local_type}")
-          req = Net::HTTP::Get.new(uri, 'Content-Type' => 'application/json', 'Coverband-Token' => ENV['COVERBAND_API_KEY'])
+          env_filter = opts.key?(:env_filter) ? opts[:env_filter] : 'production'
+          uri = URI("#{coverband_url}/api/coverage?type=#{local_type}&env_filter=#{env_filter}",)
+          req = Net::HTTP::Get.new(uri, 'Content-Type' => 'application/json', 'Coverband-Token' => api_key)
           res = Net::HTTP.start(uri.hostname, uri.port, use_ssl: uri.scheme == 'https') do |http|
             http.request(req)
           end
           coverage_data = JSON.parse(res.body)
-          # puts "coverage data: "
-          # puts coverage_data
           coverage_data
         rescue StandardError => e
-          puts "Coverband: Error while retrieving coverage #{e}"
+          logger&.error "Coverband: Error while retrieving coverage #{e}" if Coverband.configuration.verbose || COVERBAND_ENABLE_DEV_MODE
         end
 
         def save_report(report)
-          #puts caller.join(',')
           return if report.empty?
 
           # TODO: do we need dup
@@ -93,12 +111,17 @@ module Coverband
         private
 
         def save_coverage(data)
+          if api_key.nil?
+            puts "Coverband: Error: no Coverband API key was found!"
+          end
+
           uri = URI("#{coverband_url}/api/collector")
           req = Net::HTTP::Post.new(uri,
                                     'Content-Type' => 'application/json',
-                                    'Coverband-Token' => ENV['COVERBAND_API_KEY'])
-          # puts "sending #{data}"
+                                    'Coverband-Token' => api_key)
           req.body = { remote_uuid: SecureRandom.uuid, data: data }.to_json
+
+          logger&.info "Coverband: saving (#{uri}) #{req.body}" if Coverband.configuration.verbose
           res = Net::HTTP.start(
             uri.hostname,
             uri.port,
@@ -110,7 +133,7 @@ module Coverband
             http.request(req)
           end
         rescue StandardError => e
-          puts "Coverband: Error while saving coverage #{e}"
+          logger&.info "Coverband: Error while saving coverage #{e}" if Coverband.configuration.verbose || COVERBAND_ENABLE_DEV_MODE
         end
       end
     end
@@ -144,14 +167,22 @@ module Coverband
       rescue StandardError => e
         # we don't want to raise errors if Coverband can't reach redis.
         # This is a nice to have not a bring the system down
-        logger&.error "Coverband: view_tracker failed to store, error #{e.class.name}"
+        logger&.error "Coverband: view_tracker failed to store, error #{e.class.name}" if Coverband.configuration.verbose || COVERBAND_ENABLE_DEV_MODE
       end
 
       private
 
+      def api_key
+        ENV['COVERBAND_API_KEY'] || Coverband.configuration.api_key
+      end
+
+      def logger
+        Coverband.configuration.logger
+      end
+
       def save_tracked_views(views:, reported_time:)
         uri = URI("#{COVERBAND_SERVICE_URL}/api/collector")
-        req = Net::HTTP::Post.new(uri, 'Content-Type' => 'application/json', 'Coverband-Token' => ENV['COVERBAND_API_KEY'])
+        req = Net::HTTP::Post.new(uri, 'Content-Type' => 'application/json', 'Coverband-Token' => api_key)
         data = {
           collection_type: 'view_tracker_delta',
           collection_data: {
@@ -168,25 +199,49 @@ module Coverband
           http.request(req)
         end
       rescue StandardError => e
-        puts "Coverband: Error while saving coverage #{e}"
+        logger&.error "Coverband: Error while saving coverage #{e}" if Coverband.configuration.verbose || COVERBAND_ENABLE_DEV_MODE
       end
     end
   end
 end
 
+module Coverband
+  class Configuration
+    attr_accessor :api_key
+  end
+end
+
+ENV['COVERBAND_DISABLE_AUTO_START'] = COVERBAND_ORIGINAL_START
 Coverband.configure do |config|
   # Use The Test Service Adapter
-  config.store = Coverband::Adapters::Service.new(COVERBAND_SERVICE_URL)
+  config.store = Coverband::Adapters::Service.new(Coverband::COVERBAND_SERVICE_URL)
 
   # default to tracking views true
-  config.track_views = ENV['COVERBAND_ENABLE_VIEW_TRACKER'] ? true : false
+  config.track_views = if ENV['COVERBAND_DISABLE_VIEW_TRACKER']
+      false
+    elsif Coverband.service_disabled_dev_test_env?
+      false
+    else
+      true
+    end
 
   # report every 10m by default
-  config.background_reporting_sleep_seconds = COVERBAND_ENV == 'production' ? 600 : 60
+  config.background_reporting_sleep_seconds = Coverband::COVERBAND_ENV == 'production' ? Coverband::COVERBAND_REPORT_PERIOD : 60
   # add a wiggle to avoid service stampede
-  config.reporting_wiggle = COVERBAND_ENV == 'production' ? 90 : 6
+  config.reporting_wiggle = Coverband::COVERBAND_ENV == 'production' ? 90 : 6
 
-  if COVERBAND_ENV == 'test'
+  if Coverband::COVERBAND_ENV == 'test'
     config.background_reporting_enabled = false
   end
 end
+
+# NOTE: it is really hard to bypass / overload our config we should fix this in Coverband
+# this hopefully detects anyone that has both gems and was trying to configure Coverband themselves.
+if File.exist?('./config/coverband.rb')
+  puts "Warning: config/coverband.rb found, this overrides coverband service allowing one to setup open source Coverband" if Coverband.configuration.verbose || COVERBAND_ENABLE_DEV_MODE
+end
+
+Coverband.configure('./config/coverband_service.rb') if File.exist?('./config/coverband_service.rb')
+Coverband.start
+require "coverband/utils/railtie" if defined? ::Rails::Railtie
+require "coverband/integrations/resque" if defined? ::Resque

data/lib/coverband/service/client/version.rb

@@ -1,7 +1,7 @@
 module Coverband
   module Service
     module Client
-      VERSION = '0.0.6'
+      VERSION = '0.0.11'
     end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: coverband-service-client
 version: !ruby/object:Gem::Version
-  version: 0.0.6
+  version: 0.0.11
 platform: ruby
 authors:
 - Dan Mayer
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2020-03-02 00:00:00.000000000 Z
+date: 2020-05-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler