You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tinkerpop.apache.org by sp...@apache.org on 2017/03/07 12:01:41 UTC
[1/2] tinkerpop git commit: Removed references to gremlin-groovy-test
as it is deprecated
Repository: tinkerpop
Updated Branches:
refs/heads/master ad51fb24e -> 6596900bf
Removed references to gremlin-groovy-test as it is deprecated
There really isn't any point to continuing to encourage providers to implement gremlin-groovy-test on the 3.2.x line as it is deprecated. It has already been removed in 3.3.x. CTR
Project: http://git-wip-us.apache.org/repos/asf/tinkerpop/repo
Commit: http://git-wip-us.apache.org/repos/asf/tinkerpop/commit/591a65a3
Tree: http://git-wip-us.apache.org/repos/asf/tinkerpop/tree/591a65a3
Diff: http://git-wip-us.apache.org/repos/asf/tinkerpop/diff/591a65a3
Branch: refs/heads/master
Commit: 591a65a3ce3a06ac0919f77117622fa44a0456b7
Parents: a8acd67
Author: Stephen Mallette <sp...@genoprime.com>
Authored: Tue Mar 7 07:00:26 2017 -0500
Committer: Stephen Mallette <sp...@genoprime.com>
Committed: Tue Mar 7 07:00:26 2017 -0500
----------------------------------------------------------------------
docs/src/dev/provider/index.asciidoc | 33 +++++++------------------------
1 file changed, 7 insertions(+), 26 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/591a65a3/docs/src/dev/provider/index.asciidoc
----------------------------------------------------------------------
diff --git a/docs/src/dev/provider/index.asciidoc b/docs/src/dev/provider/index.asciidoc
index 71d071f..3988ee9 100644
--- a/docs/src/dev/provider/index.asciidoc
+++ b/docs/src/dev/provider/index.asciidoc
@@ -458,20 +458,16 @@ Validating with Gremlin-Test
image:gremlin-edumacated.png[width=225]
[source,xml]
+----
<dependency>
<groupId>org.apache.tinkerpop</groupId>
<artifactId>gremlin-test</artifactId>
<version>x.y.z</version>
</dependency>
-<dependency>
- <groupId>org.apache.tinkerpop</groupId>
- <artifactId>gremlin-groovy-test</artifactId>
- <version>x.y.z</version>
-</dependency>
+----
-The operational semantics of any OLTP or OLAP implementation are validated by `gremlin-test` and functional
-interoperability with the Groovy environment is ensured by `gremlin-groovy-test`. To implement these tests, provide
-test case implementations as shown below, where `XXX` below denotes the name of the graph implementation (e.g.
+The operational semantics of any OLTP or OLAP implementation are validated by `gremlin-test`. To implement these tests,
+provide test case implementations as shown below, where `XXX` below denotes the name of the graph implementation (e.g.
TinkerGraph, Neo4jGraph, HadoopGraph, etc.).
[source,java]
@@ -489,22 +485,11 @@ public class XXXProcessComputerTest {}
@RunWith(ProcessStandardSuite.class)
@GraphProviderClass(provider = XXXGraphProvider.class, graph = XXXGraph.class)
public class XXXProcessStandardTest {}
-
-@RunWith(GroovyEnvironmentSuite.class)
-@GraphProviderClass(provider = XXXProvider.class, graph = TinkerGraph.class)
-public class XXXGroovyEnvironmentTest {}
-
-@RunWith(GroovyProcessStandardSuite.class)
-@GraphProviderClass(provider = XXXGraphProvider.class, graph = TinkerGraph.class)
-public class XXXGroovyProcessStandardTest {}
-
-@RunWith(GroovyProcessComputerSuite.class)
-@GraphProviderClass(provider = XXXGraphComputerProvider.class, graph = TinkerGraph.class)
-public class XXXGroovyProcessComputerTest {}
----
The above set of tests represent the minimum test suite set to implement. There are other "integration" and
-"performance" tests that should be considered optional. Implementing those tests requires the same pattern as shown above.
+"performance" tests that should be considered optional. Implementing those tests requires the same pattern as shown
+above.
IMPORTANT: It is as important to look at "ignored" tests as it is to look at ones that fail. The `gremlin-test`
suite utilizes the `Feature` implementation exposed by the `Graph` to determine which tests to execute. If a test
@@ -526,9 +511,6 @@ annotation. See the `TinkerGraph` implementation below as an example:
@Graph.OptIn(Graph.OptIn.SUITE_STRUCTURE_STANDARD)
@Graph.OptIn(Graph.OptIn.SUITE_PROCESS_STANDARD)
@Graph.OptIn(Graph.OptIn.SUITE_PROCESS_COMPUTER)
-@Graph.OptIn(Graph.OptIn.SUITE_GROOVY_PROCESS_STANDARD)
-@Graph.OptIn(Graph.OptIn.SUITE_GROOVY_PROCESS_COMPUTER)
-@Graph.OptIn(Graph.OptIn.SUITE_GROOVY_ENVIRONMENT)
public class TinkerGraph implements Graph {
----
@@ -567,8 +549,7 @@ public class HadoopGraph implements Graph {
The above examples show how to ignore individual tests. It is also possible to:
* Ignore an entire test case (i.e. all the methods within the test) by setting the `method` to "*".
-* Ignore a "base" test class such that test that extend from those classes will all be ignored. This style of
-ignoring is useful for Gremlin "process" tests that have bases classes that are extended by various Gremlin flavors (e.g. groovy).
+* Ignore a "base" test class such that test that extend from those classes will all be ignored.
* Ignore a `GraphComputer` test based on the type of `GraphComputer` being used. Specify the "computer" attribute on
the `OptOut` (which is an array specification) which should have a value of the `GraphComputer` implementation class
that should ignore that test. This attribute should be left empty for "standard" execution and by default all
[2/2] tinkerpop git commit: Merge branch 'tp32'
Posted by sp...@apache.org.
Merge branch 'tp32'
Project: http://git-wip-us.apache.org/repos/asf/tinkerpop/repo
Commit: http://git-wip-us.apache.org/repos/asf/tinkerpop/commit/6596900b
Tree: http://git-wip-us.apache.org/repos/asf/tinkerpop/tree/6596900b
Diff: http://git-wip-us.apache.org/repos/asf/tinkerpop/diff/6596900b
Branch: refs/heads/master
Commit: 6596900bf276b088c69f58b927204c9edc48ded1
Parents: ad51fb2 591a65a
Author: Stephen Mallette <sp...@genoprime.com>
Authored: Tue Mar 7 07:01:27 2017 -0500
Committer: Stephen Mallette <sp...@genoprime.com>
Committed: Tue Mar 7 07:01:27 2017 -0500
----------------------------------------------------------------------
docs/src/dev/provider/index.asciidoc | 33 +++++++------------------------
1 file changed, 7 insertions(+), 26 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/6596900b/docs/src/dev/provider/index.asciidoc
----------------------------------------------------------------------