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 2018/10/08 20:44:53 UTC

[tinkerpop] branch master updated: Added an "empty" OptionsStrategy CTR

This is an automated email from the ASF dual-hosted git repository.

spmallette pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/tinkerpop.git


The following commit(s) were added to refs/heads/master by this push:
     new ecc7dca  Added an "empty" OptionsStrategy CTR
ecc7dca is described below

commit ecc7dca2ff6885389b962da0b478ac52b5ebd94b
Author: Stephen Mallette <sp...@genoprime.com>
AuthorDate: Mon Oct 8 16:44:27 2018 -0400

    Added an "empty" OptionsStrategy CTR
---
 .../process/traversal/strategy/decoration/OptionsStrategy.java      | 6 ++++++
 .../process/traversal/strategy/decoration/OptionsStrategyTest.java  | 5 +++++
 2 files changed, 11 insertions(+)

diff --git a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/strategy/decoration/OptionsStrategy.java b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/strategy/decoration/OptionsStrategy.java
index c66fa01..fd64d7f 100644
--- a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/strategy/decoration/OptionsStrategy.java
+++ b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/strategy/decoration/OptionsStrategy.java
@@ -37,6 +37,12 @@ import java.util.Map;
  * @author Stephen Mallette (http://stephen.genoprime.com)
  */
 public class OptionsStrategy extends AbstractTraversalStrategy<TraversalStrategy.DecorationStrategy> implements TraversalStrategy.DecorationStrategy {
+
+    /**
+     * An empty {@code OptionsStrategy} with no configuration values inside.
+     */
+    public static final OptionsStrategy EMPTY = OptionsStrategy.build().create();
+
     private final Map<String, Object> options;
 
     private OptionsStrategy(final Builder builder) {
diff --git a/gremlin-core/src/test/java/org/apache/tinkerpop/gremlin/process/traversal/strategy/decoration/OptionsStrategyTest.java b/gremlin-core/src/test/java/org/apache/tinkerpop/gremlin/process/traversal/strategy/decoration/OptionsStrategyTest.java
index 064289a..c25c39f 100644
--- a/gremlin-core/src/test/java/org/apache/tinkerpop/gremlin/process/traversal/strategy/decoration/OptionsStrategyTest.java
+++ b/gremlin-core/src/test/java/org/apache/tinkerpop/gremlin/process/traversal/strategy/decoration/OptionsStrategyTest.java
@@ -35,4 +35,9 @@ public class OptionsStrategyTest {
         assertEquals("test", strategy.getOptions().get("a"));
         assertThat(strategy.getOptions().get("b"), is(true));
     }
+
+    @Test
+    public void shouldHaveNoEntriesInEmptyOptionsStrategy() {
+        assertThat(OptionsStrategy.EMPTY.getOptions().isEmpty(), is(true));
+    }
 }