airbrake 5.0.4 → 5.0.5

data/spec/integration/rails/rails_spec.rb

@@ -99,7 +99,7 @@ RSpec.describe "Rails integration specs" do
         wait_for(
           a_request(:post, endpoint).
           with(body: /"message":"active_job error"/)
-        ).to have_been_made.twice
+        ).to have_been_made.at_least_once
       end
     end
   end

data/spec/integration/shared_examples/rack_examples.rb

@@ -85,7 +85,7 @@ RSpec.shared_examples 'rack examples' do
 
     context "when additional parameters present" do
       before do
-        get '/crash', nil, 'HTTP_USER_AGENT' => 'Bot'
+        get '/crash', nil, 'HTTP_USER_AGENT' => 'Bot', 'HTTP_REFERER' => 'bingo.com'
       end
 
       it "features url" do
@@ -103,4 +103,24 @@ RSpec.shared_examples 'rack examples' do
       end
     end
   end
+
+  describe "environment payload" do
+    before do
+      get '/crash', nil, 'HTTP_REFERER' => 'bingo.com'
+    end
+
+    it "features referer" do
+      wait_for_a_request_with_body(/"environment":{.*"referer":"bingo.com".*}/)
+    end
+
+    it "contains HTTP headers" do
+      wait_for_a_request_with_body(
+        /"environment":{.*"headers":{.*"CONTENT_LENGTH":"0".*}/
+      )
+    end
+
+    it "contains HTTP method" do
+      wait_for_a_request_with_body(/"environment":{.*"httpMethod":"GET".*}/)
+    end
+  end
 end

data/spec/unit/rack/user_spec.rb

@@ -52,13 +52,26 @@ RSpec.describe Airbrake::Rack::User do
     end
 
     context "when the current_user Rails controller method is defined" do
-      it "returns the wrapped user" do
-        controller = instance_double('DummyController')
-        env = env_for('/', 'action_controller.instance' => controller)
-        allow(controller).to receive(:current_user) { user }
+      context "and it is nil" do
+        it "returns nil" do
+          controller = instance_double('DummyController')
+          env = env_for('/', 'action_controller.instance' => controller)
+          allow(controller).to receive(:current_user) { nil }
 
-        retval = described_class.extract(env)
-        expect(retval).to be_a(described_class)
+          retval = described_class.extract(env)
+          expect(retval).to be_nil
+        end
+      end
+
+      context "and it is not nil" do
+        it "returns the wrapped user" do
+          controller = instance_double('DummyController')
+          env = env_for('/', 'action_controller.instance' => controller)
+          allow(controller).to receive(:current_user) { user }
+
+          retval = described_class.extract(env)
+          expect(retval).to be_a(described_class)
+        end
       end
     end
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: airbrake
 version: !ruby/object:Gem::Version
-  version: 5.0.4
+  version: 5.0.5
 platform: ruby
 authors:
 - Airbrake Technologies, Inc.
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-02-02 00:00:00.000000000 Z
+date: 2016-02-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: airbrake-ruby
@@ -187,7 +187,11 @@ files:
 - spec/apps/rack/dummy_app.rb
 - spec/apps/rails/dummy_app.rb
 - spec/apps/rails/dummy_task.rake
+- spec/apps/rails/logs/32.log
+- spec/apps/rails/logs/40.log
+- spec/apps/rails/logs/41.log
 - spec/apps/rails/logs/42.log
+- spec/apps/rails/logs/50.log
 - spec/apps/sinatra/dummy_app.rb
 - spec/integration/rack/rack_spec.rb
 - spec/integration/rails/rails_spec.rb
@@ -234,7 +238,11 @@ test_files:
 - spec/spec_helper.rb
 - spec/apps/rack/dummy_app.rb
 - spec/apps/sinatra/dummy_app.rb
+- spec/apps/rails/logs/40.log
 - spec/apps/rails/logs/42.log
+- spec/apps/rails/logs/50.log
+- spec/apps/rails/logs/32.log
+- spec/apps/rails/logs/41.log
 - spec/apps/rails/dummy_app.rb
 - spec/apps/rails/dummy_task.rake
 - spec/airbrake_spec.rb