Fix flaky transport mutation assertion in TestIntegrationPluginTransport#620

Merged

winhowes merged 1 commit intomainfrom

codex/find-and-fix-failing-unit-tests

Apr 13, 2026

Owner

winhowes commented Apr 13, 2026

Motivation

The test TestIntegrationPluginTransport assumed the base transport's IdleConnTimeout was 0, which can vary by environment and caused intermittent failures; the test should instead snapshot and assert the value is preserved.

Description

Capture baseTr.IdleConnTimeout before calling AddIntegration and assert baseTr.IdleConnTimeout remains equal to that snapshot after the call, updating app/integration_test.go accordingly.

Testing

Ran go test ./app -run TestIntegrationPluginTransport -count=1 which passed, and go test ./... which also passed.


          Fix flaky transport mutation assertion in integration test

2fc2095

winhowes added the codex label

— with

ChatGPT Codex Connector

winhowes merged commit 042ed41 into main

1 check passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels