[ https://issues.apache.org/jira/browse/TINKERPOP-1784?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16257031#comment-16257031 ]
ASF GitHub Bot commented on TINKERPOP-1784: ------------------------------------------- Github user jorgebay commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/747#discussion_r151696373 --- Diff: gremlin-test/features/sideEffect/Sack.feature --- @@ -0,0 +1,71 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license agreements. See the NOTICE file +# distributed with this work for additional information +# regarding copyright ownership. The ASF licenses this file +# to you under the Apache License, Version 2.0 (the +# "License"); you may not use this file except in compliance +# with the License. You may obtain a copy of the License at +# +# http://www.apache.org/licenses/LICENSE-2.0 +# +# Unless required by applicable law or agreed to in writing, +# software distributed under the License is distributed on an +# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY +# KIND, either express or implied. See the License for the +# specific language governing permissions and limitations +# under the License. + +Feature: Step - sack() + + Scenario: g_withSackXhelloX_V_outE_sackXassignX_byXlabelX_inV_sack + Given the modern graph + And the traversal of + """ + g.withSack("hello").V().outE().sack(Operator.assign).by(T.label).inV().sack() + """ + When iterated to list + Then the result should be unordered + | result | + | created | + | knows | + | knows | + | created | + | created | + | created | + + Scenario: g_withSackX0X_V_outE_sackXsumX_byXweightX_inV_sack_sum + Given the modern graph + And the traversal of + """ + g.withSack(0.0).V().outE().sack(Operator.sum).by("weight").inV().sack().sum() + """ + When iterated to list + Then the result should be unordered + | result | + | d[3.5].d | --- End diff -- The type of the result is `BigDecimal` ```bash gremlin> g.withSack(0.0).V().outE().sack(Operator.sum).by("weight").inV().sack().sum().next().getClass() ==>class java.math.BigDecimal ``` We should introduce another suffix for decimals. > Gremlin Language Test Suite > --------------------------- > > Key: TINKERPOP-1784 > URL: https://issues.apache.org/jira/browse/TINKERPOP-1784 > Project: TinkerPop > Issue Type: Improvement > Components: test-suite > Affects Versions: 3.2.6 > Reporter: stephen mallette > Assignee: stephen mallette > > Provide for a more language agnostic test framework for the Gremlin Language > so that we can properly test GLVs. The current test suite is completely tied > to the JVM. -- This message was sent by Atlassian JIRA (v6.4.14#64029)