Skip to content

Commit

Permalink
chore(iota-e2e-tests): Disable zklogin tests (#2223)
Browse files Browse the repository at this point in the history
* chore(iota-e2e-tests): Disable zklogin tests

* Ignore a few more and fix a test
  • Loading branch information
DaughterOfMars authored Sep 4, 2024
1 parent 27f2d30 commit 8c0d728
Show file tree
Hide file tree
Showing 3 changed files with 15 additions and 2 deletions.
6 changes: 6 additions & 0 deletions crates/iota-e2e-tests/tests/multisig_tests.rs
Original file line number Diff line number Diff line change
Expand Up @@ -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();
Expand Down Expand Up @@ -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)
Expand Down Expand Up @@ -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)
Expand All @@ -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];
Expand Down Expand Up @@ -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;
Expand Down Expand Up @@ -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;

Expand Down
2 changes: 1 addition & 1 deletion crates/iota-e2e-tests/tests/reconfiguration_tests.rs
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down
9 changes: 8 additions & 1 deletion crates/iota-e2e-tests/tests/zklogin_tests.rs
Original file line number Diff line number Diff line change
Expand Up @@ -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;

Expand All @@ -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;

Expand All @@ -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| {
Expand All @@ -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)
Expand Down Expand Up @@ -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)
Expand Down Expand Up @@ -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()
Expand All @@ -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())
Expand Down Expand Up @@ -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,
Expand Down

0 comments on commit 8c0d728

Please sign in to comment.