Skip to content

Commit

Permalink
zcash_client_backend: Rename TransactionSummary::new for clarity
Browse files Browse the repository at this point in the history
  • Loading branch information
str4d committed Sep 16, 2024
1 parent bf73a02 commit 45b1063
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion zcash_client_backend/src/data_api/testing.rs
Original file line number Diff line number Diff line change
Expand Up @@ -107,7 +107,7 @@ pub struct TransactionSummary<AccountId> {

impl<AccountId> TransactionSummary<AccountId> {
#[allow(clippy::too_many_arguments)]
pub fn new(
pub fn from_parts(
account_id: AccountId,
txid: TxId,
expiry_height: Option<BlockHeight>,
Expand Down
2 changes: 1 addition & 1 deletion zcash_client_sqlite/src/wallet.rs
Original file line number Diff line number Diff line change
Expand Up @@ -3194,7 +3194,7 @@ pub mod testing {

let results = stmt
.query_and_then::<TransactionSummary<AccountId>, SqliteClientError, _, _>([], |row| {
Ok(TransactionSummary::new(
Ok(TransactionSummary::from_parts(
AccountId(row.get("account_id")?),
TxId::from_bytes(row.get("txid")?),
row.get::<_, Option<u32>>("expiry_height")?
Expand Down

0 comments on commit 45b1063

Please sign in to comment.