fix(client): Add SameNeighborsBatchApiTest to test suite#673
Open
sadwitdastreetz wants to merge 3 commits intoapache:masterfrom
Open
fix(client): Add SameNeighborsBatchApiTest to test suite#673sadwitdastreetz wants to merge 3 commits intoapache:masterfrom
sadwitdastreetz wants to merge 3 commits intoapache:masterfrom
Conversation
Add SameNeighborsBatchApiTest to test suite (temporarily @ignore due to missing server API)
dosubot
bot
added
the
size:XS
sadwitdastreetz
changed the title
sadwitdastreetz
changed the title
sadwitdastreetz
changed the title
sadwitdastreetz
changed the title
sadwitdastreetz
changed the title
dosubot
bot
added
size:L
sadwitdastreetz
force-pushed
the
bugfix-branch
branch
from
b4e0292 to
879f18d
Compare
dosubot
bot
added
size:XS
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #673 +/- ##
============================================
- Coverage 62.49% 62.00% -0.50%
+ Complexity 1903 936 -967
============================================
Files 262 93 -169
Lines 9541 4572 -4969
Branches 886 531 -355
============================================
- Hits 5963 2835 -3128
+ Misses 3190 1523 -1667
+ Partials 388 214 -174 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Contributor
|
@imbajin please take a review : ) |
There was a problem hiding this comment.
Pull Request Overview
This pull request adds the SameNeighborsBatchApiTest to the test suite while temporarily ignoring it due to the missing server API.
- Added a new test class with an @ignore annotation referencing the unimplemented API.
- Updated the ApiTestSuite to include the new test class.
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| hugegraph-client/src/test/java/org/apache/hugegraph/api/traverser/SameNeighborsBatchApiTest.java | Added new test class with @ignore to temporarily bypass tests for unimplemented API. |
| hugegraph-client/src/test/java/org/apache/hugegraph/api/ApiTestSuite.java | Updated test suite by including the new SameNeighborsBatchApiTest class. |
Comments suppressed due to low confidence (1)
hugegraph-client/src/test/java/org/apache/hugegraph/api/traverser/SameNeighborsBatchApiTest.java:32
- [nitpick] Consider removing the angle brackets from the URL in the @ignore message to ensure consistent formatting and clarity.
@Ignore("Blocked by server: /traversers/sameneighborsbatch not implemented, see <https://github.com/apache/incubator-hugegraph/issues/2798>")
imbajin
reviewed
Jul 3, 2025
|
|
||
| import com.google.common.collect.ImmutableList; | ||
|
|
||
| @Ignore("Blocked by server: /traversers/sameneighborsbatch not implemented, see <https://github.com/apache/incubator-hugegraph/issues/2798>") |
Previously, I added ignore to SameNeighborsBatchApi because I found the API wasn't implemanted in server. now, I should delete this @ignore for the completion of SameNeighborsBatchApi in the community version.
|
Due to the lack of activity, the current pr is marked as stale and will be closed after 180 days, any update will remove the stale label |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add SameNeighborsBatchApiTest to test suite (temporarily Ignore due to missing server API)
temporarily Ignore due to missing server API, related to server Issue#2798
apache/hugegraph#2798
close #672