airbrake-ruby 1.4.1 → 1.4.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 2649182552b42388d488153498cd971ffa2c7ca1
-  data.tar.gz: 2d8eb9a2e0e54e8784e8f9204545519456f4b87c
+  metadata.gz: 26dd376dd3646e94ceb28cd59f49e3f5edd67321
+  data.tar.gz: 413ac77fb12f382bac026959ebc502ed0246eab1
 SHA512:
-  metadata.gz: d0b5e3f1bb6d74d84a619bfeb416688cbb7c773b14f9328365c800aa0b8532673ef58727991dd1e035256e6018f8d12186bc37ce2effc9d5ee5e10018210efbb
-  data.tar.gz: c413d29e92a294952a73db358a7a62949075ab4d6bb5ed5b6295f7682517285d601e32bf93ed6fb2641fd6610ea2822e8b0e760d207d6c9033ced88aa5b919fb
+  metadata.gz: e49ee6c9772cd21dfa62c6441218f122cfe382e27bb60200506e2a816c5c981b181815013276aa8a8933d2a204e1560e5f3a8d71300fa6fc9c9fb0d2c2ecf44b
+  data.tar.gz: 753a7a36e840edc99a555950722bb7c7eae06efb61f2f297a563f077ca8150fd474db45c6acfe5ee0df37644283d3e566b29ef87f27bf8df3caf7124873ec8d9

data/lib/airbrake-ruby/config.rb

@@ -138,6 +138,11 @@ module Airbrake
     # @return [Boolean] true if the config ignores current environment, false
     #   otherwise
     def ignored_environment?
+      if ignore_environments.any? && environment.nil?
+        logger.warn("#{LOG_LABEL} the 'environment' option is not set, " \
+                    "'ignore_environments' has no effect")
+      end
+
       ignore_environments.include?(environment)
     end
 

data/lib/airbrake-ruby/version.rb

@@ -3,5 +3,5 @@
 module Airbrake
   ##
   # @return [String] the library version
-  AIRBRAKE_RUBY_VERSION = '1.4.1'.freeze
+  AIRBRAKE_RUBY_VERSION = '1.4.2'.freeze
 end

data/spec/config_spec.rb

@@ -156,4 +156,37 @@ RSpec.describe Airbrake::Config do
       end
     end
   end
+
+  describe "#ignored_environment?" do
+    describe "warnings" do
+      let(:out) { StringIO.new }
+      let(:config) { described_class.new(logger: Logger.new(out)) }
+
+      context "when 'ignore_environments' is set and 'environment' isn't" do
+        it "prints a warning" do
+          config.ignore_environments = [:bingo]
+
+          expect(config.ignored_environment?).to be_falsey
+          expect(out.string).to match(/ignore_environments' has no effect/)
+        end
+      end
+
+      context "when 'ignore_environments' is set along with 'environment'" do
+        it "doesn't print a warning" do
+          config.environment = :bango
+          config.ignore_environments = [:bingo]
+
+          expect(config.ignored_environment?).to be_falsey
+          expect(out.string).to be_empty
+        end
+      end
+
+      context "when 'ignore_environments' isn't set and 'environment' isn't set too" do
+        it "doesn't print a warning" do
+          expect(config.ignored_environment?).to be_falsey
+          expect(out.string).to be_empty
+        end
+      end
+    end
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: airbrake-ruby
 version: !ruby/object:Gem::Version
-  version: 1.4.1
+  version: 1.4.2
 platform: ruby
 authors:
 - Airbrake Technologies, Inc.
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-06-06 00:00:00.000000000 Z
+date: 2016-06-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec