diff --git a/src/configuration.rs b/src/configuration.rs index f50b48b..ba409cc 100644 --- a/src/configuration.rs +++ b/src/configuration.rs @@ -41,7 +41,7 @@ impl DatabaseSettings { PgConnectOptions::new() .host(&self.host) .username(&self.username) - .password(&self.password.expose_secret()) + .password(self.password.expose_secret()) .port(self.port) .ssl_mode(ssl_mode) } diff --git a/src/domain/subscriber_name.rs b/src/domain/subscriber_name.rs index b3c6540..f3c0e77 100644 --- a/src/domain/subscriber_name.rs +++ b/src/domain/subscriber_name.rs @@ -72,7 +72,7 @@ mod tests { #[test] fn names_containing_an_invalid_character_are_rejected() { - for name in vec!['/', '(', ')', '"', '<', '>', '\\', '{', '}'] { + for name in &['/', '(', ')', '"', '<', '>', '\\', '{', '}'] { let name = name.to_string(); assert_err!(SubscriberName::parse(name)); } diff --git a/tests/api/helpers.rs b/tests/api/helpers.rs index 672728c..f88784c 100644 --- a/tests/api/helpers.rs +++ b/tests/api/helpers.rs @@ -62,8 +62,8 @@ impl TestApp { confirmation_link }; - let html = get_link(&body["HtmlBody"].as_str().unwrap()); - let plain_text = get_link(&body["TextBody"].as_str().unwrap()); + let html = get_link(body["HtmlBody"].as_str().unwrap()); + let plain_text = get_link(body["TextBody"].as_str().unwrap()); ConfirmationLinks { html, plain_text } } } diff --git a/tests/api/subscriptions.rs b/tests/api/subscriptions.rs index b659024..efd4551 100644 --- a/tests/api/subscriptions.rs +++ b/tests/api/subscriptions.rs @@ -78,7 +78,7 @@ async fn subscribe_sends_a_confirmation_email_with_a_link() { // Assert let email_request = &app.email_server.received_requests().await.unwrap()[0]; - let confirmation_links = app.get_confirmation_links(&email_request); + let confirmation_links = app.get_confirmation_links(email_request); // The two links should be identical assert_eq!(confirmation_links.html, confirmation_links.plain_text); diff --git a/tests/api/subscriptions_confirm.rs b/tests/api/subscriptions_confirm.rs index 23709bc..cda5de1 100644 --- a/tests/api/subscriptions_confirm.rs +++ b/tests/api/subscriptions_confirm.rs @@ -30,7 +30,7 @@ async fn the_link_returned_by_subscribe_returns_a_200_if_called() { app.post_subscriptions(body.into()).await; let email_request = &app.email_server.received_requests().await.unwrap()[0]; - let confirmation_links = app.get_confirmation_links(&email_request); + let confirmation_links = app.get_confirmation_links(email_request); // Act let response = reqwest::get(confirmation_links.html).await.unwrap(); @@ -53,7 +53,7 @@ async fn clicking_on_the_confirmation_link_confirms_a_subscriber() { app.post_subscriptions(body.into()).await; let email_request = &app.email_server.received_requests().await.unwrap()[0]; - let confirmation_links = app.get_confirmation_links(&email_request); + let confirmation_links = app.get_confirmation_links(email_request); // Act reqwest::get(confirmation_links.html)