upgrade scalatest

supports wildcard single-{test,suite} running, e.g.:

mvn -Dsuites='*PileupSuite' test
mvn -Dsuites='*PileupSuite qualities' test
remove redundant 'val' keyword
use .count instead of .filter.length

intellij suggests this; seems cleaner/simpler
add EoF newlines
break long-line of inheritance declarations
use nonEmpty instead of !.empty
simplify match statement, avoid return statement

IIRC, return statements are implemented using exceptions, and are
relatively unperformant as a result; also stylistically at odds with
functional code flow
group loci statements to match other callers

in some world we may want to reuse this boilerplate across VCs
fix variable shadowing

intellij doesn't like shadowed var names, seems reasonable
remove redundant/shadowed import
use .sort.reverse instead of .sort(-1 * _)
remove unnecessary _'s
add names to some cryptic arguments
collapse some .view.count()'s to just .count()
use ".exists()" instead of ".count() > 0"
Merge pull request #142 from hammerlab/scalatest

upgrade scalatest
Merge pull request #143 from hammerlab/style

some random style nits
new genotype format with more information
filter correlated variants in second pass after variant calling
optimizations for distributed util and pileupcalling; changes to genotype to seq and use everywhere; add kryo serializers