Skip to content

Commit 8562d4f

Browse files
committed
typosquat/test_util: Remove unused Version from crate_and_version() return type
1 parent 574f523 commit 8562d4f

File tree

3 files changed

+7
-7
lines changed

3 files changed

+7
-7
lines changed

src/typosquat/database.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -183,8 +183,8 @@ mod tests {
183183

184184
// Let's set up a team that owns both b and c, but not a.
185185
let not_the_a_team = faker::team(&mut conn, "org", "team")?;
186-
faker::add_crate_to_team(&mut conn, &user_b, &top_b.0, &not_the_a_team)?;
187-
faker::add_crate_to_team(&mut conn, &user_b, &not_top_c.0, &not_the_a_team)?;
186+
faker::add_crate_to_team(&mut conn, &user_b, &top_b, &not_the_a_team)?;
187+
faker::add_crate_to_team(&mut conn, &user_b, &not_top_c, &not_the_a_team)?;
188188

189189
let top_crates = TopCrates::new(&mut conn, 2)?;
190190

src/typosquat/test_util.rs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ pub mod faker {
3838
description: &str,
3939
user: &User,
4040
downloads: i32,
41-
) -> anyhow::Result<(Crate, Version)> {
41+
) -> anyhow::Result<Crate> {
4242
let krate = NewCrate {
4343
name,
4444
description: Some(description),
@@ -51,14 +51,14 @@ pub mod faker {
5151
.set(crate_downloads::downloads.eq(downloads as i64))
5252
.execute(conn)?;
5353

54-
let version = NewVersion::builder(krate.id, "1.0.0")
54+
NewVersion::builder(krate.id, "1.0.0")
5555
.published_by(user.id)
5656
.checksum("0000000000000000000000000000000000000000000000000000000000000000")
5757
.build()
5858
.save(conn, "[email protected]")
5959
.unwrap();
6060

61-
Ok((krate, version))
61+
Ok(krate)
6262
}
6363

6464
pub fn team(conn: &mut PgConnection, org: &str, team: &str) -> anyhow::Result<Owner> {

src/worker/jobs/typosquat.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -141,14 +141,14 @@ mod tests {
141141

142142
// Now we'll create new crates: one problematic, one not so.
143143
let other_user = faker::user(&mut conn, "b")?;
144-
let (angel, _version) = faker::crate_and_version(
144+
let angel = faker::crate_and_version(
145145
&mut conn,
146146
"innocent-crate",
147147
"I'm just a simple, innocent crate",
148148
&other_user,
149149
0,
150150
)?;
151-
let (demon, _version) = faker::crate_and_version(
151+
let demon = faker::crate_and_version(
152152
&mut conn,
153153
"mycrate",
154154
"I'm even more innocent, obviously",

0 commit comments

Comments
 (0)