[
https://issues.apache.org/jira/browse/TINKERPOP3-632?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14494730#comment-14494730
]
ASF GitHub Bot commented on TINKERPOP3-632:
-------------------------------------------
Github user mhfrantz commented on the pull request:
https://github.com/apache/incubator-tinkerpop/pull/53#issuecomment-93035115
OK, this is ready for review.
I tried for a bit to include the parameterized tests that do not descend
from AbstractGremlinSuite. However, that proved challenging because the logic
for parsing gremlin_tests is part of the gremlin-test module, but we have
parameterized unit tests in gremlin-core (which does not depend on
gremlin-test). So, the "run only this test" mode works for tests that derive
from gremlin-test harnesses.
> Env gremlin.tests should work on all test suites
> ------------------------------------------------
>
> Key: TINKERPOP3-632
> URL: https://issues.apache.org/jira/browse/TINKERPOP3-632
> Project: TinkerPop 3
> Issue Type: Improvement
> Components: test-suite
> Reporter: Matt Frantz
> Priority: Minor
>
> There are some suites that bypass the gremlin.tests env var check in
> ProcessComputerSuite and ProcessStandardSuite by supplying their own list of
> tests. The env var check should be done in AbstractGremlinSuite instead.
> Also, the root README describes using a Maven system property to specify the
> tests, but it has to be an environment variable.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
