From 3fde3dd50060b815137391e36a114824df82f0a7 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jakub=20Ber=C3=A1nek?= Date: Tue, 18 Feb 2025 16:12:56 +0100 Subject: [PATCH] Temporarily remove DB assignment tests Assignment based on review preferences is not working currently. --- src/handlers/assign.rs | 1 - src/handlers/assign/tests/tests_db.rs | 34 --------------------------- 2 files changed, 35 deletions(-) delete mode 100644 src/handlers/assign/tests/tests_db.rs diff --git a/src/handlers/assign.rs b/src/handlers/assign.rs index bf880d52..7ef91540 100644 --- a/src/handlers/assign.rs +++ b/src/handlers/assign.rs @@ -39,7 +39,6 @@ use tracing as log; #[cfg(test)] mod tests { mod tests_candidates; - mod tests_db; mod tests_from_diff; } diff --git a/src/handlers/assign/tests/tests_db.rs b/src/handlers/assign/tests/tests_db.rs deleted file mode 100644 index 825e0945..00000000 --- a/src/handlers/assign/tests/tests_db.rs +++ /dev/null @@ -1,34 +0,0 @@ -#[cfg(test)] -mod tests { - use crate::handlers::assign::filter_by_capacity; - use crate::tests::run_test; - use std::collections::HashSet; - use tokio_postgres::GenericClient; - - #[tokio::test] - async fn find_reviewers_no_review_prefs() { - run_test(|ctx| async move { - ctx.add_user("usr1", 1).await; - ctx.add_user("usr2", 2).await; - let _users = filter_by_capacity( - ctx.db_client().await.client(), - &candidates(&["usr1", "usr2"]), - ) - .await?; - // FIXME: this test fails, because the query is wrong - // check_users(users, &["usr1", "usr2"]); - Ok(ctx) - }) - .await; - } - - fn candidates(users: &[&'static str]) -> HashSet<&'static str> { - users.into_iter().copied().collect() - } - - fn check_users(users: HashSet, expected: &[&'static str]) { - let mut users: Vec = users.into_iter().collect(); - users.sort(); - assert_eq!(users, expected); - } -}