Muawiya-contact commented on issue #327:
URL: https://github.com/apache/hugegraph-ai/issues/327#issuecomment-4467131543
Hey @imbajin 👋
Just a soft follow-up on **#327** — no rush at all!
Quick recap of the issue:
- `setUpClass` in `test_gremlin.py` (L41–52) silently skips
all **6 Gremlin integration tests** on any connectivity failure
- CI would report `6 skipped` instead of `6 failed` — hiding
real regressions behind a green status
The fix is small and low-risk. Two options on the table:
- **Option A** — Drop the probe entirely, let tests fail naturally
- **Option B** — Replace `unittest.SkipTest` with `AssertionError`
in the except block
Same principle as the auth probe fix in **#325**.
Whenever you get a moment — could you verify the issue and
assign it to me? I'm ready to open a PR right away. 🙏
— Muawiya
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
