Hi,

Bob Briody is doing some work on profile()-step and said:

        "SparkGraphComputerProcessIntegrateTest only takes about 30 seconds to 
execute. Why is that integration testing?"

I thought about it, and I agree. We already have various Spark tests that are 
not integration-based. Would people object if I made both:

        SparkGraphComputerProcessIntegrateTest
        SparkGraphComputerGroovyProcessIntegrateTest

---->

        SparkGraphComputerProcessTest
        SparkGraphComputerGroovyProcessTest

This will add about 1 minute to the "mvn clean install" but will be a lot safer 
for people as lots of Serialization/VertexProgram Configuration/etc. issues get 
exposed here that you don't see with TinkerGraphComputerProcessTest.  
Definitely beneficial to people trying to submit PRs.

Thoughts?,
Marko.

http://markorodriguez.com

Reply via email to