ilpanich / axiam-kotlin-sdk / 30024376046

92%

push

github

test(coverage): Kotlin SDK 91.3% → 99.1% + enforce Kover gate (Phase D2) (#6)

* fix(test): un-hide 5 @Test methods JUnit was silently never running

Bug found while raising Kotlin SDK coverage (Phase D2): in AuthFlowTest.kt
and ClientExtraTest.kt, 5 `@Test fun ...() = runBlocking { ... }` methods
had their last statement be `assertThrows(...) { ... }`, whose return value
(the caught exception) became the function's INFERRED return type instead
of Unit. JUnit Jupiter silently excludes non-void @Test methods from
discovery — no error, no "skipped" marker, just absent from every run.

Affected tests (compiled fine, decompiled correctly, never executed):
  - AuthFlowTest: "login wrong credentials maps 401 to AuthError"
  - ClientExtraTest: "refresh fails cleanly when org id cannot be resolved",
    "explicit refresh propagates a server rejection as AuthError",
    "logout maps a server error to an exception",
    "a malformed success body surfaces as NetworkError"

Fix: declare an explicit `: Unit` return type so Kotlin discards the
expression's value (a well-known compiler-supported coercion), restoring
JUnit's void-returning discovery contract. No test logic changed.

Verified via `javap -p` bytecode inspection (return types were
AuthError/NetworkError instead of void) and JUnit's own
<testsuite tests="N"> counts before/after. This alone raised measured LINE
coverage from 513/562 (91.28%) to 519/562 (92.35%) with zero new tests,
since these 5 tests already exercised previously-"missed" branches.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01MbKy7FMMeAAXSoAzvMKKNR

* test(kotlin-sdk): direct SessionState coverage for D2-1 (91.6% -> full)

Adds SessionStateDirectTest.kt, constructing internal/SessionState.kt
directly (bypassing AxiamClient) so every error path can be driven without
a full login flow:

- isNearExpiry(): undecodable token, missing exp claim, within/outside the
  near-expiry buffer ... (continued)

245 of 306 branches covered (80.07%)

Branch coverage included in aggregate %.

553 of 559 relevant lines covered (98.93%)

7.09 hits per line