Added check for REFERENCE_SEQUENCE to avoid NPE in CollectSequencingA…

[tfenne]

…rtifactMetrics

[tfenne]

@yfarjoun This has been driving me crazy! Because it inherits REFERENCE_SEQUENCE from a parent class, and it's optional up there, CSA doesn't give an error message if it's null, it just NPEs. Trivial change to add a check for it and produce a nicer error message.

[coveralls]

Coverage increased (+0.009%) to 69.265% when pulling 82ade38 on tfenne:tf_artifact_metrics_check_for_ref into 49a8cbf on broadinstitute:master.

[yfarjoun]

👍

[nh13]

@yfarjoun I think I was assigned this in error.

[yfarjoun]

Oops. Thanks.

On Aug 28, 2016 11:36 AM, "Nils Homer" notifications@github.com wrote:

@yfarjoun https://github.com/yfarjoun I think I was assigned this in
error.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#639 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACnk0iCDjguhKzjeueS5Z0-mW9n1hMARks5qkar3gaJpZM4JuAZL
.