32662943254
85%

Ran 23 Aug 2026 08:00PM UTC

Jobs 1

Files 54

Run time 1min

Badge

Embed ▾

Committed 23 Aug 2026 07:58PM UTC coverage: 85.319%. Remained the same

Build # 32662943254

Build Type

push

github

Committed by

web-flow

Commit Message

Give each import request in TestMigrator_ImportDouble its own reader (#2220)

The test passed one strings.Reader as the body of both POSTs. client.Do
returns once the response headers arrive, and the import answers 202 before
the transport has finished copying the body, so the second http.NewRequest
reads the reader's Len to set ContentLength while the first request's
writeLoop is still advancing it. The race detector caught it on CI as a write
in strings.(*Reader).WriteTo against a read in NewRequestWithContext, failing
a test nothing had touched.

Reproduced in isolation to confirm the mechanism rather than infer it from the
trace: a handler that answers 202 without draining an 8 MiB body, two requests
sharing one reader, and -race reports strings.(*Reader).Len in
NewRequestWithContext against strings.(*Reader).Read on every run. It does not
reproduce in this package locally, which is why it reads as a flake.

Both requests now build their own reader over the same content. The second one
carries a full body where before it inherited a consumed one, which is closer
to what the case is about: a second import arriving while the first is running
still has to be refused.

Coverage Stats

6631 of 7772 relevant lines covered (85.32%)

38.37 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	32662943254.1	23 Aug 2026 08:00PM UTC	54	85.32	GitHub Action Run

umputun / remark42 / 32662943254
85%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 32662943254

umputun / remark42 / 32662943254 85%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 32662943254

umputun / remark42 / 32662943254
85%

README BADGES
x