refactor: create internal/integrationtest package infrastructure#206

Merged

pawbana merged 32 commits intomainfrom

pb/testing-cleanup-new-request-bridge

Mar 10, 2026

Commits on Mar 10, 2026

refactor: create internal/integrationtest package infrastructure
pawbana
committed
refactor: migrate metrics and trace integration tests to internal/integrationtest/
pawbana
committed
refactor: migrate responses, circuit_breaker, and apidump integration tests to internal/integrationtest/
pawbana
committed
Finish bridge_test.go migration: replace all configureFunc/inline boilerplate with NewBridgeTestServer
pawbana
committed
refactor: convert bridge_test.go from external to internal test package
pawbana
committed
refactor: use internal test package, unexport all helpers
pawbana
committed
refactor: split bridge_test.go into per-interceptor test files
pawbana
committed
fixup: apply unexport renames to remaining helper files
pawbana
committed
remove providerFunc
pawbana
committed
merge tests back into bridge_test.go
pawbana
committed
bridge.go -> setupbridge.go + helpers.go
pawbana
committed
remove withWrappedRecorder
pawbana
committed
remove actorID arg from setupInjectedToolTest
pawbana
committed
use bridgeTestServer.newRequest
pawbana
committed
refactor: update makeRequest call sites in trace_test.go for new *http.Response return type
pawbana
committed
refactor: update makeRequest call sites in responses_test.go
pawbana
committed
refactor: update makeRequest call sites in bridge_test.go
pawbana
committed
bridgeServer.makeRequest + tracerSetup + TestSessionIDTracking cleanup
pawbana
committed
cleanup
pawbana
committed
reorder tests in bridge_test.go to original order
pawbana
committed
remove not needed withLogger
pawbana
committed
fix comments
pawbana
committed
rename
pawbana
committed
formatting fix
pawbana
committed
reorder to previous
pawbana
committed
recorder cleanup
pawbana
committed
MockRecorder Total*Tokens
pawbana
committed
add Body.Close() to bridgeServer.makeRequest + some test names cleanup
pawbana
committed
changed TestResponsesParallelToolsOverwritten and TestOpenAIChatCompletions to use newBridgeTestServer + test some names cleanup
pawbana
committed
fmt fix
pawbana
committed
review 1: fixed setting headers in makeRequest, removed unneded providerName from TestFallthrough
pawbana
committed
review 2
pawbana
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

refactor: create internal/integrationtest package infrastructure#206

refactor: create internal/integrationtest package infrastructure#206
pawbana merged 32 commits intomainfrom
pb/testing-cleanup-new-request-bridge

Commits on Mar 10, 2026

refactor: create internal/integrationtest package infrastructure

refactor: migrate metrics and trace integration tests to internal/integrationtest/

refactor: migrate responses, circuit_breaker, and apidump integration tests to internal/integrationtest/

Finish bridge_test.go migration: replace all configureFunc/inline boilerplate with NewBridgeTestServer

refactor: convert bridge_test.go from external to internal test package

refactor: use internal test package, unexport all helpers

refactor: split bridge_test.go into per-interceptor test files

fixup: apply unexport renames to remaining helper files

remove providerFunc

merge tests back into bridge_test.go

bridge.go -> setupbridge.go + helpers.go

remove withWrappedRecorder

remove actorID arg from setupInjectedToolTest

use bridgeTestServer.newRequest

refactor: update makeRequest call sites in trace_test.go for new *http.Response return type

refactor: update makeRequest call sites in responses_test.go

refactor: update makeRequest call sites in bridge_test.go

bridgeServer.makeRequest + tracerSetup + TestSessionIDTracking cleanup

cleanup

reorder tests in bridge_test.go to original order

remove not needed withLogger

fix comments

rename

formatting fix

reorder to previous

recorder cleanup

MockRecorder Total*Tokens

add Body.Close() to bridgeServer.makeRequest + some test names cleanup

changed TestResponsesParallelToolsOverwritten and TestOpenAIChatCompletions to use newBridgeTestServer + test some names cleanup

fmt fix

review 1: fixed setting headers in makeRequest, removed unneded providerName from TestFallthrough

review 2