From 9c3f775a67f8a9ea313045363a8070ac8245d0c1 Mon Sep 17 00:00:00 2001 From: GW_MC <72297530+GWMCwing@users.noreply.github.com> Date: Wed, 31 Dec 2025 16:05:28 +0800 Subject: [PATCH] refactor: remove unused Filters struct and clean up imports in create_upstream tests --- apps/api/src/helpers/database.rs | 4 ---- .../restricted/nginx/upstream/create_upstream.rs | 15 +++++++++------ 2 files changed, 9 insertions(+), 10 deletions(-) diff --git a/apps/api/src/helpers/database.rs b/apps/api/src/helpers/database.rs index 757121c..b4b42ed 100644 --- a/apps/api/src/helpers/database.rs +++ b/apps/api/src/helpers/database.rs @@ -12,10 +12,6 @@ macro_rules! with_conn { }}; } -pub struct Filters { - pub pagination: Option, -} - pub struct PaginationFilter { pub page: u64, pub per_page: u64, diff --git a/apps/api/src/routes/api/restricted/nginx/upstream/create_upstream.rs b/apps/api/src/routes/api/restricted/nginx/upstream/create_upstream.rs index 7ce3d9e..426cf57 100644 --- a/apps/api/src/routes/api/restricted/nginx/upstream/create_upstream.rs +++ b/apps/api/src/routes/api/restricted/nginx/upstream/create_upstream.rs @@ -138,7 +138,7 @@ mod tests { create_upstream::{CreateUpstreamRequestBody, UpstreamTargetInfo as ReqTarget}, get_upstream_router, }, - services::{agent_client::MockAgentService, get_app_service, get_mock_app_service}, + services::{agent_client::MockAgentService, get_mock_app_service}, }; fn get_router_with_state(db: DatabaseConnection) -> axum::Router { @@ -271,10 +271,12 @@ mod tests { let mut mock = MockAgentService::new(); mock.expect_validate().returning(|_cfg| Ok(())); mock.expect_apply().returning(|_cfg| { - Err(crate::services::agent_client::AgentError::ApplicationFailed( - "internal".to_string(), - "Failed to communicate with the agent.".to_string(), - )) + Err( + crate::services::agent_client::AgentError::ApplicationFailed( + "internal".to_string(), + "Failed to communicate with the agent.".to_string(), + ), + ) }); let mock_agent = Arc::new(mock); @@ -285,7 +287,8 @@ mod tests { .into_connection(); let program_settings = ProgramSettings::mock(); - let app_service = get_mock_app_service(&Arc::new(db.clone()), &program_settings, mock_agent); + let app_service = + get_mock_app_service(&Arc::new(db.clone()), &program_settings, mock_agent); let state = Arc::new(crate::routes::AppState { database_connection: Arc::new(db), service: Arc::new(app_service),