Skip to content

Conversation

@goshawk-3
Copy link
Contributor

@goshawk-3 goshawk-3 commented Dec 2, 2025

  • add Transaction::ERC7984Transfer
  • add a simplified variant of the Transaction::BatchSubmitEncryptedBids
  • add Transaction::BatchAllowHandles to simulate high-load in L2
  • add sample json data files
  • add Transaction::BatchInputProofs scenario

@cla-bot cla-bot bot added the cla-signed label Dec 2, 2025
@goshawk-3 goshawk-3 force-pushed the georgi/stress-test/add-scenarios branch from d78c71c to 728d9d8 Compare December 2, 2025 13:02
@goshawk-3 goshawk-3 marked this pull request as ready for review December 2, 2025 16:18
@goshawk-3 goshawk-3 requested a review from a team as a code owner December 2, 2025 16:18
- add Transaction::ERC7984Transfer
- add a simplified variant of the auction::submitEncryptedBids
- add sample json data files
@goshawk-3 goshawk-3 force-pushed the georgi/stress-test/add-scenarios branch from b64295e to 2587b4e Compare December 5, 2025 06:25
pub static CONTRACT_STATE: OnceLock<Arc<RwLock<ContractState>>> = OnceLock::new();

#[allow(clippy::too_many_arguments)]
pub async fn batch_submit_encrypted_bids(
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

note: in new auction contract (that will be deployed and currently in development) there is no more batch tx but one tx = one bid

let mut user_submitted_bids = vec![];

for e_amount in bids.iter() {
let bid_price = 1; // rand::random_range(1..=1000);
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: remove comment or match it to value?
I do not think it has an impact

Comment on lines +605 to +607
.await?;

Ok((Handle::default(), Handle::default()))
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

there is no tx.commit() compared to other cases.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants