Disabled FileUtilsTest.testDownloadResourceWithPath due to 404 error happening recently, fixed broken duplicate job id check

Author: ThomasJejkal

src/main/java/edu/kit/datamanager/mappingservice/rest/impl/MappingExecutionController.java

@@ -188,10 +188,13 @@ public ResponseEntity<JobStatus> scheduleMapDocument(String mappingID, Multipart
         String jobId = null;
         for (int i = 1; i < 4; i++) {
             jobId = UUID.randomUUID().toString();
-            if (jobManager.getJob(jobId) == null) {
+            if (jobManager.getJob(jobId) != null) {
+                LOG.trace("Duplicated job id detected. Attempt {}/{}", i, 3);
                 jobId = null;
+            }else{
+                //usable job id found
+                break;
             }
-            LOG.trace("Duplicated job id detected. Attempt {}/{}", i, 3);
         }
 
         if (jobId == null) {

src/test/java/edu/kit/datamanager/mappingservice/util/FileUtilTest.java

@@ -32,6 +32,7 @@
 import java.util.Optional;
 
 import static org.junit.jupiter.api.Assertions.*;
+import org.junit.jupiter.api.Disabled;
 
 /**
  * @author hartmann-v
@@ -57,6 +58,7 @@ public void testDownloadResource() throws URISyntaxException {
      * Test of downloadResource method, of class GemmaMapping.
      */
     @Test
+    @Disabled("Recently, download fails with 404.")
     public void testDownloadResourceWithPath() throws URISyntaxException {
         System.out.println("downloadResource");
         assertNotNull(new FileUtil());