Handle strategy fallbacks properly (#809)

lib/sidekiq_unique_jobs/lock_config.rb

@@ -113,14 +113,20 @@ def errors_as_string
 
     # the strategy to use as conflict resolution from sidekiq client
     def on_client_conflict
-      @on_client_conflict ||= on_conflict["client"] || on_conflict[:client] if on_conflict.is_a?(Hash)
-      @on_client_conflict ||= on_conflict
+      if on_conflict.is_a?(Hash)
+        @on_client_conflict ||= on_conflict["client"] || on_conflict[:client]
+      else
+        @on_client_conflict ||= on_conflict
+      end
     end
 
     # the strategy to use as conflict resolution from sidekiq server
     def on_server_conflict
-      @on_server_conflict ||= on_conflict["server"] || on_conflict[:server] if on_conflict.is_a?(Hash)
-      @on_server_conflict ||= on_conflict
+      if on_conflict.is_a?(Hash)
+        @on_server_conflict ||= on_conflict["server"] || on_conflict[:server]
+      else
+        @on_server_conflict ||= on_conflict
+      end
     end
   end
 end

spec/sidekiq_unique_jobs/lock_config_spec.rb

@@ -146,6 +146,12 @@
 
       it { is_expected.to eq(:replace) }
     end
+
+    context "when on_conflict is defined only for server" do
+      let(:on_conflict) { {server: :log}}
+
+      it { is_expected.to be_nil }
+    end
   end
 
   describe "#on_server_conflict" do
@@ -158,5 +164,11 @@
 
       it { is_expected.to eq(:reschedule) }
     end
+
+    context "when on_conflict is defined only for client" do
+      let(:on_conflict) { {client: :log}}
+
+      it { is_expected.to be_nil }
+    end
   end
 end