From 4f0c93dc1a193a08e8797b1072c3d16e9e20de07 Mon Sep 17 00:00:00 2001 From: Usman Saleem Date: Thu, 13 Jul 2023 12:45:36 +1000 Subject: [PATCH 1/2] temporary disable Azure and AWS AT --- .../tests/bulkloading/AwsSecretsManagerAcceptanceTest.java | 2 ++ .../bulkloading/AwsSecretsManagerMultiValueAcceptanceTest.java | 2 ++ .../bulkloading/AwsSecretsManagerPerformanceAcceptanceTest.java | 2 ++ .../tests/bulkloading/AzureKeyVaultAcceptanceTest.java | 2 ++ .../bulkloading/AzureKeyVaultMultiValueAcceptanceTest.java | 2 ++ .../tests/publickeys/AwsKeyIdentifiersAcceptanceTest.java | 2 ++ .../tests/publickeys/KeyIdentifiersAcceptanceTest.java | 2 ++ .../web3signer/tests/signing/BlsSigningAcceptanceTest.java | 2 ++ .../web3signer/tests/signing/SecpSigningAcceptanceTest.java | 2 ++ .../keystorage/aws/AwsSecretsManagerProviderTest.java | 2 ++ .../web3signer/keystorage/aws/AwsSecretsManagerTest.java | 2 ++ .../pegasys/web3signer/keystorage/azure/AzureKeyVaultTest.java | 2 ++ .../signing/secp256k1/azure/AzureKeyVaultSignerTest.java | 2 ++ 13 files changed, 26 insertions(+) diff --git a/acceptance-tests/src/test/java/tech/pegasys/web3signer/tests/bulkloading/AwsSecretsManagerAcceptanceTest.java b/acceptance-tests/src/test/java/tech/pegasys/web3signer/tests/bulkloading/AwsSecretsManagerAcceptanceTest.java index 2f1c8b369..5ee66401c 100644 --- a/acceptance-tests/src/test/java/tech/pegasys/web3signer/tests/bulkloading/AwsSecretsManagerAcceptanceTest.java +++ b/acceptance-tests/src/test/java/tech/pegasys/web3signer/tests/bulkloading/AwsSecretsManagerAcceptanceTest.java @@ -38,12 +38,14 @@ import org.apache.logging.log4j.Logger; import org.junit.jupiter.api.AfterAll; import org.junit.jupiter.api.BeforeAll; +import org.junit.jupiter.api.Disabled; import org.junit.jupiter.api.Test; import org.junit.jupiter.api.TestInstance; import org.junit.jupiter.api.condition.EnabledIfEnvironmentVariable; import org.junit.jupiter.params.ParameterizedTest; import org.junit.jupiter.params.provider.ValueSource; +@Disabled(value = "Temporary Disabled") @EnabledIfEnvironmentVariable( named = "RW_AWS_ACCESS_KEY_ID", matches = ".*", diff --git a/acceptance-tests/src/test/java/tech/pegasys/web3signer/tests/bulkloading/AwsSecretsManagerMultiValueAcceptanceTest.java b/acceptance-tests/src/test/java/tech/pegasys/web3signer/tests/bulkloading/AwsSecretsManagerMultiValueAcceptanceTest.java index d51501654..52f64f27b 100644 --- a/acceptance-tests/src/test/java/tech/pegasys/web3signer/tests/bulkloading/AwsSecretsManagerMultiValueAcceptanceTest.java +++ b/acceptance-tests/src/test/java/tech/pegasys/web3signer/tests/bulkloading/AwsSecretsManagerMultiValueAcceptanceTest.java @@ -37,11 +37,13 @@ import org.apache.logging.log4j.Logger; import org.junit.jupiter.api.AfterAll; import org.junit.jupiter.api.BeforeAll; +import org.junit.jupiter.api.Disabled; import org.junit.jupiter.api.TestInstance; import org.junit.jupiter.api.condition.EnabledIfEnvironmentVariable; import org.junit.jupiter.params.ParameterizedTest; import org.junit.jupiter.params.provider.ValueSource; +@Disabled(value = "Temporary Disabled") @EnabledIfEnvironmentVariable( named = "RW_AWS_ACCESS_KEY_ID", matches = ".*", diff --git a/acceptance-tests/src/test/java/tech/pegasys/web3signer/tests/bulkloading/AwsSecretsManagerPerformanceAcceptanceTest.java b/acceptance-tests/src/test/java/tech/pegasys/web3signer/tests/bulkloading/AwsSecretsManagerPerformanceAcceptanceTest.java index a94378f2f..a8dc507cf 100644 --- a/acceptance-tests/src/test/java/tech/pegasys/web3signer/tests/bulkloading/AwsSecretsManagerPerformanceAcceptanceTest.java +++ b/acceptance-tests/src/test/java/tech/pegasys/web3signer/tests/bulkloading/AwsSecretsManagerPerformanceAcceptanceTest.java @@ -39,6 +39,7 @@ import org.apache.logging.log4j.Logger; import org.junit.jupiter.api.AfterAll; import org.junit.jupiter.api.BeforeAll; +import org.junit.jupiter.api.Disabled; import org.junit.jupiter.api.Test; import org.junit.jupiter.api.TestInstance; import org.junit.jupiter.api.condition.EnabledIfEnvironmentVariable; @@ -47,6 +48,7 @@ NOTE: This AT attempts to create and load large number of keys from AWS Secrets Manager which may take several minutes, hence it should only be manually run in dev/test environment instead of automatically via CI */ +@Disabled(value = "Temporary Disabled") @EnabledIfEnvironmentVariable( named = "AWS_PERF_AT_ENABLED", matches = "true", diff --git a/acceptance-tests/src/test/java/tech/pegasys/web3signer/tests/bulkloading/AzureKeyVaultAcceptanceTest.java b/acceptance-tests/src/test/java/tech/pegasys/web3signer/tests/bulkloading/AzureKeyVaultAcceptanceTest.java index e01776e62..5643f7d6e 100644 --- a/acceptance-tests/src/test/java/tech/pegasys/web3signer/tests/bulkloading/AzureKeyVaultAcceptanceTest.java +++ b/acceptance-tests/src/test/java/tech/pegasys/web3signer/tests/bulkloading/AzureKeyVaultAcceptanceTest.java @@ -30,10 +30,12 @@ import io.vertx.core.json.JsonObject; import org.junit.jupiter.api.Assumptions; import org.junit.jupiter.api.BeforeAll; +import org.junit.jupiter.api.Disabled; import org.junit.jupiter.api.Test; import org.junit.jupiter.params.ParameterizedTest; import org.junit.jupiter.params.provider.ValueSource; +@Disabled(value = "Temporary Disabled") public class AzureKeyVaultAcceptanceTest extends AcceptanceTestBase { private static final String CLIENT_ID = System.getenv("AZURE_CLIENT_ID"); diff --git a/acceptance-tests/src/test/java/tech/pegasys/web3signer/tests/bulkloading/AzureKeyVaultMultiValueAcceptanceTest.java b/acceptance-tests/src/test/java/tech/pegasys/web3signer/tests/bulkloading/AzureKeyVaultMultiValueAcceptanceTest.java index 56843fabf..dc81dccff 100644 --- a/acceptance-tests/src/test/java/tech/pegasys/web3signer/tests/bulkloading/AzureKeyVaultMultiValueAcceptanceTest.java +++ b/acceptance-tests/src/test/java/tech/pegasys/web3signer/tests/bulkloading/AzureKeyVaultMultiValueAcceptanceTest.java @@ -45,9 +45,11 @@ import org.apache.tuweni.bytes.Bytes32; import org.junit.jupiter.api.AfterAll; import org.junit.jupiter.api.BeforeAll; +import org.junit.jupiter.api.Disabled; import org.junit.jupiter.api.Test; import org.junit.jupiter.api.condition.EnabledIfEnvironmentVariable; +@Disabled(value = "Temporary Disabled") @EnabledIfEnvironmentVariable( named = "AZURE_CLIENT_ID", matches = ".*", diff --git a/acceptance-tests/src/test/java/tech/pegasys/web3signer/tests/publickeys/AwsKeyIdentifiersAcceptanceTest.java b/acceptance-tests/src/test/java/tech/pegasys/web3signer/tests/publickeys/AwsKeyIdentifiersAcceptanceTest.java index 1049fe6b8..2cdc94bba 100644 --- a/acceptance-tests/src/test/java/tech/pegasys/web3signer/tests/publickeys/AwsKeyIdentifiersAcceptanceTest.java +++ b/acceptance-tests/src/test/java/tech/pegasys/web3signer/tests/publickeys/AwsKeyIdentifiersAcceptanceTest.java @@ -24,10 +24,12 @@ import io.restassured.response.Response; import org.junit.jupiter.api.AfterAll; import org.junit.jupiter.api.BeforeAll; +import org.junit.jupiter.api.Disabled; import org.junit.jupiter.api.Test; import org.junit.jupiter.api.TestInstance; import org.junit.jupiter.api.condition.EnabledIfEnvironmentVariable; +@Disabled(value = "Temporary Disabled") @TestInstance(TestInstance.Lifecycle.PER_CLASS) @EnabledIfEnvironmentVariable( named = "RW_AWS_ACCESS_KEY_ID", diff --git a/acceptance-tests/src/test/java/tech/pegasys/web3signer/tests/publickeys/KeyIdentifiersAcceptanceTest.java b/acceptance-tests/src/test/java/tech/pegasys/web3signer/tests/publickeys/KeyIdentifiersAcceptanceTest.java index 220dde3aa..ba15f6cbd 100644 --- a/acceptance-tests/src/test/java/tech/pegasys/web3signer/tests/publickeys/KeyIdentifiersAcceptanceTest.java +++ b/acceptance-tests/src/test/java/tech/pegasys/web3signer/tests/publickeys/KeyIdentifiersAcceptanceTest.java @@ -36,6 +36,7 @@ import org.apache.commons.lang3.ArrayUtils; import org.apache.tuweni.bytes.Bytes32; import org.awaitility.Awaitility; +import org.junit.jupiter.api.Disabled; import org.junit.jupiter.api.Test; import org.junit.jupiter.api.condition.EnabledIfEnvironmentVariable; import org.junit.jupiter.api.condition.EnabledIfEnvironmentVariables; @@ -206,6 +207,7 @@ public void allLoadedKeysAreReturnedPublicKeyResponseWithEmptyAccept(final KeyTy validateApiResponse(response, containsInAnyOrder(keys)); } + @Disabled(value = "Temporary Disabled") @Test @EnabledIfEnvironmentVariables({ @EnabledIfEnvironmentVariable(named = "AZURE_CLIENT_ID", matches = ".*"), diff --git a/acceptance-tests/src/test/java/tech/pegasys/web3signer/tests/signing/BlsSigningAcceptanceTest.java b/acceptance-tests/src/test/java/tech/pegasys/web3signer/tests/signing/BlsSigningAcceptanceTest.java index 3be7abd93..7c76cd543 100644 --- a/acceptance-tests/src/test/java/tech/pegasys/web3signer/tests/signing/BlsSigningAcceptanceTest.java +++ b/acceptance-tests/src/test/java/tech/pegasys/web3signer/tests/signing/BlsSigningAcceptanceTest.java @@ -46,6 +46,7 @@ import io.vertx.core.json.JsonObject; import org.apache.tuweni.bytes.Bytes; import org.apache.tuweni.bytes.Bytes32; +import org.junit.jupiter.api.Disabled; import org.junit.jupiter.api.Test; import org.junit.jupiter.api.condition.EnabledIfEnvironmentVariable; import org.junit.jupiter.api.condition.EnabledIfEnvironmentVariables; @@ -154,6 +155,7 @@ public void ableToSignUsingHashicorpWithHttpProtocolOverride( } } + @Disabled(value = "Temporary Disabled") @Test @EnabledIfEnvironmentVariables({ @EnabledIfEnvironmentVariable(named = "AZURE_CLIENT_ID", matches = ".*"), diff --git a/acceptance-tests/src/test/java/tech/pegasys/web3signer/tests/signing/SecpSigningAcceptanceTest.java b/acceptance-tests/src/test/java/tech/pegasys/web3signer/tests/signing/SecpSigningAcceptanceTest.java index 99d88e990..5756736a2 100644 --- a/acceptance-tests/src/test/java/tech/pegasys/web3signer/tests/signing/SecpSigningAcceptanceTest.java +++ b/acceptance-tests/src/test/java/tech/pegasys/web3signer/tests/signing/SecpSigningAcceptanceTest.java @@ -34,6 +34,7 @@ import com.google.common.io.Resources; import io.restassured.response.Response; import org.apache.tuweni.bytes.Bytes; +import org.junit.jupiter.api.Disabled; import org.junit.jupiter.api.Test; import org.junit.jupiter.api.condition.EnabledIfEnvironmentVariable; import org.junit.jupiter.api.condition.EnabledIfEnvironmentVariables; @@ -92,6 +93,7 @@ public void signDataWithKeyFromHashicorp() { } } + @Disabled(value = "Temporary Disabled") @Test @EnabledIfEnvironmentVariables({ @EnabledIfEnvironmentVariable(named = "AZURE_CLIENT_ID", matches = ".*"), diff --git a/keystorage/src/test/java/tech/pegasys/web3signer/keystorage/aws/AwsSecretsManagerProviderTest.java b/keystorage/src/test/java/tech/pegasys/web3signer/keystorage/aws/AwsSecretsManagerProviderTest.java index 9c1355f57..ee41b5787 100644 --- a/keystorage/src/test/java/tech/pegasys/web3signer/keystorage/aws/AwsSecretsManagerProviderTest.java +++ b/keystorage/src/test/java/tech/pegasys/web3signer/keystorage/aws/AwsSecretsManagerProviderTest.java @@ -19,10 +19,12 @@ import org.junit.jupiter.api.AfterEach; import org.junit.jupiter.api.BeforeEach; +import org.junit.jupiter.api.Disabled; import org.junit.jupiter.api.Test; import org.junit.jupiter.api.TestInstance; import org.junit.jupiter.api.condition.EnabledIfEnvironmentVariable; +@Disabled(value = "Temporary disabled") @TestInstance(TestInstance.Lifecycle.PER_CLASS) @EnabledIfEnvironmentVariable( named = "RW_AWS_ACCESS_KEY_ID", diff --git a/keystorage/src/test/java/tech/pegasys/web3signer/keystorage/aws/AwsSecretsManagerTest.java b/keystorage/src/test/java/tech/pegasys/web3signer/keystorage/aws/AwsSecretsManagerTest.java index f608490fd..0f5c61193 100644 --- a/keystorage/src/test/java/tech/pegasys/web3signer/keystorage/aws/AwsSecretsManagerTest.java +++ b/keystorage/src/test/java/tech/pegasys/web3signer/keystorage/aws/AwsSecretsManagerTest.java @@ -29,6 +29,7 @@ import org.assertj.core.api.Assertions; import org.junit.jupiter.api.AfterAll; import org.junit.jupiter.api.BeforeAll; +import org.junit.jupiter.api.Disabled; import org.junit.jupiter.api.Test; import org.junit.jupiter.api.TestInstance; import org.junit.jupiter.api.condition.EnabledIfEnvironmentVariable; @@ -48,6 +49,7 @@ import software.amazon.awssdk.services.secretsmanager.model.UntagResourceRequest; import software.amazon.awssdk.services.secretsmanager.model.UpdateSecretRequest; +@Disabled(value = "Temporary Disabled") @TestInstance(TestInstance.Lifecycle.PER_CLASS) @EnabledIfEnvironmentVariable( named = "RW_AWS_ACCESS_KEY_ID", diff --git a/keystorage/src/test/java/tech/pegasys/web3signer/keystorage/azure/AzureKeyVaultTest.java b/keystorage/src/test/java/tech/pegasys/web3signer/keystorage/azure/AzureKeyVaultTest.java index d60a39253..3828cb96c 100644 --- a/keystorage/src/test/java/tech/pegasys/web3signer/keystorage/azure/AzureKeyVaultTest.java +++ b/keystorage/src/test/java/tech/pegasys/web3signer/keystorage/azure/AzureKeyVaultTest.java @@ -26,8 +26,10 @@ import org.assertj.core.api.Assertions; import org.junit.jupiter.api.Assumptions; import org.junit.jupiter.api.BeforeAll; +import org.junit.jupiter.api.Disabled; import org.junit.jupiter.api.Test; +@Disabled(value = "Temporary Disabled") public class AzureKeyVaultTest { private static final String CLIENT_ID = System.getenv("AZURE_CLIENT_ID"); private static final String CLIENT_SECRET = System.getenv("AZURE_CLIENT_SECRET"); diff --git a/signing/src/test/java/tech/pegasys/web3signer/signing/secp256k1/azure/AzureKeyVaultSignerTest.java b/signing/src/test/java/tech/pegasys/web3signer/signing/secp256k1/azure/AzureKeyVaultSignerTest.java index a4e3b3e76..c647782f4 100644 --- a/signing/src/test/java/tech/pegasys/web3signer/signing/secp256k1/azure/AzureKeyVaultSignerTest.java +++ b/signing/src/test/java/tech/pegasys/web3signer/signing/secp256k1/azure/AzureKeyVaultSignerTest.java @@ -27,12 +27,14 @@ import org.assertj.core.api.Assertions; import org.junit.jupiter.api.Assumptions; import org.junit.jupiter.api.BeforeAll; +import org.junit.jupiter.api.Disabled; import org.junit.jupiter.api.Test; import org.web3j.crypto.Hash; import org.web3j.crypto.Sign; import org.web3j.crypto.Sign.SignatureData; import org.web3j.utils.Numeric; +@Disabled(value = "Temporary Disabled") public class AzureKeyVaultSignerTest { private static final String clientId = System.getenv("AZURE_CLIENT_ID"); private static final String clientSecret = System.getenv("AZURE_CLIENT_SECRET"); From 97257d0b1a29b38e69ee6040b3ebe4b7bd143698 Mon Sep 17 00:00:00 2001 From: Gabriel Fukushima Date: Thu, 13 Jul 2023 13:19:27 +1000 Subject: [PATCH 2/2] Disabling aws Signed-off-by: Gabriel Fukushima --- .../web3signer/tests/signing/BlsSigningAcceptanceTest.java | 1 + 1 file changed, 1 insertion(+) diff --git a/acceptance-tests/src/test/java/tech/pegasys/web3signer/tests/signing/BlsSigningAcceptanceTest.java b/acceptance-tests/src/test/java/tech/pegasys/web3signer/tests/signing/BlsSigningAcceptanceTest.java index 7c76cd543..13f3b9fd2 100644 --- a/acceptance-tests/src/test/java/tech/pegasys/web3signer/tests/signing/BlsSigningAcceptanceTest.java +++ b/acceptance-tests/src/test/java/tech/pegasys/web3signer/tests/signing/BlsSigningAcceptanceTest.java @@ -178,6 +178,7 @@ public void ableToSignUsingAzure() throws JsonProcessingException { signAndVerifySignature(ArtifactType.BLOCK); } + @Disabled(value = "Temporary Disabled") @Test @EnabledIfEnvironmentVariables({ @EnabledIfEnvironmentVariable(