Skip to content
Snippets Groups Projects
Commit 8bf162ad authored by Peter van 't Hof's avatar Peter van 't Hof
Browse files

Remove "reference" from testing

parent c82dc34f
No related branches found
No related tags found
No related merge requests found
Showing
with 12 additions and 10 deletions
...@@ -115,7 +115,6 @@ object ShivaTest { ...@@ -115,7 +115,6 @@ object ShivaTest {
"dir" -> "test", "dir" -> "test",
"vep_script" -> "test", "vep_script" -> "test",
"output_dir" -> outputDir, "output_dir" -> outputDir,
"reference" -> (outputDir + File.separator + "ref.fa"),
"reference_fasta" -> (outputDir + File.separator + "ref.fa"), "reference_fasta" -> (outputDir + File.separator + "ref.fa"),
"gatk_jar" -> "test", "gatk_jar" -> "test",
"samtools" -> Map("exe" -> "test"), "samtools" -> Map("exe" -> "test"),
......
...@@ -131,7 +131,6 @@ object ShivaVariantcallingTest { ...@@ -131,7 +131,6 @@ object ShivaVariantcallingTest {
"cache" -> true, "cache" -> true,
"dir" -> "test", "dir" -> "test",
"vep_script" -> "test", "vep_script" -> "test",
"reference" -> (outputDir + File.separator + "ref.fa"),
"reference_fasta" -> (outputDir + File.separator + "ref.fa"), "reference_fasta" -> (outputDir + File.separator + "ref.fa"),
"gatk_jar" -> "test", "gatk_jar" -> "test",
"samtools" -> Map("exe" -> "test"), "samtools" -> Map("exe" -> "test"),
......
...@@ -17,12 +17,13 @@ package nl.lumc.sasc.biopet.extensions.picard ...@@ -17,12 +17,13 @@ package nl.lumc.sasc.biopet.extensions.picard
import java.io.File import java.io.File
import nl.lumc.sasc.biopet.core.Reference
import nl.lumc.sasc.biopet.utils.config.Configurable import nl.lumc.sasc.biopet.utils.config.Configurable
import nl.lumc.sasc.biopet.core.summary.Summarizable import nl.lumc.sasc.biopet.core.summary.Summarizable
import org.broadinstitute.gatk.utils.commandline.{ Argument, Input, Output } import org.broadinstitute.gatk.utils.commandline.{ Argument, Input, Output }
/** Extension for picard CollectAlignmentSummaryMetrics */ /** Extension for picard CollectAlignmentSummaryMetrics */
class CollectAlignmentSummaryMetrics(val root: Configurable) extends Picard with Summarizable { class CollectAlignmentSummaryMetrics(val root: Configurable) extends Picard with Summarizable with Reference {
javaMainClass = new picard.analysis.CollectAlignmentSummaryMetrics().getClass.getName javaMainClass = new picard.analysis.CollectAlignmentSummaryMetrics().getClass.getName
@Input(doc = "The input SAM or BAM files to analyze. Must be coordinate sorted.", required = true) @Input(doc = "The input SAM or BAM files to analyze. Must be coordinate sorted.", required = true)
...@@ -41,7 +42,7 @@ class CollectAlignmentSummaryMetrics(val root: Configurable) extends Picard with ...@@ -41,7 +42,7 @@ class CollectAlignmentSummaryMetrics(val root: Configurable) extends Picard with
var output: File = _ var output: File = _
@Argument(doc = "Reference file", required = false) @Argument(doc = "Reference file", required = false)
var reference: File = config("reference") var reference: File = _
@Argument(doc = "ASSUME_SORTED", required = false) @Argument(doc = "ASSUME_SORTED", required = false)
var assumeSorted: Boolean = config("assumeSorted", default = true) var assumeSorted: Boolean = config("assumeSorted", default = true)
...@@ -52,6 +53,11 @@ class CollectAlignmentSummaryMetrics(val root: Configurable) extends Picard with ...@@ -52,6 +53,11 @@ class CollectAlignmentSummaryMetrics(val root: Configurable) extends Picard with
@Argument(doc = "STOP_AFTER", required = false) @Argument(doc = "STOP_AFTER", required = false)
var stopAfter: Option[Long] = config("stopAfter") var stopAfter: Option[Long] = config("stopAfter")
override def beforeGraph(): Unit = {
super.beforeGraph()
if (reference == null) reference = referenceFasta()
}
/** Returns command to execute */ /** Returns command to execute */
override def cmdLine = super.cmdLine + override def cmdLine = super.cmdLine +
required("INPUT=", input, spaceSeparated = false) + required("INPUT=", input, spaceSeparated = false) +
......
...@@ -17,6 +17,7 @@ package nl.lumc.sasc.biopet.extensions.picard ...@@ -17,6 +17,7 @@ package nl.lumc.sasc.biopet.extensions.picard
import java.io.File import java.io.File
import nl.lumc.sasc.biopet.core.Reference
import nl.lumc.sasc.biopet.utils.config.Configurable import nl.lumc.sasc.biopet.utils.config.Configurable
import nl.lumc.sasc.biopet.core.summary.Summarizable import nl.lumc.sasc.biopet.core.summary.Summarizable
import org.broadinstitute.gatk.utils.commandline.{ Argument, Input, Output } import org.broadinstitute.gatk.utils.commandline.{ Argument, Input, Output }
...@@ -24,7 +25,7 @@ import org.broadinstitute.gatk.utils.commandline.{ Argument, Input, Output } ...@@ -24,7 +25,7 @@ import org.broadinstitute.gatk.utils.commandline.{ Argument, Input, Output }
import scala.collection.immutable.Nil import scala.collection.immutable.Nil
/** Extension for picard CollectInsertSizeMetrics */ /** Extension for picard CollectInsertSizeMetrics */
class CollectInsertSizeMetrics(val root: Configurable) extends Picard with Summarizable { class CollectInsertSizeMetrics(val root: Configurable) extends Picard with Summarizable with Reference {
javaMainClass = new picard.analysis.CollectInsertSizeMetrics().getClass.getName javaMainClass = new picard.analysis.CollectInsertSizeMetrics().getClass.getName
@Input(doc = "The input SAM or BAM files to analyze. Must be coordinate sorted.", required = true) @Input(doc = "The input SAM or BAM files to analyze. Must be coordinate sorted.", required = true)
...@@ -37,7 +38,7 @@ class CollectInsertSizeMetrics(val root: Configurable) extends Picard with Summa ...@@ -37,7 +38,7 @@ class CollectInsertSizeMetrics(val root: Configurable) extends Picard with Summa
protected var outputHistogram: File = null protected var outputHistogram: File = null
@Argument(doc = "Reference file", required = false) @Argument(doc = "Reference file", required = false)
var reference: File = config("reference") var reference: File = _
@Argument(doc = "DEVIATIONS", required = false) @Argument(doc = "DEVIATIONS", required = false)
var deviations: Option[Double] = config("deviations") var deviations: Option[Double] = config("deviations")
...@@ -59,6 +60,7 @@ class CollectInsertSizeMetrics(val root: Configurable) extends Picard with Summa ...@@ -59,6 +60,7 @@ class CollectInsertSizeMetrics(val root: Configurable) extends Picard with Summa
override def beforeGraph() { override def beforeGraph() {
outputHistogram = new File(output + ".pdf") outputHistogram = new File(output + ".pdf")
if (reference == null) reference = referenceFasta()
} }
/** Returns command to execute */ /** Returns command to execute */
......
...@@ -116,7 +116,6 @@ object CarpTest { ...@@ -116,7 +116,6 @@ object CarpTest {
copyFile("ref.fa.fai") copyFile("ref.fa.fai")
val executables = Map( val executables = Map(
"reference" -> (outputDir + File.separator + "ref.fa"),
"reference_fasta" -> (outputDir + File.separator + "ref.fa"), "reference_fasta" -> (outputDir + File.separator + "ref.fa"),
"fastqc" -> Map("exe" -> "test"), "fastqc" -> Map("exe" -> "test"),
"seqtk" -> Map("exe" -> "test"), "seqtk" -> Map("exe" -> "test"),
......
...@@ -196,7 +196,6 @@ object GentrapTest { ...@@ -196,7 +196,6 @@ object GentrapTest {
copyFile("ref.fa.fai") copyFile("ref.fa.fai")
val executables = Map( val executables = Map(
"reference" -> (outputDir + File.separator + "ref.fa"),
"reference_fasta" -> (outputDir + File.separator + "ref.fa"), "reference_fasta" -> (outputDir + File.separator + "ref.fa"),
"refFlat" -> "test", "refFlat" -> "test",
"annotation_gtf" -> "test", "annotation_gtf" -> "test",
......
...@@ -109,7 +109,6 @@ object ShivaTest { ...@@ -109,7 +109,6 @@ object ShivaTest {
"cache" -> true, "cache" -> true,
"dir" -> "test", "dir" -> "test",
"vep_script" -> "test", "vep_script" -> "test",
"reference" -> (outputDir + File.separator + "ref.fa"),
"reference_fasta" -> (outputDir + File.separator + "ref.fa"), "reference_fasta" -> (outputDir + File.separator + "ref.fa"),
"gatk_jar" -> "test", "gatk_jar" -> "test",
"samtools" -> Map("exe" -> "test"), "samtools" -> Map("exe" -> "test"),
......
...@@ -112,7 +112,6 @@ object ShivaVariantcallingTest { ...@@ -112,7 +112,6 @@ object ShivaVariantcallingTest {
"cache" -> true, "cache" -> true,
"dir" -> "test", "dir" -> "test",
"vep_script" -> "test", "vep_script" -> "test",
"reference" -> (outputDir + File.separator + "ref.fa"),
"reference_fasta" -> (outputDir + File.separator + "ref.fa"), "reference_fasta" -> (outputDir + File.separator + "ref.fa"),
"gatk_jar" -> "test", "gatk_jar" -> "test",
"samtools" -> Map("exe" -> "test"), "samtools" -> Map("exe" -> "test"),
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment