test correctly

Author: wow-such-code

Diff for: src/test/groovy/life/qbic/registration/handler/DatasetLocatorImplSpec.groovy

@@ -30,7 +30,8 @@ class DatasetLocatorImplSpec extends Specification {
     def "Given a relative path, when a DatasetLocator is created then an IllegalArgumentException is thrown"() {
         when:
         DatasetLocator locator = DatasetLocatorImpl.of(relativePathExample,
-                new Provenance("origin","user","id",new ArrayList<String>()))
+                new Provenance("origin","user","id",new ArrayList<String>(),
+                        new ArrayList<String>()))
 
         then:
         thrown(IllegalArgumentException.class)
@@ -39,7 +40,8 @@ class DatasetLocatorImplSpec extends Specification {
     def "Given an absolute path, when a DatasetLocator is created then an instance of this class is returned"() {
         when:
         DatasetLocator locator = DatasetLocatorImpl.of(flatDataStructureExample,
-                new Provenance("origin","user","id", new ArrayList<String>()))
+                new Provenance("origin","user","id", new ArrayList<String>(),
+                        new ArrayList<String>()))
 
         then:
         noExceptionThrown()
@@ -48,7 +50,9 @@ class DatasetLocatorImplSpec extends Specification {
     def "Given a structure with more than one file, when the path to the dataset folder is requested, return the absolute path to the parent folder"() {
         given:
         DatasetLocator locator = DatasetLocatorImpl.of(multiFileDataStructureExample,
-                new Provenance("origin","user","NGSQABCD006AO-25838529214608",new ArrayList<String>()))
+                new Provenance("origin","user","NGSQABCD006AO-25838529214608",
+                        new ArrayList<String>(Arrays.asList("NGSQABCD006AO_left.fastq",
+                                "NGSQABCD006AO_right.fastq")),new ArrayList<String>()))
 
         when:
         String datasetPath = locator.getPathToDataset()
@@ -60,7 +64,9 @@ class DatasetLocatorImplSpec extends Specification {
     def "Given a flat structure, when the path to the dataset folder is requested, return the given path"() {
         given:
         DatasetLocator locator = DatasetLocatorImpl.of(flatDataStructureExample,
-                new Provenance("origin","user","NGSQABCD006AO-25838529214608",new ArrayList<String>()))
+                new Provenance("origin","user","NGSQABCD006AO-25838529214608",
+                        new ArrayList<String>(Arrays.asList("NGSQABCD006AO-25838529214608.fastq")),
+                        new ArrayList<String>()))
 
         when:
         String datasetPath = locator.getPathToDataset()
@@ -70,19 +76,15 @@ class DatasetLocatorImplSpec extends Specification {
     }
 
     def setupMultiFileDataset() {
-        File file = File.createTempDir()
-
-        File dataFolders = new File(file.getAbsolutePath()+
-                '/NGSQABCD006AO-25838529214608/NGSQABCD006AO-25838529214608_dataset')
-        dataFolders.mkdirs()
+        File tempDir = File.createTempDir()
 
-        multiFileContainer = dataFolders.getAbsolutePath()
+        multiFileContainer = tempDir.getAbsolutePath() + File.separator + "data"
 
-        new File(dataFolders.getAbsolutePath() + File.separator + "NGSQABCD006AO_left.fastq").createNewFile()
-        new File(dataFolders.getAbsolutePath() + File.separator + "NGSQABCD006AO_right.fastq").createNewFile()
-        new File(file.getAbsolutePath() + File.separator + "provenance.json").createNewFile()
+                new File(tempDir.getAbsolutePath() + File.separator + "NGSQABCD006AO_left.fastq").createNewFile()
+        new File(tempDir.getAbsolutePath() + File.separator + "NGSQABCD006AO_right.fastq").createNewFile()
+        new File(tempDir.getAbsolutePath() + File.separator + "provenance.json").createNewFile()
 
-        multiFileDataStructureExample = file.getAbsolutePath()
+        multiFileDataStructureExample = tempDir.getAbsolutePath()
     }
 
     def setupRelativePathExample() {
@@ -91,18 +93,15 @@ class DatasetLocatorImplSpec extends Specification {
     }
 
     def setupFlatDatasetExample() {
-        File file = File.createTempDir()
-
-        File dataFolders = new File(file.getAbsolutePath()+'/NGSQABCD006AO-25838529214608.fastq_dataset')
-        dataFolders.mkdirs()
+        File tempDir = File.createTempDir()
 
-        File singleFile = new File(dataFolders.getAbsolutePath() + File.separator + "NGSQABCD006AO-25838529214608.fastq")
+        File singleFile = new File(tempDir.getAbsolutePath() + File.separator + "NGSQABCD006AO-25838529214608.fastq")
         singleFile.createNewFile()
         singleDatasetFile = singleFile.getAbsolutePath()
 
-        new File(file.getAbsolutePath() + File.separator + "provenance.json").createNewFile()
+        new File(tempDir.getAbsolutePath() + File.separator + "provenance.json").createNewFile()
 
-        flatDataStructureExample = file.getAbsolutePath()
+        flatDataStructureExample = tempDir.getAbsolutePath()
     }
 
 }