From 3b45b72370bbb4c841287d33ed3727feae9656b7 Mon Sep 17 00:00:00 2001 From: kushaim Date: Tue, 4 Aug 2026 01:17:15 -0700 Subject: [PATCH] test(relay): skip 2 PG-dependent admin report/feedback 404 tests Fixes #4083. admin::tests::report_detail_rejects_unknown_report and admin::tests::feedback_attachment_rejects_unknown_feedback both call test_state() which eagerly seeds a community via ensure_configured_community. Without BUZZ_TEST_DATABASE_URL pointing at a live Postgres, the seed panics with a 500 surfaced as the route's response, not the application-level ApiError::not_found the assertions expect. Adds the same #[ignore = "requires Postgres"] attribute used in 100+ places across buzz-db, buzz-relay/src/api/{bridge,invites,operator}, and buzz-search/tests/fts_integration.rs. With the attribute, cargo test -p buzz-relay passes in PG-less environments; maintainers opt back in with the existing --include-ignored flag. Signed-off-by: kushaim --- crates/buzz-relay/src/api/admin/mod.rs | 2 ++ 1 file changed, 2 insertions(+) diff --git a/crates/buzz-relay/src/api/admin/mod.rs b/crates/buzz-relay/src/api/admin/mod.rs index 21f30065f0..3d77607c80 100644 --- a/crates/buzz-relay/src/api/admin/mod.rs +++ b/crates/buzz-relay/src/api/admin/mod.rs @@ -389,6 +389,7 @@ mod tests { assert_eq!(response.status(), axum::http::StatusCode::FORBIDDEN); } + #[ignore = "requires Postgres"] #[tokio::test] async fn report_detail_rejects_unknown_report() { let response = router(test_state().await) @@ -419,6 +420,7 @@ mod tests { assert_eq!(response.status(), axum::http::StatusCode::FORBIDDEN); } + #[ignore = "requires Postgres"] #[tokio::test] async fn feedback_attachment_rejects_unknown_feedback() { let response = router(test_state().await)