Integrate NVHTests and NVSTests in BlockchainSanity

[kushti]

No description provided.

[catena2w]

Style fixes required

[catena2w]

We use one line codestyle, like
} finally {

[ceilican]

See #118 . I don't have push access to this repo. That's why I couldn't push the fix directly into this pull request.

[ceilican]

This is the test that failed:

[info] - NodeViewHolder: forking - switching *** FAILED ***
[info]   java.lang.NullPointerException:
[info]   at scorex.testkit.properties.NodeViewHolderTests.$anonfun$$init$$39(NodeViewHolderTests.scala:239)
[info]   at scorex.testkit.properties.NodeViewHolderTests.$anonfun$$init$$39$adapted(NodeViewHolderTests.scala:211)
[info]   at scorex.testkit.properties.NodeViewHolderTests.withFixture(NodeViewHolderTests.scala:47)
[info]   at scorex.testkit.properties.NodeViewHolderTests.$anonfun$$init$$38(NodeViewHolderTests.scala:211)
[info]   at scala.runtime.java8.JFunction0$mcV$sp.apply(JFunction0$mcV$sp.java:12)
[info]   at org.scalatest.OutcomeOf.outcomeOf(OutcomeOf.scala:85)
[info]   at org.scalatest.OutcomeOf.outcomeOf$(OutcomeOf.scala:83)
[info]   at org.scalatest.OutcomeOf$.outcomeOf(OutcomeOf.scala:104)
[info]   at org.scalatest.Transformer.apply(Transformer.scala:22)
[info]   at org.scalatest.Transformer.apply(Transformer.scala:20)
[info]   ...

[coveralls]

Changes Unknown when pulling 19d9f9e on more-props into ** on master**.

[ceilican]

Ok. I re-executed the failing build in Travis, and it succeeded this time...

[coveralls]

Coverage increased (+0.06%) to 45.681% when pulling f8ac41a on more-props into f66dd57 on master.

[ceilican]

@catena2w I fixed the style, as you requested.

@kushti I fresolved the conflict.

@catena2w @kushti, I think this PR can be accepted now.

[ceilican]

I removed the obsolete failing test.

There is another test that sometimes fails, but this is not caused by this MR. The problem is discussed here (#119).

[coveralls]

Coverage remained the same at 45.532% when pulling 25221be on more-props into a8cf1ef on master.