alibaba/GraphScope

[GIE/tests] Add `PathExpand` test cases for GIE

Open

#2,882 opened on Jun 14, 2023

View on GitHub
 (0 comments) (0 reactions) (1 assignee)HTML (301 forks)batch import
component:giegood first issue

Repository metrics

Stars
 (2,401 stars)
PR merge metrics
 (Avg merge 1m) (7 merged PRs in 30d)

Description

There currently lacks in PathExpand test cases in GIE (e2e), especially that we provide syntactic sugars in PathExpand to denote some complex situations. About the syntactic, you can refer to the Doc: https://graphscope.io/docs/latest/interactive_engine/supported_gremlin_steps#pathexpand

It's thus welcome to add more PathExpand test cases. It's easy to do so, simply prepare the PathExpand case and the expected result (by using Tinkerpop) in: https://github.com/alibaba/GraphScope/blob/main/interactive_engine/compiler/src/main/java/com/alibaba/graphscope/gremlin/integration/suite/standard/IrGremlinQueryTest.java

You can run the test locally according to: https://graphscope.io/docs/latest/interactive_engine/dev_and_test The case will also be verified in the ci workflow.

Contributor guide

Research direction
Read the documentation on PathExpand syntax, then locate the test file and add a new test case with expected results from TinkerPop.
Tech stack
java
Domain
backendtesting
Issue type
Test
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
JavaGit
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
70