rails_spec: fix failing test

* CHANGELOG: mention airbrake#591
ramkumar-kr · Aug 10, 2016 · 6fb4675 · 6fb4675
1 parent 01c8d74
commit 6fb4675
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 0 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -3,6 +3,9 @@ Airbrake Changelog
 
 ### master
 
+* **IMPORTANT**: fixed ActiveJob integration not re-raising exceptions, which
+  resulted into marking failed jobs as successfully completed
+  ([#591](https://github.com/airbrake/airbrake/issues/591))
 * Fixed Rails runner integration not reporting errors
   ([#582](https://github.com/airbrake/airbrake/issues/580))
 * Fixed bug with the [enum_field](https://github.com/jamesgolick/enum_field) gem

diff --git a/spec/integration/rails/rails_spec.rb b/spec/integration/rails/rails_spec.rb
@@ -117,6 +117,14 @@
           allow(Airbrake).to receive(:build_notice).and_return(nil)
           allow(Airbrake).to receive(:notify)
 
+          # Make sure we don't call `build_notice` more than 1 time. Rack
+          # integration will try to handle error 500 and we want to prevent
+          # that: https://github.com/airbrake/airbrake/pull/583
+          allow_any_instance_of(Airbrake::Rack::Middleware).to(
+            receive(:notify_airbrake).
+            and_return(nil)
+          )
+
           get '/active_job'
           sleep 2