From 74e041c8f8845efc588e26372fc2d8793144caa2 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Tom=20Houl=C3=A9?= Date: Thu, 26 Mar 2020 11:43:37 +0100 Subject: [PATCH] Fix unused value warning in test It flew under the radar for a long time, this is worrying. --- .../tests/migration_persistence_tests.rs | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/migration-engine/migration-engine-tests/tests/migration_persistence_tests.rs b/migration-engine/migration-engine-tests/tests/migration_persistence_tests.rs index f45c668687c2..9a1f741df856 100644 --- a/migration-engine/migration-engine-tests/tests/migration_persistence_tests.rs +++ b/migration-engine/migration-engine-tests/tests/migration_persistence_tests.rs @@ -11,11 +11,13 @@ async fn last_should_return_none_if_there_is_no_migration(api: &TestApi) { } #[test_each_connector] -async fn last_must_return_none_if_there_is_no_successful_migration(api: &TestApi) { +async fn last_must_return_none_if_there_is_no_successful_migration(api: &TestApi) -> TestResult { let persistence = api.migration_persistence(); - persistence.create(Migration::new("my_migration".to_string())); - let loaded = persistence.last().await.unwrap(); + persistence.create(Migration::new("my_migration".to_string())).await?; + let loaded = persistence.last().await?; assert_eq!(loaded, None); + + Ok(()) } #[test_each_connector]