catch the correct type of error
clean up if-else block
ensure that if-else block covers all cases
test coverage for resolve_plan_attributes subroutine
add test for get_attribute subroutine
Use status_msg instead of debug_msg in (un)archive

Maybe now it will be clear why some allocations silently fail to
archive...
make sure that we can call rethrow
pull error handling logic into separate subroutine
enhance error checking around sam-dump
check result of chdir() in _dump_bam_from_sra
remove file name decision from dump_aligned_bam()
Catch exceptions in SraToBam::execute()

Especially exceptions coming out of validation
I think this should have changed in an earlier commit
Handle case where there are no unalinged reads
Break long line into two
Extract dump_unaligned_fastq()
enhance error reporting in dump_unaligned_fastq()
Move bam size checking outside of dump_aligned_bam()
Extract do_shellcmd() and use it
rename do_shellcmd to do_shellcmd_with_stdout