minitest_rollbar 0.3.1 → 0.3.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: f85c64bd781dc64fa18e568f441420d3a519affc
-  data.tar.gz: d468c26e153becb6a59d6fa0f5f027c3f22c5a66
+  metadata.gz: 32ce6e93eae1a3574774ebb77611485bf366e6d7
+  data.tar.gz: f3ebec98ec6b0d9e91908fb1b172191d4b79c352
 SHA512:
-  metadata.gz: 6e90e7b0e11c0daa0126112805b62bc9158026d62ad5b62291429ef70a08713ebd480e76d32ae59917a2755a6240a26e7344e0d2c2ce1cf3ca5cc7ffea7366a8
-  data.tar.gz: 7d5d29fcf1728faef3a421e58cedeb002d3cfbae70d97cd58240accb1b80c437c6ab63d884dca54b525cf581000515194abb9ef99f5fb46255c0df15810e193f
+  metadata.gz: e25c6103a31fb6263fac2f80e0be6b5c101c7ecd695c89dcfbd09f5702be15e75f3ed2719ef4608c64e3bcc4fb09c12dddf7397ddcc170bea1ed46a02e833a04
+  data.tar.gz: d65d445cd3b9f57b9fc5a5a5d6a5a020bd3e05009d5624fc21e026bb56d9d5c33d56d56f3b176a289dd733c6fb788c1c3893c007433a63d43838de728b17eed4

data/lib/minitest_rollbar/reporters.rb

@@ -1,29 +1,24 @@
 require 'rollbar'
 require 'minitest/reporters'
 module MinitestRollbar
-  class << self
-    attr_accessor :use_default_grouping
-  end
 
   class RollbarReporter < Minitest::Reporters::BaseReporter
+
+    attr_accessor :use_default_grouping
+
     def initialize(options = {})
-      rollbar_config = options.delete(:rollbar_config) || {}
+      @rollbar_config = options.delete(:rollbar_config) || {}
       super(options)
 
       @sequential_exception_count = 0
-      # Inspect will return ExceptionType + Message.
-      # E.g #<Selenium::WebDriver::Error::NoSuchWindowError: Window not found. The browser window may have been closed.>
-      # Use this as a criteria for log batch grouping
+      @use_default_grouping = false
       @previous_exception_inspect_result = nil
       @previous_exception = nil
 
-      Rollbar.configure do |config|
-        rollbar_config.each do |key, value|
-          config.send "#{key}=", value
-        end
-      end
+      # Rollbar global setting, notifier instance won't report if this is not set
+      Rollbar.configuration.enabled = true
 
-      raise 'Must set rollbar access token' if Rollbar.configuration.access_token.nil?
+      raise 'Must set rollbar access token' if @rollbar_config[:access_token].nil?
     end
 
     def record(result)
@@ -67,11 +62,15 @@ module MinitestRollbar
     end
 
     def notifier
-      if MinitestRollbar.use_default_grouping.nil?
-        Rollbar.scope({count: @sequential_exception_count, commit_hash: git_commit_hash, build_config: build_config_name,  fingerprint: @previous_exception_inspect_result})
+      if @use_default_grouping
+        @notifier = Rollbar.scope({environment: 'PossibleInfraFlaky',count: @sequential_exception_count, commit_hash: git_commit_hash, build_config: build_config_name})
       else
-        Rollbar.scope({count: @sequential_exception_count, commit_hash: git_commit_hash, build_config: build_config_name})
+        @notifier = Rollbar.scope({environment: 'PossibleInfraFlaky',count: @sequential_exception_count, commit_hash: git_commit_hash, build_config: build_config_name, fingerprint: @previous_exception_inspect_result})
+      end
+      @rollbar_config.each do |key,value|
+        @notifier.configuration.send("#{key}=", value)
       end
+      @notifier
     end
 
     def report_error_to_rollbar(notifier)

data/lib/minitest_rollbar/version.rb

@@ -1,3 +1,3 @@
 module MinitestRollbar
-  VERSION = '0.3.1'.freeze
+  VERSION = '0.3.3'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: minitest_rollbar
 version: !ruby/object:Gem::Version
-  version: 0.3.1
+  version: 0.3.3
 platform: ruby
 authors:
 - Yuesong Wang
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-07-23 00:00:00.000000000 Z
+date: 2016-08-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler