Repository: tinkerpop Updated Branches: refs/heads/tp32 f84f454ec -> 70cf34b2c
TINKERPOP-1610 Deprecated Groovy test suites Graph Providers no longer need to implement these going forward. Project: http://git-wip-us.apache.org/repos/asf/tinkerpop/repo Commit: http://git-wip-us.apache.org/repos/asf/tinkerpop/commit/168d5b37 Tree: http://git-wip-us.apache.org/repos/asf/tinkerpop/tree/168d5b37 Diff: http://git-wip-us.apache.org/repos/asf/tinkerpop/diff/168d5b37 Branch: refs/heads/tp32 Commit: 168d5b37e97b15afc1c9c599035f879e92d6c1c1 Parents: 97cc07d Author: Stephen Mallette <sp...@genoprime.com> Authored: Fri Jan 20 14:59:25 2017 -0500 Committer: Stephen Mallette <sp...@genoprime.com> Committed: Fri Jan 20 14:59:25 2017 -0500 ---------------------------------------------------------------------- CHANGELOG.asciidoc | 1 + .../upgrade/release-3.2.x-incubating.asciidoc | 17 +++++++++++++++-- .../tinkerpop/gremlin/structure/Graph.java | 20 ++++++++++++++++++++ .../groovy/GroovyEnvironmentIntegrateSuite.java | 2 ++ .../gremlin/groovy/GroovyEnvironmentSuite.java | 2 ++ .../process/GroovyProcessComputerSuite.java | 2 ++ .../process/GroovyProcessStandardSuite.java | 2 ++ 7 files changed, 44 insertions(+), 2 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/168d5b37/CHANGELOG.asciidoc ---------------------------------------------------------------------- diff --git a/CHANGELOG.asciidoc b/CHANGELOG.asciidoc index 4f3f9ce..21486e9 100644 --- a/CHANGELOG.asciidoc +++ b/CHANGELOG.asciidoc @@ -27,6 +27,7 @@ TinkerPop 3.2.4 (Release Date: NOT OFFICIALLY RELEASED YET) ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ * SASL negotiation supports both a byte array and Base64 encoded bytes as a string for authentication to Gremlin Server. +* Deprecated all test suites in `gremlin-groovy-test` - Graph Providers no longer need to implement these. * Deprecated `TinkerIoRegistry` replacing it with the more consistently named `TinkerIoRegistryV1d0`. * Made error messaging more consistent during result iteration timeouts in Gremlin Server. * `PathRetractionStrategy` does not add a `NoOpBarrierStep` to the end of local children as its wasted computation in 99% of traversals. http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/168d5b37/docs/src/upgrade/release-3.2.x-incubating.asciidoc ---------------------------------------------------------------------- diff --git a/docs/src/upgrade/release-3.2.x-incubating.asciidoc b/docs/src/upgrade/release-3.2.x-incubating.asciidoc index b478b96..7b5d638 100644 --- a/docs/src/upgrade/release-3.2.x-incubating.asciidoc +++ b/docs/src/upgrade/release-3.2.x-incubating.asciidoc @@ -199,8 +199,8 @@ multi-properties have more flexibility in describing their graph capabilities. See: link:https://issues.apache.org/jira/browse/TINKERPOP-919[TINKERPOP-919] -Deprecated Performance OptIn -++++++++++++++++++++++++++++ +Deprecated OptIn +++++++++++++++++ In 3.2.1, all `junit-benchmark` performance tests were deprecated. At that time, the `OptIn` representations of these tests should have been deprecated as well, but they were not. That omission has been remedied now. Specifically, the @@ -210,6 +210,19 @@ following fields were deprecated: * `OptIn.SUITE_PROCESS_PERFORMANCE` * `OptIn.SUITE_STRUCTURE_PERFORMANCE` +As of 3.2.4, the following test suites were also deprecated: + +* `OptIn.SUITE_GROOVY_PROCESS_STANDARD` +* `OptIn.SUITE_GROOVY_PROCESS_COMPUTER` +* `OptIn.SUITE_GROOVY_ENVIRONMENT` +* `OptIn.SUITE_GROOVY_ENVIRONMENT_INTEGRATE` + +Future testing of `gremlin-groovy` (and language variants in general) will be handled differently and will not require +a Graph Provider to validate its operations with it. Graph Providers may now choose to remove these tests from their +test suites, which should reduce the testing burden. + +See: link:https://issues.apache.org/jira/browse/TINKERPOP-1610[TINKERPOP-1610] + Deprecated getInstance() ++++++++++++++++++++++++ http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/168d5b37/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/Graph.java ---------------------------------------------------------------------- diff --git a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/Graph.java b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/Graph.java index 255fbca..1027e9b 100644 --- a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/Graph.java +++ b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/Graph.java @@ -1226,9 +1226,29 @@ public interface Graph extends AutoCloseable, Host { */ @Deprecated public static String SUITE_PROCESS_PERFORMANCE = "org.apache.tinkerpop.gremlin.process.ProcessPerformanceSuite"; + + /** + * @deprecated As of release 3.2.4, not replaced as a test suite that Graph Providers need to implement. + */ + @Deprecated public static String SUITE_GROOVY_PROCESS_STANDARD = "org.apache.tinkerpop.gremlin.process.GroovyProcessStandardSuite"; + + /** + * @deprecated As of release 3.2.4, not replaced as a test suite that Graph Providers need to implement. + */ + @Deprecated public static String SUITE_GROOVY_PROCESS_COMPUTER = "org.apache.tinkerpop.gremlin.process.GroovyProcessComputerSuite"; + + /** + * @deprecated As of release 3.2.4, not replaced as a test suite that Graph Providers need to implement. + */ + @Deprecated public static String SUITE_GROOVY_ENVIRONMENT = "org.apache.tinkerpop.gremlin.groovy.GroovyEnvironmentSuite"; + + /** + * @deprecated As of release 3.2.4, not replaced as a test suite that Graph Providers need to implement. + */ + @Deprecated public static String SUITE_GROOVY_ENVIRONMENT_INTEGRATE = "org.apache.tinkerpop.gremlin.groovy.GroovyEnvironmentIntegrateSuite"; /** http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/168d5b37/gremlin-groovy-test/src/main/java/org/apache/tinkerpop/gremlin/groovy/GroovyEnvironmentIntegrateSuite.java ---------------------------------------------------------------------- diff --git a/gremlin-groovy-test/src/main/java/org/apache/tinkerpop/gremlin/groovy/GroovyEnvironmentIntegrateSuite.java b/gremlin-groovy-test/src/main/java/org/apache/tinkerpop/gremlin/groovy/GroovyEnvironmentIntegrateSuite.java index b6cd29c..ecd481f 100644 --- a/gremlin-groovy-test/src/main/java/org/apache/tinkerpop/gremlin/groovy/GroovyEnvironmentIntegrateSuite.java +++ b/gremlin-groovy-test/src/main/java/org/apache/tinkerpop/gremlin/groovy/GroovyEnvironmentIntegrateSuite.java @@ -46,7 +46,9 @@ import java.util.stream.Stream; * For more information on the usage of this suite, please see {@link StructureStandardSuite}. * * @author Stephen Mallette (http://stephen.genoprime.com) + * @deprecated As of release 3.2.4, not replaced as a test suite that Graph Providers need to implement. */ +@Deprecated public class GroovyEnvironmentIntegrateSuite extends AbstractGremlinSuite { /** http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/168d5b37/gremlin-groovy-test/src/main/java/org/apache/tinkerpop/gremlin/groovy/GroovyEnvironmentSuite.java ---------------------------------------------------------------------- diff --git a/gremlin-groovy-test/src/main/java/org/apache/tinkerpop/gremlin/groovy/GroovyEnvironmentSuite.java b/gremlin-groovy-test/src/main/java/org/apache/tinkerpop/gremlin/groovy/GroovyEnvironmentSuite.java index be89f65..8a4e138 100644 --- a/gremlin-groovy-test/src/main/java/org/apache/tinkerpop/gremlin/groovy/GroovyEnvironmentSuite.java +++ b/gremlin-groovy-test/src/main/java/org/apache/tinkerpop/gremlin/groovy/GroovyEnvironmentSuite.java @@ -47,7 +47,9 @@ import org.junit.runners.model.RunnerBuilder; * For more information on the usage of this suite, please see {@link StructureStandardSuite}. * * @author Stephen Mallette (http://stephen.genoprime.com) + * @deprecated As of release 3.2.4, not replaced as a test suite that Graph Providers need to implement. */ +@Deprecated public class GroovyEnvironmentSuite extends AbstractGremlinSuite { /** http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/168d5b37/gremlin-groovy-test/src/main/java/org/apache/tinkerpop/gremlin/process/GroovyProcessComputerSuite.java ---------------------------------------------------------------------- diff --git a/gremlin-groovy-test/src/main/java/org/apache/tinkerpop/gremlin/process/GroovyProcessComputerSuite.java b/gremlin-groovy-test/src/main/java/org/apache/tinkerpop/gremlin/process/GroovyProcessComputerSuite.java index e411c1e..d408da6 100644 --- a/gremlin-groovy-test/src/main/java/org/apache/tinkerpop/gremlin/process/GroovyProcessComputerSuite.java +++ b/gremlin-groovy-test/src/main/java/org/apache/tinkerpop/gremlin/process/GroovyProcessComputerSuite.java @@ -97,7 +97,9 @@ import org.junit.runners.model.RunnerBuilder; * For more information on the usage of this suite, please see {@link StructureStandardSuite}. * * @author Stephen Mallette (http://stephen.genoprime.com) + * @deprecated As of release 3.2.4, not replaced as a test suite that Graph Providers need to implement. */ +@Deprecated public class GroovyProcessComputerSuite extends ProcessComputerSuite { /** http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/168d5b37/gremlin-groovy-test/src/main/java/org/apache/tinkerpop/gremlin/process/GroovyProcessStandardSuite.java ---------------------------------------------------------------------- diff --git a/gremlin-groovy-test/src/main/java/org/apache/tinkerpop/gremlin/process/GroovyProcessStandardSuite.java b/gremlin-groovy-test/src/main/java/org/apache/tinkerpop/gremlin/process/GroovyProcessStandardSuite.java index edd06f3..45aea91 100644 --- a/gremlin-groovy-test/src/main/java/org/apache/tinkerpop/gremlin/process/GroovyProcessStandardSuite.java +++ b/gremlin-groovy-test/src/main/java/org/apache/tinkerpop/gremlin/process/GroovyProcessStandardSuite.java @@ -94,7 +94,9 @@ import org.junit.runners.model.RunnerBuilder; * For more information on the usage of this suite, please see {@link StructureStandardSuite}. * * @author Stephen Mallette (http://stephen.genoprime.com) + * @deprecated As of release 3.2.4, not replaced as a test suite that Graph Providers need to implement. */ +@Deprecated public class GroovyProcessStandardSuite extends ProcessStandardSuite { /**