Skip to content

Commit

Permalink
fix: increase test timeout (#16305)
Browse files Browse the repository at this point in the history
## Description 

Describe the changes or additions included in this PR.

## Test Plan 

How did you test the new or updated feature?

---
If your changes are not user-facing and do not break anything, you can
skip the following section. Otherwise, please briefly describe what has
changed under the Release Notes section.

### Type of Change (Check all that apply)

- [ ] protocol change
- [ ] user-visible impact
- [ ] breaking change for a client SDKs
- [ ] breaking change for FNs (FN binary must upgrade)
- [ ] breaking change for validators or node operators (must upgrade
binaries)
- [ ] breaking change for on-chain data layout
- [ ] necessitate either a data wipe or data migration

### Release notes
  • Loading branch information
joyqvq authored Feb 21, 2024
1 parent 7d7c8a3 commit c3e39d9
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 5 deletions.
10 changes: 6 additions & 4 deletions crates/sui-e2e-tests/tests/multisig_tests.rs
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@

use fastcrypto::traits::EncodeDecodeBase64;
use shared_crypto::intent::{Intent, IntentMessage};
use std::time::Duration;
use sui_core::authority_client::AuthorityAPI;
use sui_macros::sim_test;
use sui_test_transaction_builder::TestTransactionBuilder;
Expand All @@ -21,7 +22,6 @@ use sui_types::{
zk_login_authenticator::ZkLoginAuthenticator,
};
use test_cluster::{TestCluster, TestClusterBuilder};

async fn do_upgraded_multisig_test() -> SuiResult {
let test_cluster = TestClusterBuilder::new().build().await;
let tx = make_upgraded_multisig_tx();
Expand Down Expand Up @@ -641,7 +641,10 @@ async fn test_expired_epoch_zklogin_in_multisig() {
.with_epoch_duration_ms(1000)
.build()
.await;
test_cluster.wait_for_epoch(Some(11)).await;
test_cluster.wait_for_authenticator_state_update().await;
test_cluster
.wait_for_epoch_with_timeout(Some(11), Duration::from_secs(300))
.await;
let tx = construct_simple_zklogin_multisig_tx(&test_cluster).await;
let res = test_cluster.wallet.execute_transaction_may_fail(tx).await;
assert!(res
Expand Down Expand Up @@ -742,6 +745,7 @@ async fn test_zklogin_inside_multisig_feature_deny() {
config
});
let test_cluster = TestClusterBuilder::new().with_default_jwks().build().await;
test_cluster.wait_for_authenticator_state_update().await;
let tx = construct_simple_zklogin_multisig_tx(&test_cluster).await;
let res = test_cluster.wallet.execute_transaction_may_fail(tx).await;
assert!(res
Expand All @@ -759,8 +763,6 @@ async fn construct_simple_zklogin_multisig_tx(test_cluster: &TestCluster) -> Tra
.unwrap(),
);
let multisig_pk = MultiSigPublicKey::insecure_new(vec![(zklogin_pk.clone(), 1)], 1);

test_cluster.wait_for_authenticator_state_update().await;
let rgp = test_cluster.get_reference_gas_price().await;

let multisig_addr = SuiAddress::from(&multisig_pk);
Expand Down
2 changes: 1 addition & 1 deletion crates/sui-e2e-tests/tests/zklogin_tests.rs
Original file line number Diff line number Diff line change
Expand Up @@ -94,7 +94,7 @@ async fn zklogin_end_to_end_test() {
.build()
.await;
test_cluster
.wait_for_epoch_with_timeout(Some(11), Duration::from_secs(180))
.wait_for_epoch_with_timeout(Some(11), Duration::from_secs(300))
.await;
let rgp = test_cluster.get_reference_gas_price().await;

Expand Down

0 comments on commit c3e39d9

Please sign in to comment.