typosquat/test_util: Use tests::util::github::next_gh_id()

Author: Turbo87

src/lib.rs

@@ -60,7 +60,7 @@ pub mod tasks;
 pub mod team_repo;
 mod test_util;
 #[cfg(test)]
-mod tests;
+pub mod tests;
 pub mod typosquat;
 pub mod util;
 pub mod views;

src/tests/mod.rs

@@ -32,7 +32,7 @@ mod team;
 mod token;
 mod unhealthy_database;
 mod user;
-mod util;
+pub mod util;
 mod version;
 mod worker;
 

src/tests/util.rs

@@ -85,6 +85,7 @@ pub fn encode_session_header(session_key: &cookie::Key, user_id: i32) -> String
 /// A collection of helper methods for the 3 authentication types
 ///
 /// Helper methods go through public APIs, and should not modify the database directly
+#[allow(async_fn_in_trait)]
 pub trait RequestHelper {
     fn request_builder(&self, method: Method, path: &str) -> MockRequest;
     fn app(&self) -> &TestApp;

src/typosquat/test_util.rs

@@ -1,5 +1,6 @@
 use diesel::{prelude::*, PgConnection};
 
+use crate::tests::util::github::next_gh_id;
 use crate::{
     models::{
         Crate, CrateOwner, NewCrate, NewTeam, NewUser, NewVersion, Owner, OwnerKind, User, Version,
@@ -10,14 +11,12 @@ use crate::{
 
 pub struct Faker {
     emails: Emails,
-    id: i32,
 }
 
 impl Faker {
     pub fn new() -> Self {
         Self {
             emails: Emails::new_in_memory(),
-            id: Default::default(),
         }
     }
 
@@ -83,8 +82,8 @@ impl Faker {
         Ok(Owner::Team(
             NewTeam::new(
                 &format!("github:{org}:{team}"),
-                self.next_id(),
-                self.next_id(),
+                next_gh_id(),
+                next_gh_id(),
                 Some(team.to_string()),
                 None,
             )
@@ -94,16 +93,11 @@ impl Faker {
 
     pub fn user(&mut self, conn: &mut PgConnection, login: &str) -> anyhow::Result<User> {
         Ok(
-            NewUser::new(self.next_id(), login, None, None, "token").create_or_update(
+            NewUser::new(next_gh_id(), login, None, None, "token").create_or_update(
                 None,
                 &self.emails,
                 conn,
             )?,
         )
     }
-
-    fn next_id(&mut self) -> i32 {
-        self.id += 1;
-        self.id
-    }
 }