Removing tests I shouldn't have put back in

Author: carols10cents

src/user/mod.rs

@@ -443,54 +443,3 @@ pub fn stats(req: &mut Request) -> CargoResult<Response> {
     }
     Ok(req.json(&R { total_downloads: data }))
 }
-
-#[cfg(test)]
-mod tests {
-    use super::*;
-    use diesel::pg::PgConnection;
-    use dotenv::dotenv;
-    use std::env;
-
-    fn connection() -> PgConnection {
-        let _ = dotenv();
-        let database_url =
-            env::var("TEST_DATABASE_URL").expect("TEST_DATABASE_URL must be set to run tests");
-        let conn = PgConnection::establish(&database_url).unwrap();
-        conn.begin_test_transaction().unwrap();
-        conn
-    }
-
-    #[test]
-    fn new_users_have_different_api_tokens() {
-        let conn = connection();
-        let user1 = NewUser::new(1, "foo", None, None, None, "foo")
-            .create_or_update(&conn)
-            .unwrap();
-        let user2 = NewUser::new(2, "bar", None, None, None, "bar")
-            .create_or_update(&conn)
-            .unwrap();
-
-        assert_ne!(user1.id, user2.id);
-        assert_ne!(user1.api_token, user2.api_token);
-        assert_eq!(32, user1.api_token.len());
-    }
-
-    #[test]
-    fn updating_existing_user_doesnt_change_api_token() {
-        let conn = connection();
-        let user_after_insert = NewUser::new(1, "foo", None, None, None, "foo")
-            .create_or_update(&conn)
-            .unwrap();
-        let original_token = user_after_insert.api_token;
-        NewUser::new(1, "bar", None, None, None, "bar_token")
-            .create_or_update(&conn)
-            .unwrap();
-        let mut users = users::table.load::<User>(&conn).unwrap();
-        assert_eq!(1, users.len());
-        let user = users.pop().unwrap();
-
-        assert_eq!("bar", user.gh_login);
-        assert_eq!("bar_token", user.gh_access_token);
-        assert_eq!(original_token, user.api_token);
-    }
-}