Removing symbolic test resource links, read from test classpath instead

[heuermh]

See
BD2KGenomics/cgl-docker-lib#72 (comment)

[AmplabJenkins]

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/1070/

Build result: FAILURE

GitHub pull request #927 of commit e35be75 automatically merged.Notifying endpoint 'HTTP:https://webhooks.gitter.im/e/ac8bb6e9f53357bc8aa8'[EnvInject] - Loading node environment variables.Building remotely on amp-jenkins-worker-05 (centos spark-test) in workspace /home/jenkins/workspace/ADAM-prb > git rev-parse --is-inside-work-tree # timeout=10Fetching changes from the remote Git repository > git config remote.origin.url https://github.com/bigdatagenomics/adam.git # timeout=10Fetching upstream changes from https://github.com/bigdatagenomics/adam.git > git --version # timeout=10 > git fetch --tags --progress https://github.com/bigdatagenomics/adam.git +refs/pull/:refs/remotes/origin/pr/ > git rev-parse origin/pr/927/merge^{commit} # timeout=10 > git branch -a --contains 28b97df659ef197bb80819f27b847b827dc3aef5 # timeout=10 > git rev-parse remotes/origin/pr/927/merge^{commit} # timeout=10Checking out Revision 28b97df659ef197bb80819f27b847b827dc3aef5 (origin/pr/927/merge) > git config core.sparsecheckout # timeout=10 > git checkout -f 28b97df659ef197bb80819f27b847b827dc3aef5First time build. Skipping changelog.Triggering ADAM-prb ? 2.6.0,2.11,1.4.1,centosTriggering ADAM-prb ? 2.6.0,2.10,1.4.1,centosTouchstone configurations resulted in FAILURE, so aborting...Notifying endpoint 'HTTP:https://webhooks.gitter.im/e/ac8bb6e9f53357bc8aa8'
Test FAILed.

[fnothaft]

LGTM once build is passing.

[heuermh]

Sorry, I was foiled by classpath caching locally. Thank you, Jenkins.

It appears that ADAMContext.loadBam can't handle classpath jar file paths. Should I just copy the resource to a temporary file first?

[fnothaft]

It appears that ADAMContext.loadBam can't handle classpath jar file paths. Should I just copy the resource to a temporary file first?

Ah, yeah, I think I've run into that before... I've done exactly what you're talking about, although that was for a special case. If we'd need to do something hacky like that just to pass unit tests, this approach may not be worth doing, alas... I defer to you. Sorry—I thought this approach would work better...

[AmplabJenkins]

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/1071/
Test PASSed.

[fnothaft]

Thanks @heuermh ! Committed as efe4dfa.