gchq / Gaffer

A large-scale entity and relation database supporting aggregation of properties
Apache License 2.0
1.75k stars 354 forks source link

Gh-3200: Additional Federated Stores Tinkerpop tests #3204

Closed cn337131 closed 1 month ago

cn337131 commented 2 months ago

Added a few more tests of a federated store with a GafferPopGraph. These test whether the .with() config options allow users to specify the graph they wish to target for results using Gremlin.

Also an additional fix in GafferPopGraph which prevented Gremlin queries seeding with edge IDs in cases where edges were not directed.

Note: docs will need to be added that explain to users how to use the .with() to query federated graphs.

Related issue

codecov[bot] commented 2 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 66.86%. Comparing base (49a8d38) to head (c08d63d).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #3204 +/- ## ============================================= - Coverage 66.87% 66.86% -0.02% + Complexity 2558 2556 -2 ============================================= Files 912 912 Lines 29284 29284 Branches 3268 3268 ============================================= - Hits 19584 19580 -4 - Misses 8256 8258 +2 - Partials 1444 1446 +2 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.