ap-5119: Temp commit to allow for testing on uat/locally

do not merge

app/models/provider.rb

@@ -18,7 +18,8 @@ class Provider < ApplicationRecord
   delegate :name, to: :firm, prefix: true, allow_nil: true
 
   def update_details
-    return unless HostEnv.staging_or_production?
+    # Temporarily disable this guard clause to allow for testing on uat
+    # return unless HostEnv.staging_or_production?
 
     # only schedule a background job to update details for staging and live
     ProviderDetailsCreatorWorker.perform_async(id)

spec/models/provider_spec.rb

@@ -20,14 +20,14 @@
         end
       end
 
-      context "when not staging or production" do
-        before { allow(HostEnv).to receive(:staging_or_production?).and_return(false) }
-
-        it "does not schedule a background job" do
-          expect(ProviderDetailsCreatorWorker).not_to receive(:perform_async).with(provider.id)
-          provider.update_details
-        end
-      end
+      # context "when not staging or production" do
+      #   before { allow(HostEnv).to receive(:staging_or_production?).and_return(false) }
+
+      #   it "does not schedule a background job" do
+      #     expect(ProviderDetailsCreatorWorker).not_to receive(:perform_async).with(provider.id)
+      #     provider.update_details
+      #   end
+      # end
     end
   end
 

spec/requests/saml_sessions_spec.rb

@@ -199,8 +199,8 @@
 
           it "does not schedule a job to update the provider details" do
             expect(provider).to receive(:update_details).and_call_original
-            expect(ProviderDetailsCreatorWorker).not_to receive(:perform_async).with(provider.id)
-            expect(ProviderDetailsCreator).not_to receive(:call).with(provider)
+            # expect(ProviderDetailsCreatorWorker).not_to receive(:perform_async).with(provider.id)
+            # expect(ProviderDetailsCreator).not_to receive(:call).with(provider)
             post_request
           end
         end