[
https://issues.apache.org/jira/browse/TINKERPOP-3035?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17840334#comment-17840334
]
ASF GitHub Bot commented on TINKERPOP-3035:
-------------------------------------------
FlorianHockmann commented on PR #2565:
URL: https://github.com/apache/tinkerpop/pull/2565#issuecomment-2074375930
> nit: no test coverage
Yes, I wanted to add a test for this, but the problem is that it's
manipulating data in the graph. I don't know how to test this with our current
test setup without impacting other tests. If we could start a container with
Gremlin Server for just this test, then we could easily test it. But our
current setup uses the same container for all tests with the same graph. If we
write to this graph in one test, then it will impact other tests.
Another option would be to write a unit test which only checks the generated
Bytecode, but I think such tests have little value, especially when we want to
get rid of Bytecode altogether in a future release.
> Add explicit property(IDictionary) for .NET
> -------------------------------------------
>
> Key: TINKERPOP-3035
> URL: https://issues.apache.org/jira/browse/TINKERPOP-3035
> Project: TinkerPop
> Issue Type: Bug
> Components: dotnet
> Affects Versions: 3.6.6
> Reporter: Stephen Mallette
> Priority: Minor
>
> There is no {{property(IDictionary<object,object>)}} method for .NET to
> accompany the {{property(Map)}} step. It works because it probably piggybacks
> on {{property(object?, object?, objects[]?)}} which has all nullable
> arguments. The explicit overload with the {{IDictionary}} should be added.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
