typosquat/test_util: Replace Owner with Team

There appears to be no reason for the `team()` fn to return an `Owner` instead of a `Team`, aside from the `Owner` being required by the `add_crate_to_team()` fn. But that fn literally has `team` in the name, so it might as well take a `Team` as an argument too :D

Author: Turbo87

src/typosquat/test_util.rs

@@ -2,7 +2,7 @@ use diesel::{prelude::*, PgConnection};
 
 use crate::tests::util::github::next_gh_id;
 use crate::{
-    models::{Crate, CrateOwner, NewTeam, NewUser, Owner, OwnerKind, User},
+    models::{Crate, CrateOwner, NewTeam, NewUser, OwnerKind, Team, User},
     schema::{crate_owners, users},
 };
 
@@ -15,14 +15,14 @@ pub mod faker {
         conn: &mut PgConnection,
         user: &User,
         krate: &Crate,
-        team: &Owner,
+        team: &Team,
     ) -> anyhow::Result<()> {
         // We have to do a bunch of this by hand, since normally adding a team owner triggers
         // various checks.
         diesel::insert_into(crate_owners::table)
             .values(&CrateOwner {
                 crate_id: krate.id,
-                owner_id: team.id(),
+                owner_id: team.id,
                 created_by: user.id,
                 owner_kind: OwnerKind::Team,
                 email_notifications: true,
@@ -47,7 +47,7 @@ pub mod faker {
             .map_err(|err| anyhow!(err.to_string()))
     }
 
-    pub fn team(conn: &mut PgConnection, org: &str, team: &str) -> anyhow::Result<Owner> {
+    pub fn team(conn: &mut PgConnection, org: &str, team: &str) -> anyhow::Result<Team> {
         let login = format!("github:{org}:{team}");
         let team = NewTeam::builder()
             .login(&login)
@@ -56,7 +56,7 @@ pub mod faker {
             .name(team)
             .build();
 
-        Ok(Owner::Team(team.create_or_update(conn)?))
+        Ok(team.create_or_update(conn)?)
     }
 
     pub fn user(conn: &mut PgConnection, login: &str) -> QueryResult<User> {