remove needless pass by value

examples/sweeping.rs

@@ -105,7 +105,7 @@ async fn run() -> anyhow::Result<()> {
                 &merchant_account.id,
                 &SetupSweepingRequest {
                     max_amount_in_minor: amount,
-                    currency: merchant_account.currency.clone(),
+                    currency: merchant_account.currency,
                     frequency: frequency.clone(),
                 },
             )

src/apis/payments/model.rs

@@ -123,7 +123,7 @@ pub enum PaymentStatus {
     },
 }
 
-#[derive(Serialize, Deserialize, Debug, Clone, Eq, PartialEq, Hash)]
+#[derive(Serialize, Deserialize, Debug, Clone, Copy, Eq, PartialEq, Hash)]
 #[serde(rename_all = "UPPERCASE")]
 pub enum Currency {
     Gbp,

src/authenticator.rs

@@ -17,10 +17,11 @@ pub struct Authenticator {
 impl Authenticator {
     /// Starts a new authenticator with the given initial credentials.
     pub fn new(client: ClientWithMiddleware, auth_url: Url, credentials: Credentials) -> Self {
+        let client_id = credentials.client_id().to_string();
         let state = AuthenticatorState {
             client,
             auth_url,
-            credentials: credentials.clone(),
+            credentials,
             access_token: None,
         };
 
@@ -36,10 +37,7 @@ impl Authenticator {
             process_loop(state, rx).await;
         });
 
-        Self {
-            tx,
-            client_id: credentials.client_id().into(),
-        }
+        Self { tx, client_id }
     }
 
     /// Returns the current access token used for authentication against the TrueLayer APIs.

src/lib.rs

@@ -137,6 +137,7 @@
 //! ```
 
 #![deny(missing_debug_implementations)]
+#![warn(clippy::needless_pass_by_value)]
 #![forbid(unsafe_code)]
 
 pub mod apis;

tests/common/mock_server/routes.rs

@@ -75,7 +75,7 @@ pub(super) async fn create_payment(
         Payment {
             id: id.clone(),
             amount_in_minor: create_payment_request.amount_in_minor,
-            currency: create_payment_request.currency.clone(),
+            currency: create_payment_request.currency,
             user: user.clone(),
             payment_method: create_payment_request.payment_method.clone(),
             created_at: Utc::now(),
@@ -544,7 +544,7 @@ pub(super) async fn create_payout(
             id: payout_id.clone(),
             merchant_account_id: request.merchant_account_id.clone(),
             amount_in_minor: request.amount_in_minor,
-            currency: request.currency.clone(),
+            currency: request.currency,
             beneficiary: request.beneficiary.clone(),
             created_at: Utc::now(),
             status: PayoutStatus::Pending,

tests/integration_tests/payments.rs

@@ -179,7 +179,7 @@ impl CreatePaymentScenario {
         // Create a payment
         let create_payment_request = CreatePaymentRequest {
             amount_in_minor: 1,
-            currency: self.currency.clone(),
+            currency: self.currency,
             payment_method: PaymentMethod::BankTransfer {
                 provider_selection,
                 beneficiary: match self.beneficiary {