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