The RSA keyring now puts requirements on the RSA Keyring

Author: seebees

DynamoDbEncryption/runtimes/java/src/test/java/software/aws/cryptography/dynamoDbEncryption/OtherTests.java

@@ -165,23 +165,25 @@ public void TestBuildHierarchyKeyringWithSupplier() {
     }
      */
 
-    @Test
-    public void TestBuildEncryptDecryptAwsKmsRsaKeyring() {
-        MaterialProviders matProv = MaterialProviders.builder()
-                .MaterialProvidersConfig(MaterialProvidersConfig.builder().build())
-                .build();
-
-        ByteBuffer key = ByteBuffer.wrap(new byte[32]);
-
-        CreateAwsKmsRsaKeyringInput input = CreateAwsKmsRsaKeyringInput.builder()
-                .kmsClient(KmsClient.create())
-                .kmsKeyId(KMS_TEST_KEY_ID)
-                .encryptionAlgorithm(RSAES_OAEP_SHA_1)
-                .publicKey(key)
-                .build();
-        IKeyring keyring = matProv.CreateAwsKmsRsaKeyring(input);
-        assertNotNull(keyring);
-    }
+    // This test can no longer pass.
+    // Because there is now a Modulus length check on the public key.
+//    @Test
+//    public void TestBuildEncryptDecryptAwsKmsRsaKeyring() {
+//        MaterialProviders matProv = MaterialProviders.builder()
+//                .MaterialProvidersConfig(MaterialProvidersConfig.builder().build())
+//                .build();
+//
+//        ByteBuffer key = ByteBuffer.wrap(new byte[32]);
+//
+//        CreateAwsKmsRsaKeyringInput input = CreateAwsKmsRsaKeyringInput.builder()
+//                .kmsClient(KmsClient.create())
+//                .kmsKeyId(KMS_TEST_KEY_ID)
+//                .encryptionAlgorithm(RSAES_OAEP_SHA_1)
+//                .publicKey(key)
+//                .build();
+//        IKeyring keyring = matProv.CreateAwsKmsRsaKeyring(input);
+//        assertNotNull(keyring);
+//    }
 
     class TestSupplier implements BranchKeyFromItemSupplier {
         public String getBranchKeyFromItem(GetBranchKeyFromItemInput input) {