Skip to content
This repository was archived by the owner on Sep 17, 2025. It is now read-only.

Commit ad93347

Browse files
committed
tests fixes
1 parent cf51f48 commit ad93347

File tree

2 files changed

+9
-6
lines changed

2 files changed

+9
-6
lines changed

src/main/java/com/arangodb/tinkerpop/gremlin/client/ArangoDBGraphClient.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -276,6 +276,7 @@ public void updateGraphVariables(ArangoDBGraphVariables document) {
276276
* @return ArangoDBBaseQuery the query object
277277
*/
278278

279+
// FIXME: use multi-docs API
279280
public ArangoIterable<VertexData> getGraphVertices(final List<String> ids) {
280281
logger.debug("Get all {} graph vertices, filtered by ids: {}", graph.name(), ids);
281282
List<String> prefixedColNames = graph.vertexCollections().stream().map(graph::getPrefixedCollectioName).collect(Collectors.toList());
@@ -288,6 +289,7 @@ public ArangoIterable<VertexData> getGraphVertices(final List<String> ids) {
288289
* @param ids the ids to match
289290
* @return ArangoDBBaseQuery the query object
290291
*/
292+
// FIXME: use multi-docs API
291293
public ArangoIterable<EdgeData> getGraphEdges(List<String> ids) {
292294
logger.debug("Get all {} graph edges, filtered by ids: {}", graph.name(), ids);
293295
List<String> prefixedColNames = graph.edgeCollections().stream().map(graph::getPrefixedCollectioName).collect(Collectors.toList());
@@ -305,6 +307,7 @@ private <V> ArangoIterable<V> getGraphDocuments(List<String> ids, List<String> p
305307
}
306308
} else {
307309
List<String> prunedIds = ids.stream()
310+
.map(graph::getPrefixedCollectioName)
308311
.filter(it -> prefixedColNames.contains(ArangoDBUtil.extractCollection(it)))
309312
.collect(Collectors.toList());
310313
queryBuilder.with(prefixedColNames, bindVars).documentsById(prunedIds, "d", bindVars);

src/test/java/com/arangodb/tinkerpop/gremlin/TestGraph.java

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -29,27 +29,27 @@
2929
@Graph.OptOut(
3030
test = "org.apache.tinkerpop.gremlin.structure.util.star.StarGraphTest",
3131
method = "shouldAttachWithCreateMethod",
32-
reason = "FIXME")
32+
reason = "FIXME: DE-997")
3333
@Graph.OptOut(
3434
test = "org.apache.tinkerpop.gremlin.structure.util.star.StarGraphTest",
3535
method = "shouldCopyFromGraphAToGraphB",
36-
reason = "FIXME")
36+
reason = "FIXME: DE-996")
3737
@Graph.OptOut(
3838
test = "org.apache.tinkerpop.gremlin.structure.VertexTest$BasicVertexTest",
3939
method = "shouldEvaluateEquivalentVertexHashCodeWithSuppliedIds",
40-
reason = "FIXME")
40+
reason = "FIXME: DE-996")
4141
@Graph.OptOut(
4242
test = "org.apache.tinkerpop.gremlin.structure.VertexTest$BasicVertexTest",
4343
method = "shouldEvaluateVerticesEquivalentWithSuppliedIdsViaTraversal",
44-
reason = "FIXME")
44+
reason = "FIXME: DE-996")
4545
@Graph.OptOut(
4646
test = "org.apache.tinkerpop.gremlin.structure.VertexTest$BasicVertexTest",
4747
method = "shouldEvaluateVerticesEquivalentWithSuppliedIdsViaIterators",
48-
reason = "FIXME")
48+
reason = "FIXME: DE-996")
4949
@Graph.OptOut(
5050
test = "org.apache.tinkerpop.gremlin.structure.VertexTest$AddEdgeTest",
5151
method = "shouldAddEdgeWithUserSuppliedStringId",
52-
reason = "FIXME")
52+
reason = "FIXME: DE-996")
5353
@Graph.OptOut(
5454
test = "org.apache.tinkerpop.gremlin.process.traversal.step.map.MergeEdgeTest$Traversals",
5555
method = "*",

0 commit comments

Comments
 (0)