From 8c0d728705e5c3d9514ef264573ec739f3f75a35 Mon Sep 17 00:00:00 2001 From: DaughterOfMars Date: Wed, 4 Sep 2024 03:46:48 -0400 Subject: [PATCH] chore(iota-e2e-tests): Disable zklogin tests (#2223) * chore(iota-e2e-tests): Disable zklogin tests * Ignore a few more and fix a test --- crates/iota-e2e-tests/tests/multisig_tests.rs | 6 ++++++ crates/iota-e2e-tests/tests/reconfiguration_tests.rs | 2 +- crates/iota-e2e-tests/tests/zklogin_tests.rs | 9 ++++++++- 3 files changed, 15 insertions(+), 2 deletions(-) diff --git a/crates/iota-e2e-tests/tests/multisig_tests.rs b/crates/iota-e2e-tests/tests/multisig_tests.rs index 00c8feccf7b..a184ca30084 100644 --- a/crates/iota-e2e-tests/tests/multisig_tests.rs +++ b/crates/iota-e2e-tests/tests/multisig_tests.rs @@ -22,6 +22,7 @@ use iota_types::{ }; use shared_crypto::intent::{Intent, IntentMessage}; use test_cluster::{TestCluster, TestClusterBuilder}; + async fn do_upgraded_multisig_test() -> IotaResult { let test_cluster = TestClusterBuilder::new().build().await; let tx = make_upgraded_multisig_tx(); @@ -179,6 +180,7 @@ async fn test_multisig_e2e() { } #[sim_test] +#[ignore = "https://github.com/iotaledger/iota/issues/1777"] async fn test_multisig_with_zklogin_scenerios() { let test_cluster = TestClusterBuilder::new() .with_epoch_duration_ms(15000) @@ -669,6 +671,7 @@ async fn test_multisig_with_zklogin_scenerios() { } #[sim_test] +#[ignore = "https://github.com/iotaledger/iota/issues/1777"] async fn test_expired_epoch_zklogin_in_multisig() { let test_cluster = TestClusterBuilder::new() .with_epoch_duration_ms(10000) @@ -688,6 +691,7 @@ async fn test_expired_epoch_zklogin_in_multisig() { } #[sim_test] +#[ignore = "https://github.com/iotaledger/iota/issues/1777"] async fn test_random_zklogin_in_multisig() { let test_vectors = &load_test_vectors("../iota-types/src/unit_tests/zklogin_test_vectors.json") .unwrap()[1..11]; @@ -734,6 +738,7 @@ async fn test_random_zklogin_in_multisig() { let tx = Transaction::from_generic_sig_data(tx_data.clone(), vec![multisig]); let _ = context.execute_transaction_must_succeed(tx).await; } + #[sim_test] async fn test_multisig_legacy_works() { let test_cluster = TestClusterBuilder::new().build().await; @@ -771,6 +776,7 @@ async fn test_multisig_legacy_works() { } #[sim_test] +#[ignore = "https://github.com/iotaledger/iota/issues/1777"] async fn test_zklogin_inside_multisig_feature_deny() { use iota_protocol_config::ProtocolConfig; diff --git a/crates/iota-e2e-tests/tests/reconfiguration_tests.rs b/crates/iota-e2e-tests/tests/reconfiguration_tests.rs index 2aef29365bf..20b1ff758f6 100644 --- a/crates/iota-e2e-tests/tests/reconfiguration_tests.rs +++ b/crates/iota-e2e-tests/tests/reconfiguration_tests.rs @@ -724,7 +724,7 @@ async fn do_test_reconfig_with_committee_change_stress() { .fullnode_handle .iota_node .with(|node| node.state().epoch_store_for_testing().committee().clone()); - assert_eq!(committee.num_members(), 7); + assert_eq!(committee.num_members(), 9); assert!(committee.authority_exists(&handle1.state().name)); assert!(committee.authority_exists(&handle2.state().name)); removed_validators diff --git a/crates/iota-e2e-tests/tests/zklogin_tests.rs b/crates/iota-e2e-tests/tests/zklogin_tests.rs index 4d2caf5a86f..f9f84aefc38 100644 --- a/crates/iota-e2e-tests/tests/zklogin_tests.rs +++ b/crates/iota-e2e-tests/tests/zklogin_tests.rs @@ -39,6 +39,7 @@ async fn do_zklogin_test(address: IotaAddress, legacy: bool) -> IotaResult { } #[sim_test] +#[ignore = "https://github.com/iotaledger/iota/issues/1777"] async fn test_zklogin_feature_deny() { use iota_protocol_config::ProtocolConfig; @@ -55,6 +56,7 @@ async fn test_zklogin_feature_deny() { } #[sim_test] +#[ignore = "https://github.com/iotaledger/iota/issues/1777"] async fn test_zklogin_feature_legacy_address_deny() { use iota_protocol_config::ProtocolConfig; @@ -70,6 +72,7 @@ async fn test_zklogin_feature_legacy_address_deny() { } #[sim_test] +#[ignore = "https://github.com/iotaledger/iota/issues/1777"] async fn test_legacy_zklogin_address_accept() { use iota_protocol_config::ProtocolConfig; let _guard = ProtocolConfig::apply_overrides_for_testing(|_, mut config| { @@ -85,6 +88,7 @@ async fn test_legacy_zklogin_address_accept() { } #[sim_test] +#[ignore = "https://github.com/iotaledger/iota/issues/1777"] async fn zklogin_end_to_end_test() { let test_cluster = TestClusterBuilder::new() .with_epoch_duration_ms(15000) @@ -139,6 +143,7 @@ async fn zklogin_end_to_end_test() { } #[sim_test] +#[ignore = "https://github.com/iotaledger/iota/issues/1777"] async fn test_expired_zklogin_sig() { let test_cluster = TestClusterBuilder::new() .with_epoch_duration_ms(10000) @@ -189,6 +194,7 @@ async fn test_expired_zklogin_sig() { } #[sim_test] +#[ignore = "https://github.com/iotaledger/iota/issues/1777"] async fn test_auth_state_creation() { // Create test cluster without auth state object in genesis let test_cluster = TestClusterBuilder::new() @@ -206,6 +212,7 @@ async fn test_auth_state_creation() { } #[sim_test] +#[ignore = "https://github.com/iotaledger/iota/issues/1777"] async fn test_create_authenticator_state_object() { let test_cluster = TestClusterBuilder::new() .with_protocol_version(23.into()) @@ -247,8 +254,8 @@ async fn test_create_authenticator_state_object() { // This test is intended to look for forks caused by conflicting / repeated JWK // votes from validators. -#[cfg(msim)] #[sim_test] +#[ignore = "https://github.com/iotaledger/iota/issues/1777"] async fn test_conflicting_jwks() { use std::{ collections::HashSet,