[ 
https://issues.apache.org/jira/browse/TINKERPOP-1691?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16056147#comment-16056147
 ] 

ASF GitHub Bot commented on TINKERPOP-1691:
-------------------------------------------

GitHub user spmallette opened a pull request:

    https://github.com/apache/tinkerpop/pull/636

    TINKERPOP-1691 Changed semantics of EventStrategyProcessTest

    https://issues.apache.org/jira/browse/TINKERPOP-1691
    
    Tests no longer assume that element state is sync'd in memory. 
@pietermartin can you please review this one? I think this does what you need.
    
    Builds with `mvn clean install -DincludeNeo4j`
    
    VOTE +1

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/apache/tinkerpop TINKERPOP-1691

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/tinkerpop/pull/636.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #636
    
----
commit 2cf151939b07beac2fa967f2d422f6dc6167f329
Author: Stephen Mallette <sp...@genoprime.com>
Date:   2017-06-20T15:38:44Z

    TINKERPOP-1691 Changed semantics of EventStrategyProcessTest
    
    Tests no longer assume that element state is sync'd in memory.

----


> Some EventStrategyProcessTest assume element state is synced in memory
> ----------------------------------------------------------------------
>
>                 Key: TINKERPOP-1691
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-1691
>             Project: TinkerPop
>          Issue Type: Bug
>          Components: test-suite
>    Affects Versions: 3.2.5
>            Reporter: pieter martin
>            Priority: Minor
>
> The following 4 tests assumes that the element variable is updated in memory.
> {{EventStrategyProcessTest.shouldDetachVertexPropertyWhenRemoved}}
> {{EventStrategyProcessTest.shouldDetachPropertyOfEdgeWhenNew}}
> {{EventStrategyProcessTest.shouldDetachPropertyOfEdgeWhenRemoved}}
> {{EventStrategyProcessTest.shouldDetachVertexPropertyWhenNew}}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to