-
Notifications
You must be signed in to change notification settings - Fork 2.7k
feat(coprocessor): implement additional stress-test scenarios #1432
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
d78c71c to
728d9d8
Compare
- add Transaction::ERC7984Transfer - add a simplified variant of the auction::submitEncryptedBids - add sample json data files
b64295e to
2587b4e
Compare
| pub static CONTRACT_STATE: OnceLock<Arc<RwLock<ContractState>>> = OnceLock::new(); | ||
|
|
||
| #[allow(clippy::too_many_arguments)] | ||
| pub async fn batch_submit_encrypted_bids( |
There was a problem hiding this comment.
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); |
There was a problem hiding this comment.
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
| .await?; | ||
|
|
||
| Ok((Handle::default(), Handle::default())) |
There was a problem hiding this comment.
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.
Transaction::ERC7984TransferTransaction::BatchSubmitEncryptedBidsTransaction::BatchAllowHandlesto simulate high-load in L2Transaction::BatchInputProofsscenario