sync

Author: Lucas McDonald

DynamoDbEncryption/runtimes/python/src/aws_dbesdk_dynamodb/encrypted/client.py

@@ -421,7 +421,7 @@ def update_item(self, **kwargs):
 
     def execute_statement(self, **kwargs):
         """
-        Calls ``execute_statement`` on the underlying client if the table is not configured for encryption.
+        Call ``execute_statement`` on the underlying client if the table is not configured for encryption.
 
         If the table is configured for encryption, this operation will raise DynamoDbEncryptionTransformsException.
 
@@ -455,7 +455,7 @@ def execute_statement(self, **kwargs):
 
     def execute_transaction(self, **kwargs):
         """
-        Calls ``execute_transaction`` on the underlying client if the table is not configured for encryption.
+        Call ``execute_transaction`` on the underlying client if the table is not configured for encryption.
 
         If the table is configured for encryption, this operation will raise DynamoDbEncryptionTransformsException.
 
@@ -489,7 +489,7 @@ def execute_transaction(self, **kwargs):
 
     def batch_execute_statement(self, **kwargs):
         """
-        Calls ``batch_execute_statement`` on the underlying client if the table is not configured for encryption.
+        Call ``batch_execute_statement`` on the underlying client if the table is not configured for encryption.
 
         If the table is configured for encryption, this operation will raise DynamoDbEncryptionTransformsException.
 
@@ -498,8 +498,8 @@ def batch_execute_statement(self, **kwargs):
         https://boto3.amazonaws.com/v1/documentation/api/latest/reference/services/dynamodb/client/batch_execute_statement.html
 
         Args:
-            **kwargs: Keyword arguments to pass to the operation. This matches the boto3 client ``batch_execute_statement``
-                request syntax.
+            **kwargs: Keyword arguments to pass to the operation. This matches the boto3 client
+                ``batch_execute_statement`` request syntax.
 
         Returns:
             dict: The response from DynamoDB. This matches the boto3 client ``batch_execute_statement`` response syntax.

DynamoDbEncryption/runtimes/python/test/integ/encrypted/test_client.py

@@ -24,6 +24,8 @@
     simple_key_dict,
 )
 from ...requests import (
+    basic_batch_execute_statement_request_encrypted_table,
+    basic_batch_execute_statement_request_plaintext_table,
     basic_batch_get_item_request_ddb,
     basic_batch_get_item_request_dict,
     basic_batch_write_item_delete_request_ddb,
@@ -36,8 +38,6 @@
     basic_execute_statement_request_plaintext_table,
     basic_execute_transaction_request_encrypted_table,
     basic_execute_transaction_request_plaintext_table,
-    basic_batch_execute_statement_request_encrypted_table,
-    basic_batch_execute_statement_request_plaintext_table,
     basic_get_item_request_ddb,
     basic_get_item_request_dict,
     basic_put_item_request_ddb,
@@ -435,6 +435,7 @@ def test_WHEN_update_item_with_signed_attribute_THEN_raises_DynamoDbEncryptionTr
 def execute_uses_encrypted_table(request):
     return request.param
 
+
 @pytest.fixture
 def execute_statement_request(execute_uses_encrypted_table):
     if execute_uses_encrypted_table:
@@ -461,6 +462,7 @@ def test_WHEN_execute_statement_for_encrypted_table_THEN_raises_DynamoDbEncrypti
     else:
         pytest.skip("Skipping test for plaintext table; this test is only for encrypted tables")
 
+
 def test_WHEN_execute_statement_for_plaintext_table_THEN_passes(
     client,
     execute_statement_request,
@@ -475,6 +477,7 @@ def test_WHEN_execute_statement_for_plaintext_table_THEN_passes(
     # Then: Success
     assert response["ResponseMetadata"]["HTTPStatusCode"] == 200
 
+
 @pytest.fixture
 def execute_transaction_request(execute_uses_encrypted_table, test_item):
     if execute_uses_encrypted_table:
@@ -501,6 +504,7 @@ def test_WHEN_execute_transaction_for_encrypted_table_THEN_raises_DynamoDbEncryp
     else:
         pytest.skip("Skipping test for plaintext table; this test is only for encrypted tables")
 
+
 def test_WHEN_execute_transaction_for_plaintext_table_THEN_passes(
     client,
     execute_transaction_request,
@@ -519,6 +523,7 @@ def test_WHEN_execute_transaction_for_plaintext_table_THEN_passes(
     # Then: Success
     assert response["ResponseMetadata"]["HTTPStatusCode"] == 200
 
+
 @pytest.fixture
 def batch_execute_statement_request(execute_uses_encrypted_table):
     if execute_uses_encrypted_table:
@@ -545,6 +550,7 @@ def test_WHEN_batch_execute_statement_for_encrypted_table_THEN_raises_DynamoDbEn
     else:
         pytest.skip("Skipping test for plaintext table; this test is only for encrypted tables")
 
+
 def test_WHEN_batch_execute_statement_for_plaintext_table_THEN_passes(
     client,
     batch_execute_statement_request,
@@ -559,6 +565,7 @@ def test_WHEN_batch_execute_statement_for_plaintext_table_THEN_passes(
     # Then: Success
     assert response["ResponseMetadata"]["HTTPStatusCode"] == 200
 
+
 def test_WHEN_get_paginator_THEN_correct_paginator_is_returned():
     """Test get_paginator for scan and query operations."""
     # Given: Encrypted client