divviup / divviup-api / 26256510268

69%

Build Type

push

github

Committed by web-flow

Commit Message

Migrate from Trillium [part 9B]: Rewrite test infrastructure + remove all trillium deps (#2266)

Replace the entire trillium-based test harness with axum-native equivalents and remove every
trillium dependency from the workspace. Are you ready for a PARTY? This one removes Trillium.
Say goodnight.

test-support/* rewrite:
- TestRequest/TestResponse types using tower::ServiceExt::oneshot()
- ClientLogs rewritten as axum middleware capturing request/response bodies
- ApiMocks rewritten with host-based axum::Router dispatch
- User injection via X-Integration-Testing-User header (was in the trillium conn state)

src/ crate cleanup:
- Delete handler::origin_router, handler::proxy, ErrorHandler, and the DivviupApi trillium
  Handler struct
- Delete inject_integration_testing_user middleware (the header-based inject_test_header_user is
  sufficient)
- Rewrote src/api_mocks/*

tests/integration/*:
- All response_json() calls are now synchronous (body eagerly consumed)

Here we see the final destruction of the trillium alliance, and the end of its insignificant
rebellion against our dependency tree.

I've left some more cleanup for Part 10, including refactoring 'with_state' to 'with_user' in
~100 callsites, and the restoration of remaining HTTP metrics.

Coverage Stats

345 of 366 new or added lines in 14 files covered. (94.26%)

2 existing lines in 2 files now uncovered.

4422 of 6430 relevant lines covered (68.77%)

69.11 hits per line