368
88%

Ran 06 Oct 2025 05:55PM UTC

Jobs 1

Files 160

Run time 1min

Badge

Embed ▾

Committed 06 Oct 2025 05:54PM UTC coverage: 50.747% (-0.5%) from 51.295%

Build # 368

Build Type

push

circleci

Committed by BRX0002735_prosegur

Commit Message

fix(middleware/logger): resolve race condition in JSON logger test

Fixed intermittent test failures in TestLoggerMiddlewareJSON caused by
a race condition in the captureOutput helper function.

Changes:
- Modified captureOutput to properly synchronize pipe reading with a
  done channel, ensuring all output is captured before returning
- Changed from channel-based output retrieval to direct buffer access
  with proper synchronization
- Added stderr capture alongside stdout to handle loggers that may
  write to either stream
- Added io.ReadAll(resp.Body) in test cases to ensure response is
  fully consumed before closing the pipe

The race condition occurred because the pipe was being closed and read
before the logger middleware had finished writing its output. This
resulted in empty captures on the first test run (~33% failure rate
with --count=3).

The fix ensures deterministic test behavior by waiting for the reader
goroutine to complete via the done channel before returning the
captured output.

Tests now pass consistently across multiple runs.

Run Details

4146 of 8170 relevant lines covered (50.75%)

1326.08 hits per line

Uncovered Existing Lines

Lines	Coverage	∆	File
2	98.5	-0.74%	middleware/cache/cache.go
2	95.0	-5.0%	middleware/compress/compress.go
2	96.15	-3.85%	middleware/recover/recover.go
59	61.33	-1.11%	middleware/logger/logger.go

Jobs

ID	Job ID	Ran	Files	Coverage
1	368.1	06 Oct 2025 05:55PM UTC	160	50.75	CircleCI Job

jeffotoni / quick / 368
88%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Uncovered Existing Lines

Jobs

Source Files on build 368

jeffotoni / quick / 368 88%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Uncovered Existing Lines

Jobs

Source Files on build 368

jeffotoni / quick / 368
88%

README BADGES
x