control-service: fix disabled jobs not building (#3376)

Fix the deployment logic, so that a job would be
built, even if it is disabled.

Remove redundant test.

---------

Signed-off-by: Dako Dakov <ddakov@vmware.com>
Co-authored-by: github-actions <>

Author: dakodakov

projects/control-service/projects/pipelines_control_service/src/main/java/com/vmware/taurus/service/deploy/DeploymentServiceV2.java

@@ -160,8 +160,9 @@ public void updateDeployment(
           dockerRegistryService.dataJobImage(
               desiredJobDeployment.getDataJobName(), desiredJobDeployment.getGitCommitSha());
 
-      // If the job suspension operation is performed, we don't want to build the image.
-      if (Boolean.FALSE.equals(desiredJobDeployment.getEnabled())
+      // If a previously enabled job is being disabled, we don't want to build the image.
+      if ((Boolean.FALSE.equals(desiredJobDeployment.getEnabled())
+              && Boolean.TRUE.equals(actualJobDeployment.getEnabled()))
           || jobImageBuilder.buildImage(
               imageName, dataJob, desiredJobDeployment, actualJobDeployment, sendNotification)) {
         ActualDataJobDeployment actualJobDeploymentResult =

projects/control-service/projects/pipelines_control_service/src/test/java/com/vmware/taurus/service/deploy/DeploymentServiceV2TestIT.java

@@ -78,13 +78,6 @@ public void updateDeployment_withDesiredDeploymentStatusNone_shouldStartDeployme
     updateDeployment(false, true, false, 1);
   }
 
-  @Test
-  public void
-      updateDeployment_withDesiredDeploymentEnabledFalseAndActualDeploymentEnabledFalseAndBuildImageSucceededFalse_shouldUpdateDeployment()
-          throws IOException, InterruptedException, ApiException {
-    updateDeployment(false, false, false, 1);
-  }
-
   @Test
   public void
       updateDeployment_withDesiredDeploymentEnabledTrueAndActualDeploymentEnabledFalseAndBuildImageSucceededTrue_shouldUpdateDeployment()