hyperledger / fabric-x-committer / 21863706156

90%

Build Type

push

github

Committed by web-flow

Commit Message

Fix flaky coordinator loadgen test by waiting for setup TX commits (#304)

#### Type of change

- Bug fix
 
#### Description

CoordinatorAdapter.Progress() was returning the send counter
(nextBlockNum) instead of the commit counter (TransactionsCommitted).
This caused submitWorkloadSetupTXs to proceed before the config TX was
actually committed, so subsequent namespace TXs could arrive before
policies were established. Without the sidecar (which serializes the
config TX), the coordinator test relies on Progress() to gate setup TX
ordering.

Also fix error swallowing in coordinator's sendTxStatus goroutine, which
could cause cascading deadlocks on stream errors.

Fix flaky TestLoadGenForLoadGen by explicitly stopping the sub-client
before test cleanup tears down the main server. Test cleanup runs in
LIFO order, so the main server's gRPC Stop ran before the sub-client was
cancelled, causing in-flight AppendBatch RPCs to hit "connection reset
by peer"

#### Related issues

 - resolves #303

---------

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

Run Details

0 of 1 new or added line in 1 file covered. (0.0%)

33 existing lines in 9 files now uncovered.

9058 of 10210 relevant lines covered (88.72%)

1.29 hits per line