fix tests and small bug

Author: JoaoJandre

plugins/backup/kboss/src/main/java/org/apache/cloudstack/backup/KbossBackupProvider.java

@@ -1337,6 +1337,9 @@ protected List<BackupDeltaTO> getBackupDeltaTOList(long backupId) {
     }
 
     protected void cleanupValidation(boolean startedVm, UserVmVO validationVm, BackupVO backupVO, List<VolumeVO> volumeVOs) {
+        if (validationVm == null) {
+            return;
+        }
         if (startedVm) {
             userVmManager.stopVirtualMachine(validationVm.getId(), true);
         }

plugins/backup/kboss/src/test/java/org/apache/cloudstack/backup/KbossBackupProviderTest.java

@@ -1561,7 +1561,7 @@ public void validateWithValidationVmTestValidationVmIsNull() {
         boolean result = kbossBackupProviderSpy.validateWithValidationVm(backupId, 2L, backupVoMock);
 
         assertFalse(result);
-        verify(kbossBackupProviderSpy, never()).cleanupValidation(anyBoolean(), any(), any(), any());
+        verify(kbossBackupProviderSpy).cleanupValidation(anyBoolean(), any(), any(), any());
     }
 
     @Test
@@ -2819,7 +2819,7 @@ public void validateCompressionStateForRestoreAndGetBackupTestUnableToLock() {
         Pair<Boolean, BackupVO> result = kbossBackupProviderSpy.validateCompressionStateForRestoreAndGetBackup(backupId);
 
         assertFalse(result.first());
-        verify(kbossBackupProviderSpy).releaseBackup(backupId);
+        verify(kbossBackupProviderSpy, never()).releaseBackup(backupId);
     }
 
     @Test
@@ -2856,7 +2856,7 @@ public void validateBackupStateForRemovalTestUnableToLock() {
         boolean result = kbossBackupProviderSpy.validateBackupStateForRemoval(backupId);
 
         assertFalse(result);
-        verify(kbossBackupProviderSpy).releaseBackup(backupId);
+        verify(kbossBackupProviderSpy, never()).releaseBackup(backupId);
     }
 
     @Test
@@ -2922,7 +2922,7 @@ public void validateBackupStateForStartCompressionAndUpdateCompressionStatusTest
 
         assertFalse(result.first());
         verify(backupDaoMock, never()).update(backupId, backupVoMock);
-        verify(kbossBackupProviderSpy).releaseBackup(backupId);
+        verify(kbossBackupProviderSpy, never()).releaseBackup(backupId);
     }
 
     @Test
@@ -2961,7 +2961,7 @@ public void validateBackupStateForFinalizeCompressionTestUnableToLock() {
 
         assertFalse(result.first());
         verify(backupDaoMock, never()).update(backupId, backupVoMock);
-        verify(kbossBackupProviderSpy).releaseBackup(backupId);
+        verify(kbossBackupProviderSpy, never()).releaseBackup(backupId);
     }
 
     @Test
@@ -3028,7 +3028,7 @@ public void validateBackupStateForRestoreBackupToVMTestUnableToLock() {
 
         assertFalse(result.first());
         verify(backupDaoMock, never()).update(backupId, backupVoMock);
-        verify(kbossBackupProviderSpy).releaseBackup(backupId);
+        verify(kbossBackupProviderSpy, never()).releaseBackup(backupId);
     }
 
     @Test
@@ -3077,7 +3077,7 @@ public void validateBackupStateForValidationTestUnableToLock() {
         boolean result = kbossBackupProviderSpy.validateBackupStateForValidation(backupId);
 
         assertFalse(result);
-        verify(kbossBackupProviderSpy).releaseBackup(backupId);
+        verify(kbossBackupProviderSpy, never()).releaseBackup(backupId);
     }
 
     @Test