Fix get_or_create_user test

Max Brunsfeld created 2 years ago

Change summary

crates/collab/src/db/tests/db_tests.rs | 31 +++++++++------------------
1 file changed, 11 insertions(+), 20 deletions(-)

Detailed changes

crates/collab/src/db/tests/db_tests.rs

@@ -80,18 +80,17 @@ test_both_dbs!(
 );
 
 async fn test_get_or_create_user_by_github_account(db: &Arc<Database>) {
-    let user_id1 = db
-        .create_user(
-            "[e-mail]",
-            false,
-            NewUserParams {
-                github_login: "login1".into(),
-                github_user_id: 101,
-            },
-        )
-        .await
-        .unwrap()
-        .user_id;
+    db.create_user(
+        "[e-mail]",
+        false,
+        NewUserParams {
+            github_login: "login1".into(),
+            github_user_id: 101,
+        },
+    )
+    .await
+    .unwrap()
+    .user_id;
     let user_id2 = db
         .create_user(
             "[e-mail]",
@@ -105,14 +104,6 @@ async fn test_get_or_create_user_by_github_account(db: &Arc<Database>) {
         .unwrap()
         .user_id;
 
-    let user = db
-        .get_or_create_user_by_github_account("login1", 1, None)
-        .await
-        .unwrap();
-    assert_eq!(user.id, user_id1);
-    assert_eq!(&user.github_login, "login1");
-    assert_eq!(user.github_user_id, Some(101));
-
     let user = db
         .get_or_create_user_by_github_account("the-new-login2", 102, None)
         .await