Addressed comments and fixed checkstyle issue

Author: sandeeplocharla

plugins/storage/volume/ontap/src/main/java/org/apache/cloudstack/storage/service/StorageStrategy.java

@@ -19,7 +19,6 @@
 
 package org.apache.cloudstack.storage.service;
 
-import java.util.ArrayList;
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;

plugins/storage/volume/ontap/src/test/java/org/apache/cloudstack/storage/service/StorageStrategyTest.java

@@ -643,6 +643,25 @@ public void testDeleteStorageVolume_feignException() {
         assertTrue(ex.getMessage().contains("Failed to delete volume"));
     }
 
+    @Test
+    public void testDeleteStorageVolume_notFound_404_returnsWithoutThrowing() {
+        // Setup
+        Volume volume = new Volume();
+        volume.setName("test-volume");
+        volume.setUuid("vol-uuid-1");
+
+        FeignException feignEx = mock(FeignException.class);
+        when(feignEx.status()).thenReturn(404);
+        when(volumeFeignClient.deleteVolume(anyString(), eq("vol-uuid-1")))
+                .thenThrow(feignEx);
+
+        // Execute - 404 means volume already gone on ONTAP, treated as no-op
+        storageStrategy.deleteStorageVolume(volume);
+
+        // Verify the delete was attempted
+        verify(volumeFeignClient).deleteVolume(anyString(), eq("vol-uuid-1"));
+    }
+
     // ========== getStoragePath() Tests ==========
 
     @Test