From 4108a294f76e051f9c14cc8884e49a90a46411c9 Mon Sep 17 00:00:00 2001 From: Mark DePristo Date: Mon, 17 Oct 2011 13:58:46 -0400 Subject: [PATCH] Better error message when a RodBinding file doesn't exist --- .../sting/commandline/ArgumentTypeDescriptor.java | 2 +- .../broadinstitute/sting/utils/exceptions/UserException.java | 4 ++++ 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/public/java/src/org/broadinstitute/sting/commandline/ArgumentTypeDescriptor.java b/public/java/src/org/broadinstitute/sting/commandline/ArgumentTypeDescriptor.java index 5fff8f609..94157963f 100644 --- a/public/java/src/org/broadinstitute/sting/commandline/ArgumentTypeDescriptor.java +++ b/public/java/src/org/broadinstitute/sting/commandline/ArgumentTypeDescriptor.java @@ -380,7 +380,7 @@ class RodBindingArgumentTypeDescriptor extends ArgumentTypeDescriptor { if ( tribbleType == null ) if ( ! file.canRead() | ! file.isFile() ) { - throw new UserException.BadArgumentValue(name, "Couldn't read file to determine type: " + file); + throw new UserException.CouldNotReadInputFile(file, "file does exist or couldn't be read"); } else { throw new UserException.CommandLineException( String.format("No tribble type was provided on the command line and the type of the file could not be determined dynamically. " + diff --git a/public/java/src/org/broadinstitute/sting/utils/exceptions/UserException.java b/public/java/src/org/broadinstitute/sting/utils/exceptions/UserException.java index 274c64f42..9d131ae0c 100755 --- a/public/java/src/org/broadinstitute/sting/utils/exceptions/UserException.java +++ b/public/java/src/org/broadinstitute/sting/utils/exceptions/UserException.java @@ -111,6 +111,10 @@ public class UserException extends ReviewedStingException { super(String.format("Couldn't read file because %s caused by %s", message, e.getMessage())); } + public CouldNotReadInputFile(File file) { + super(String.format("Couldn't read file %s", file.getAbsolutePath())); + } + public CouldNotReadInputFile(File file, String message) { super(String.format("Couldn't read file %s because %s", file.getAbsolutePath(), message)); }