diff --git a/carp/src/test/scala/nl/lumc/sasc/biopet/pipelines/carp/CarpTest.scala b/carp/src/test/scala/nl/lumc/sasc/biopet/pipelines/carp/CarpTest.scala index 5d8bbd8ad77129961813b0e328e8df27bce28b00..beb8669763a6cddc74755d963dcf3c7dbebe3c5b 100644 --- a/carp/src/test/scala/nl/lumc/sasc/biopet/pipelines/carp/CarpTest.scala +++ b/carp/src/test/scala/nl/lumc/sasc/biopet/pipelines/carp/CarpTest.scala @@ -14,19 +14,19 @@ */ package nl.lumc.sasc.biopet.pipelines.carp -import java.io.{ File, FileOutputStream } +import java.io.{File, FileOutputStream} import com.google.common.io.Files import nl.lumc.sasc.biopet.utils.config.Config import nl.lumc.sasc.biopet.extensions.bwa.BwaMem import nl.lumc.sasc.biopet.extensions.macs2.Macs2CallPeak -import nl.lumc.sasc.biopet.extensions.picard.{ MergeSamFiles, SortSam } -import nl.lumc.sasc.biopet.utils.ConfigUtils +import nl.lumc.sasc.biopet.extensions.picard.{MergeSamFiles, SortSam} +import nl.lumc.sasc.biopet.utils.{ConfigUtils, Logging} import org.apache.commons.io.FileUtils import org.broadinstitute.gatk.queue.QSettings import org.scalatest.Matchers import org.scalatest.testng.TestNGSuite -import org.testng.annotations.{ AfterClass, DataProvider, Test } +import org.testng.annotations.{AfterClass, DataProvider, Test} /** * Test class for [[Carp]] @@ -72,6 +72,7 @@ class CarpTest extends TestNGSuite with Matchers { intercept[IllegalStateException] { initPipeline(map).script() } + Logging.errors.clear() } else { // When samples are correct val carp = initPipeline(map) carp.script() diff --git a/gears/src/test/scala/nl/lumc/sasc/biopet/pipelines/gears/GearsTest.scala b/gears/src/test/scala/nl/lumc/sasc/biopet/pipelines/gears/GearsTest.scala index c3c13da1bbe96505847a70625f014acdd4f7a1dd..01332f4c3f8e94a9b8d9b27d7b0246ba2fa5b731 100644 --- a/gears/src/test/scala/nl/lumc/sasc/biopet/pipelines/gears/GearsTest.scala +++ b/gears/src/test/scala/nl/lumc/sasc/biopet/pipelines/gears/GearsTest.scala @@ -72,7 +72,7 @@ abstract class GearsTest extends TestNGSuite with Matchers { } if (!sample1 && !sample2) { // When no samples - intercept[IllegalArgumentException] { + intercept[IllegalStateException] { initPipeline(map).script() } Logging.errors.clear() diff --git a/mapping/src/test/scala/nl/lumc/sasc/biopet/pipelines/mapping/MultisampleMappingTest.scala b/mapping/src/test/scala/nl/lumc/sasc/biopet/pipelines/mapping/MultisampleMappingTest.scala index 525005fdde51037050ea079cff46b5313f3c0e15..4c5f5cdba753a3db5e4df82ae13342d182b65ed5 100644 --- a/mapping/src/test/scala/nl/lumc/sasc/biopet/pipelines/mapping/MultisampleMappingTest.scala +++ b/mapping/src/test/scala/nl/lumc/sasc/biopet/pipelines/mapping/MultisampleMappingTest.scala @@ -14,17 +14,17 @@ */ package nl.lumc.sasc.biopet.pipelines.mapping -import java.io.{ File, FileOutputStream } +import java.io.{File, FileOutputStream} import com.google.common.io.Files import nl.lumc.sasc.biopet.extensions.kraken.Kraken -import nl.lumc.sasc.biopet.extensions.picard.{ MarkDuplicates, MergeSamFiles } -import nl.lumc.sasc.biopet.utils.ConfigUtils +import nl.lumc.sasc.biopet.extensions.picard.{MarkDuplicates, MergeSamFiles} +import nl.lumc.sasc.biopet.utils.{ConfigUtils, Logging} import nl.lumc.sasc.biopet.utils.config.Config import org.broadinstitute.gatk.queue.QSettings import org.scalatest.Matchers import org.scalatest.testng.TestNGSuite -import org.testng.annotations.{ DataProvider, Test } +import org.testng.annotations.{DataProvider, Test} /** * Created by pjvanthof on 15/05/16. @@ -72,9 +72,10 @@ trait MultisampleMappingTestTrait extends TestNGSuite with Matchers { } if (!sample1 && !sample2 && !sample3 && !sample4) { // When no samples - intercept[IllegalArgumentException] { + intercept[IllegalStateException] { initPipeline(map).script() } + Logging.errors.clear() } else if (sample4 && !bamToFastq && !correctReadgroups) { intercept[IllegalStateException] { initPipeline(map).script() diff --git a/shiva/src/test/scala/nl/lumc/sasc/biopet/pipelines/shiva/ShivaTest.scala b/shiva/src/test/scala/nl/lumc/sasc/biopet/pipelines/shiva/ShivaTest.scala index 56881f2dd5f2d48ef5cda84b87216106c57ce1ba..35c2baabdbdfdf264886b8d5a39948e86be19f1d 100644 --- a/shiva/src/test/scala/nl/lumc/sasc/biopet/pipelines/shiva/ShivaTest.scala +++ b/shiva/src/test/scala/nl/lumc/sasc/biopet/pipelines/shiva/ShivaTest.scala @@ -14,18 +14,18 @@ */ package nl.lumc.sasc.biopet.pipelines.shiva -import java.io.{ File, FileOutputStream } +import java.io.{File, FileOutputStream} import com.google.common.io.Files -import nl.lumc.sasc.biopet.extensions.gatk.{ BaseRecalibrator, IndelRealigner, PrintReads, RealignerTargetCreator } +import nl.lumc.sasc.biopet.extensions.gatk.{BaseRecalibrator, IndelRealigner, PrintReads, RealignerTargetCreator} import nl.lumc.sasc.biopet.extensions.picard.MarkDuplicates import nl.lumc.sasc.biopet.extensions.tools.VcfStats -import nl.lumc.sasc.biopet.utils.ConfigUtils +import nl.lumc.sasc.biopet.utils.{ConfigUtils, Logging} import nl.lumc.sasc.biopet.utils.config.Config import org.broadinstitute.gatk.queue.QSettings import org.scalatest.Matchers import org.scalatest.testng.TestNGSuite -import org.testng.annotations.{ DataProvider, Test } +import org.testng.annotations.{DataProvider, Test} /** * Class for testing shiva @@ -86,6 +86,7 @@ trait ShivaTestTrait extends TestNGSuite with Matchers { intercept[IllegalArgumentException] { initPipeline(map).script() } + Logging.errors.clear() } else { val pipeline = initPipeline(map) pipeline.script()