[Bug] test: fix silent skip anti-pattern in test_gremlin.py setUpClass probe

[Muawiya-contact]

Bug Type (问题类型)

None

Before submit

• I had searched in the issues and found no similar issues.

Environment (环境信息)

---

Environment

Item	Detail
Module	python-client
File	hugegraph-python-client/src/tests/test_gremlin.py
Affected Lines	L41–52 (setUpClass)

---

Expected & Actual behavior (期望与实际表现)

Expected & Actual Behavior

Problem

The skip_gremlin_tests probe inside setUpClass (L41–52) silently skips all 6 Gremlin integration tests whenever the Gremlin endpoint returns a 404, times out, or raises a connection error.

This is a silent-skip anti-pattern: a future regression would surface in CI as 6 skipped rather than 6 failed, making it invisible to reviewers and breaking the reliability of the test suite.

Expected

Any connectivity or response failure during the probe should fail loudly — the same principle applied to the auth probe fix in #325.

Actual

The probe swallows 404 / timeout / connection error and marks the entire class as skipped via unittest.skip, hiding real failures behind a green (skipped) status.

---

Suggested Fix

Two equivalent options:

Option A — Drop the probe entirely

Remove skip_gremlin_tests and let the first test fail naturally if the endpoint is unavailable. This is the simplest fix and mirrors standard integration-test practice.

Option B — Convert the 404 branch to AssertionError

# Before (silent skip)
except (ConnectionError, Timeout, HTTPError):
    raise unittest.SkipTest("Gremlin endpoint unavailable")
After (loud failure)
except (ConnectionError, Timeout, HTTPError) as e:

raise AssertionError(

f"Gremlin endpoint probe failed — endpoint may be down or misconfigured: {e}"

)

Either option ensures regressions surface as failures, not silent skips.

---

Related Issues

• # fix: implement graphspace-scoped auth routing in Python client #325
• Upgrade HugeGraph Python Client CI from 1.3.0 to 1.7.0 #320

---

Labels

test python-client

Vertex/Edge example (问题点 / 边数据举例)

Schema [VertexLabel, EdgeLabel, IndexLabel] (元数据结构)

[Muawiya-contact]

@imbajin, can you please verify that everything is in order and then assign it to me?

[Muawiya-contact]

Implementation Plan — #327

Goal

Remove the silent-skip anti-pattern in test_gremlin.py so CI failures surface as failed, not skipped.

---

Steps

1. Locate setUpClass in hugegraph-python-client/src/tests/test_gremlin.py (L41–52)
2. Choose a fix:
   • Option A — Delete the skip_gremlin_tests probe entirely; let tests fail naturally
   • Option B — Replace unittest.SkipTest with AssertionError in the except block
3. Run the full Gremlin test suite locally against a live endpoint to confirm no false skips
4. Verify CI reports 6 failed (not 6 skipped) when the endpoint is down
5. Open PR referencing [Bug] test: fix silent skip anti-pattern in test_gremlin.py setUpClass probe #327, # fix: implement graphspace-scoped auth routing in Python client #325, Upgrade HugeGraph Python Client CI from 1.3.0 to 1.7.0 #320

---

Scope

Item	Detail
File	hugegraph-python-client/src/tests/test_gremlin.py
Lines	L41–52
Tests affected	6 Gremlin integration tests
Labels	test python-client