Blob Blame History Raw
From 499a87c80b56b299fdd8980c2361bf1e378d5e54 Mon Sep 17 00:00:00 2001
From: Ken Dreyer <ktdreyer@ktdreyer.com>
Date: Tue, 26 May 2015 21:53:45 -0600
Subject: [PATCH 5/5] make sidekiq gem optional for test suite

Fedora does not yet have a rubygem-sidekiq package. Instead of bailing
out when sidekiq is missing, just print a warning and move on.

When Fedora does get a sidekiq package, we can drop this patch.
---
 test/exception_notifier/sidekiq_test.rb | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/test/exception_notifier/sidekiq_test.rb b/test/exception_notifier/sidekiq_test.rb
index 95169c2..e634598 100644
--- a/test/exception_notifier/sidekiq_test.rb
+++ b/test/exception_notifier/sidekiq_test.rb
@@ -3,7 +3,11 @@ require "test_helper"
 # To allow sidekiq error handlers to be registered, sidekiq must be in
 # "server mode". This mode is triggered by loading sidekiq/cli. Note this
 # has to be loaded before exception_notification/sidekiq.
-require "sidekiq/cli"
+begin
+  require "sidekiq/cli"
+rescue LoadError
+  warn 'warning: sidekiq gem not found; skipping SidekiqTest'
+else
 
 require "exception_notification/sidekiq"
 
@@ -25,3 +29,5 @@ class SidekiqTest < ActiveSupport::TestCase
     server.handle_exception(exception, message)
   end
 end
+
+end
-- 
2.4.3