1
85%
master: 85%

Ran 23 Apr 2026 02:24PM UTC

Files 310

Run time 15s

Badge

Embed ▾

Committed 23 Apr 2026 02:17PM UTC coverage: 85.288% (-4.0%) from 89.306%

Job # 24840434322.1

Build Type

Pull #2090

gh-ci

Committed by

vchaindz

Commit Message

test(pgsql): fix closure-capture race in TestSession_MessageReader

The test declared a single c1/c2 pair at the top of the function and
reused it across three independent sub-scenarios. Each block spawned a
goroutine that wrote/closed c2, but the goroutines captured c2 by
*reference* through the enclosing closure, not by value. When the main
goroutine reassigned c2 via 'c1, c2 = net.Pipe()' for the next block
before a previous goroutine had finished executing its trailing
c2.Close(), that Close would hit the freshly-assigned pipe — closing the
new pipe before the new goroutine had written anything. The reader then
surfaced io.EOF where the test expected ErrMalformedMessage. The race
was benign locally but reproducible on ubuntu CI runners.

Fix: put each block in its own scope with its own c1/c2, and pass the
peer connection into the goroutine as a function parameter so the
capture is by value. No behaviour change to the production code under
test — only test hygiene.

Pull Request Pull Request #2090: v1.11.0: PostgreSQL compatibility and SQL feature expansion

Coverage Stats

44604 of 52298 relevant lines covered (85.29%)

127532.52 hits per line

codenotary / immudb / 24840434322 / 1
85%
master: 85%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 24840434322.1

codenotary / immudb / 24840434322 / 1 85% master: 85%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 24840434322.1

codenotary / immudb / 24840434322 / 1
85%
master: 85%

README BADGES
x