From ad0d781a2f56ac14d3484f7f253c975eb4b3b7fb Mon Sep 17 00:00:00 2001 From: Bryan Newbold Date: Fri, 1 Feb 2019 17:59:09 -0800 Subject: remove test client helper --- rust/tests/helpers.rs | 36 ------------------------------------ 1 file changed, 36 deletions(-) diff --git a/rust/tests/helpers.rs b/rust/tests/helpers.rs index f4cd93e9..4a664403 100644 --- a/rust/tests/helpers.rs +++ b/rust/tests/helpers.rs @@ -14,42 +14,6 @@ use std::str::FromStr; pub static TEST_ADMIN_EDITOR_ID: &str = "aaaaaaaaaaaabkvkaaaaaaaaae"; //static TEST_ADMIN_EDITOR_ID: FatcatId = FatcatId::from_str("aaaaaaaaaaaabkvkaaaaaaaaae").unwrap(); -// A current problem with this method is that if the test fails (eg, panics, assert fails), the -// server never gets closed, and the server thread hangs forever. -// One workaround might be to invert the function, take a closure, capture the panic/failure, and -// cleanup. -#[allow(dead_code)] -pub fn setup_client() -> (Client, Context, Listening) { - let server = server::create_test_server().unwrap(); - - // setup auth as admin user - let admin_id = FatcatId::from_str(TEST_ADMIN_EDITOR_ID).unwrap(); - let token = server - .auth_confectionary - .create_token(admin_id, None) - .unwrap(); - let client_context = Context { - x_span_id: None, - authorization: None, - auth_data: Some(swagger::auth::AuthData::ApiKey(token)), - }; - - let router = fatcat_api_spec::router(server); - let mut chain = Chain::new(router); - chain.link_before(fatcat_api_spec::server::ExtractAuthData); - chain.link_before(MacaroonAuthMiddleware::new()); - - let mut iron_server = Iron::new(chain); - iron_server.threads = 1; - // XXX: this isn't supposed to block, but it is. Disabling these tests for now. - let iron_server = iron_server - .http("localhost:9300") - .expect("Failed to start HTTP server"); - - let client = Client::try_new_http("http://localhost:9144").unwrap(); - (client, client_context, iron_server) -} - #[allow(dead_code)] pub fn setup_http() -> ( Headers, -- cgit v1.2.3