Skip to content

Commit 3dff767

Browse files
committed
worker/jobs/expiry_notification: Avoid NewUser::create_or_update() fn
We don't need to send emails here, so this isn't needed.
1 parent fbd8899 commit 3dff767

File tree

1 file changed

+10
-5
lines changed

1 file changed

+10
-5
lines changed

src/worker/jobs/expiry_notification.rs

Lines changed: 10 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -166,7 +166,7 @@ The crates.io team"#,
166166
#[cfg(test)]
167167
mod tests {
168168
use super::*;
169-
use crate::models::NewUser;
169+
use crate::models::{NewEmail, NewUser};
170170
use crate::{models::token::ApiToken, schema::api_tokens, util::token::PlainToken};
171171
use crates_io_test_db::TestDatabase;
172172
use diesel::dsl::IntervalDsl;
@@ -182,10 +182,15 @@ mod tests {
182182
.gh_id(0)
183183
.gh_login("a")
184184
.gh_access_token("token")
185-
.build();
186-
let emails = Emails::new_in_memory();
187-
let user = user
188-
.create_or_update(Some("testuser@test.com"), &emails, &mut conn)
185+
.build()
186+
.insert(&mut conn)
187+
.await?;
188+
189+
NewEmail::builder()
190+
.user_id(user.id)
191+
.email("testuser@test.com")
192+
.build()
193+
.insert(&mut conn)
189194
.await?;
190195

191196
let token = PlainToken::generate();

0 commit comments

Comments
 (0)