close
Skip to content

chore(ci): enable hugegraph-struct tests#3038

Merged
imbajin merged 1 commit into
apache:masterfrom
contrueCT:task/enable-struct-ci-tests
May 27, 2026
Merged

chore(ci): enable hugegraph-struct tests#3038
imbajin merged 1 commit into
apache:masterfrom
contrueCT:task/enable-struct-ci-tests

Conversation

@contrueCT
Copy link
Copy Markdown
Contributor

Purpose of the PR

Enable CI coverage for hugegraph-struct tests in the PD/Store workflow.

Currently, hugegraph-struct is installed as an upstream dependency with -DskipTests, so regressions covered only by struct-side tests may not be caught by PR CI.

Main Changes

  • Keep the existing build/fetch behavior for hugegraph-struct
  • Add a dedicated Run hugegraph-struct test step when the module source exists
  • Continue to let PD/Store/HStore jobs depend on the struct job as before

Verifying these changes

  • Trivial rework / code cleanup without any test coverage. (No Need)
  • Already covered by existing tests, such as (please modify tests here).
  • Need tests and can be verified as follows:
    • mvn -U -ntp "-Deditorconfig.skip=true" -pl hugegraph-struct test

Does this PR potentially affect the following parts?

Documentation Status

  • Doc - TODO
  • Doc - Done
  • Doc - No Need

@dosubot dosubot Bot added size:XS This PR changes 0-9 lines, ignoring generated files. ci-cd Build or deploy tests Add or improve test cases labels May 24, 2026
@contrueCT contrueCT changed the title chore: enable hugegraph-struct tests chore(ci): enable hugegraph-struct tests May 24, 2026
@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label May 27, 2026
@imbajin imbajin merged commit b9a3dd9 into apache:master May 27, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-cd Build or deploy lgtm This PR has been approved by a maintainer size:XS This PR changes 0-9 lines, ignoring generated files. tests Add or improve test cases

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature] enable hugegraph-struct tests in CI

2 participants