Upgrade to Picard 2.23.3 (#6717)

* Upgrade to picard 2.23.3
broadinstitute · Aug 4, 2020 · 5a14c7c · 5a14c7c
1 parent 6a0853d
commit 5a14c7c
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/build.gradle b/build.gradle
@@ -62,7 +62,7 @@ repositories {
 }
 
 final htsjdkVersion = System.getProperty('htsjdk.version','2.23.0')
-final picardVersion = System.getProperty('picard.version','2.22.8')
+final picardVersion = System.getProperty('picard.version','2.23.3')
 final barclayVersion = System.getProperty('barclay.version','3.0.0')
 final sparkVersion = System.getProperty('spark.version', '2.4.5')
 final scalaVersion = System.getProperty('scala.version', '2.11')

diff --git a/src/test/java/org/broadinstitute/hellbender/MainTest.java b/src/test/java/org/broadinstitute/hellbender/MainTest.java
@@ -118,9 +118,7 @@ public void testNonZeroPicardReturnValue() {
     public void testEnsureShortDescriptionsAreShort() {
         // Test each command line tool to ensure that one line summaries don't exceed the maximum allowable length
 
-        //TODO: replace this constant with the Picard-defined length when its available in GATK
-        //final int MAX_ALLOWABLE_ONE_LINE_SUMMARY_LENGTH = picard.cmdline.CommandLineProgram.MAX_ALLOWABLE_ONE_LINE_SUMMARY_LENGTH;
-        final int MAX_ALLOWABLE_ONE_LINE_SUMMARY_LENGTH = 120;
+        final int MAX_ALLOWABLE_ONE_LINE_SUMMARY_LENGTH = picard.cmdline.CommandLineProgram.MAX_ALLOWABLE_ONE_LINE_SUMMARY_LENGTH;
 
         // Picard tools are validated independently by a similar test in Picard, and also use a CommandLineProgram
         // class from a different package, so bypass tools from that package for this test