fixed linting on sync and added test for RedisMutex lock error (#19237)

* fixed linting on sync and added test for RedisMutex lock error

* added epe id to display in error message

* fixed linting

* rubocop long line fixes

* fixed new line causing test failure

* fixed new lines

* fixed rubocop long lines

* APPEALS-28960 Updated formatting of log message and content to include "#sync!".

---------

Co-authored-by: TuckerRose <tuckerrose@tuckerroses-mbp.lan>
Co-authored-by: Jeffrey Aaron Willis <Willis_Jeffrey@bah.com>

app/models/end_product_establishment.rb

@@ -233,7 +233,8 @@ def sync!
       save_updated_end_product_code!
     end
   rescue RedisMutex::LockError
-    Rails.logger.error("failed to acquire lock! EPE sync is being called by another process. Please try again later.")
+    Rails.logger.error("Failed to acquire lock for EPE ID: #{id}!  #sync! is being"\
+                       " called by another process. Please try again later.")
   rescue EstablishedEndProductNotFound, AppealRepository::AppealNotValidToReopen => error
     raise error
   rescue StandardError => error

spec/models/end_product_establishment_spec.rb

@@ -851,6 +851,19 @@
       ]
     end
 
+    context "when lock acquisition fails" do
+      before do
+        allow(RedisMutex).to receive(:with_lock).and_raise(RedisMutex::LockError)
+      end
+
+      it "logs the error message" do
+        expect(Rails.logger).to receive(:error)
+          .with("Failed to acquire lock for EPE ID: #{end_product_establishment.id}!"\
+                "  #sync! is being called by another process. Please try again later.")
+        end_product_establishment.sync!
+      end
+    end
+
     context "when matching end product has not yet been established" do
       it "raises EstablishedEndProductNotFound error" do
         expect { subject }.to raise_error(EndProductEstablishment::EstablishedEndProductNotFound)