hyperledger / fabric-x-committer / 21949663635

90%

Build Type

push

github

Committed by web-flow

Commit Message

Fix flaky test by tracking connection state per gRPC target (#313)

#### Type of change

- Bug fix
 
#### Description

ConnectionMetrics used a single connected boolean for all gRPC targets.
This is wrong when there are multiple targets because the boolean cannot
distinguish which target set it.

Example of the bug with 2 verifiers (A, B):
  1. Verifier A connects    → sets shared connected=true
  2. Verifier B fails to open stream → calls Disconnected()
  3. Disconnected() sees connected=true (set by A, not B)
  4. Assumes B was previously connected → increments B's failure counter
  5. But B was never connected — the failure count is wrong

This commit replaces the single boolean with a per-target map so
Disconnected() only counts a failure if that specific target was
previously connected.

#### Related issues

  - resolves #278

Signed-off-by: Senthilnathan <cendhu@gmail.com>

Run Details

2 of 2 new or added lines in 1 file covered. (100.0%)

23 existing lines in 6 files now uncovered.

9092 of 10249 relevant lines covered (88.71%)

1.29 hits per line