Uses build-helper-maven-plugin to reserve unused ports, which are then used in
the jetty-maven-plugin configuration and made available to test classes via the
system property "test.port".
Code cleanup: removed unused imports, added suppression for unneccessary generics warnings, etc.
jetty-maven-plugin configuration now only uses the build-helper:reserve-ports goal for integration tests.
So mvn jetty:run now operates as it did before (defaulting to port 8080, etc)
deprecated getDatastreamHistoryOld method removed from org.fcrepo.api.FedoraDatastreams
new locking settings
xml validity is good
Removes SpringJunit4ClassRunner from SanityCheckIT in fcrepo-webapp since we're
relying on jetty-maven-plugin to start/stop the container for testing.
Uses build-helper-maven-plugin:reserve-network-ports to set test.port property
for all integration tests. The property ${test.port} is made available to the tests
via maven-failsafe-plugin.
Cleans up imports. Switches from junit.framework.Assert.assertEquals (which is deprecated)
to org.junit.Assert.assertEquals.
minor: cleans up imports, formatting
Removed dependency on HTTP APIs for testing per: https://www.pivotaltracker.com/story/show/47962441
Removed unused import
Closes the InputStream after we're done with it.
Demonstrated try-with-resources
Adds null checks for apply(F), throwing an NPE if parameter is null as evidently
required by the @Nullable contract of apply() and as FindBug's
NP_PARAMETER_MUST_BE_NONNULL_BUT_MARKED_AS_NULLABLE
has gently reminded.
bump lock timeout back up, seems to make matters worse
Another attempt to resolve FindBug's
NP_PARAMETER_MUST_BE_NONNULL_BUT_MARKED_AS_NULLABLE
warning.

Also see: http://code.google.com/p/guava-libraries/issues/detail?id=920
Corrects some checkstyle, pmd issues with lack of braces on if statements and
unnecessary use of toString() on Strings
Cleans up some more FindBugs warnings
minor: tweaks to test clearing PMD warnings