m

Author: Lucas McDonald

DynamoDbEncryption/runtimes/python/test/requests.py

@@ -142,15 +142,15 @@ def basic_batch_execute_statement_request_plaintext_table():
 
 # No exhaustive requests are intended to be able to be used as real requests.
 # Some parameters conflict with each other when sent to DynamoDB.
-# These are only intended to test the conversion of the structure from client to resource format.
+# These are only intended to test the conversion of the structure between client and resource formats.
 
 
 def base_exhaustive_put_item_request(item):
     """
     Base structure for exhaustive put_item requests.
     This is not intended to be able to be used as a real request.
     Some parameters conflict with each other when sent to DynamoDB.
-    This is only intended to test the conversion of the request from client to resource format.
+    This is only intended to test the conversion of the request between client and resource formats.
     """
     return {
         # Expected is legacy, but still in the boto3 docs.
@@ -174,7 +174,7 @@ def base_exhaustive_get_item_request(item):
     Base structure for exhaustive get_item requests.
     This is not intended to be able to be used as a real request.
     Some parameters conflict with each other when sent to DynamoDB.
-    This is only intended to test the conversion of the request from client to resource format.
+    This is only intended to test the conversion of the request between client and resource formats.
     """
     return {
         "ReturnConsumedCapacity": "TOTAL",
@@ -196,7 +196,7 @@ def base_exhaustive_delete_item_request(item):
     Base structure for exhaustive delete_item requests.
     This is not intended to be able to be used as a real request.
     Some parameters conflict with each other when sent to DynamoDB.
-    This is only intended to test the conversion of the request from client to resource format.
+    This is only intended to test the conversion of the request between client and resource formats.
     """
     return {
         "ReturnConsumedCapacity": "TOTAL",
@@ -211,7 +211,7 @@ def base_exhaustive_query_request(item):
     Base structure for exhaustive query requests.
     This is not intended to be able to be used as a real request.
     Some parameters conflict with each other when sent to DynamoDB.
-    This is only intended to test the conversion of the request from client to resource format.
+    This is only intended to test the conversion of the request between client and resource formats.
     """
     return {
         "IndexName": "index_name",
@@ -240,7 +240,7 @@ def base_exhaustive_scan_request(item):
     Base structure for exhaustive scan requests.
     This is not intended to be able to be used as a real request.
     Some parameters conflict with each other when sent to DynamoDB.
-    This is only intended to test the conversion of the request from client to resource format.
+    This is only intended to test the conversion of the request between client and resource formats.
     """
     return {
         "IndexName": "index_name",
@@ -310,7 +310,7 @@ def exhaustive_query_request_ddb(item):
     Query request with all possible parameters.
     This is not intended to be able to be used as a real request.
     Some parameters conflict with each other when sent to DynamoDB.
-    This is only intended to test the conversion of the request from client to resource format.
+    This is only intended to test the conversion of the request between client and resource formats.
     """
     base = basic_query_request_ddb(item)
     additional_keys = base_exhaustive_query_request(item)
@@ -427,7 +427,7 @@ def exhaustive_put_item_request_dict(item):
     Get a put_item request in dict format for any item.
     This is not intended to be able to be used as a real request.
     Some parameters conflict with each other when sent to DynamoDB.
-    This is only intended to test the conversion of the request from client to resource format.
+    This is only intended to test the conversion of the request between client and resource formats.
     """
     base = basic_put_item_request_dict(item)
     # Replace the default ConditionExpression string with a ConditionExpression object
@@ -452,7 +452,7 @@ def exhaustive_get_item_request_dict(item):
     Get a get_item request in dict format for any item.
     This is not intended to be able to be used as a real request.
     Some parameters conflict with each other when sent to DynamoDB.
-    This is only intended to test the conversion of the request from client to resource format.
+    This is only intended to test the conversion of the request between client and resource formats.
     """
     base = basic_get_item_request_dict(item)
     additional_keys = base_exhaustive_get_item_request(item)
@@ -478,7 +478,7 @@ def exhaustive_query_request_dict(item):
     Get a query request in dict format for any item.
     This is not intended to be able to be used as a real request.
     Some parameters conflict with each other when sent to DynamoDB.
-    This is only intended to test the conversion of the request from client to resource format.
+    This is only intended to test the conversion of the request between client and resource formats.
     """
     base = basic_query_request_dict(item)
     additional_keys = base_exhaustive_query_request(item)
@@ -495,7 +495,7 @@ def exhaustive_scan_request_dict(item):
     Get a scan request in dict format for any item.
     This is not intended to be able to be used as a real request.
     Some parameters conflict with each other when sent to DynamoDB.
-    This is only intended to test the conversion of the request from client to resource format.
+    This is only intended to test the conversion of the request between client and resource formats.
     """
     base = basic_scan_request_dict(item)
     additional_keys = base_exhaustive_scan_request(item)

DynamoDbEncryption/runtimes/python/test/responses.py

@@ -13,7 +13,7 @@ def exhaustive_put_item_response(item):
     Get a put_item response in resource (ddb) format for any item.
     This is not intended to be a real response that DynamoDB would return,
     but the response should contain additional attributes that DynamoDB could return.
-    This is only intended to exhaustively test the conversion of the request from client to resource format.
+    This is only intended to exhaustively test the conversion of the request between client and resource formats.
     """
     base = basic_put_item_response(item)
     additional_keys = {
@@ -38,7 +38,7 @@ def exhaustive_get_item_response(item):
     Get a get_item response in resource (ddb) format for any item.
     This is not intended to be a real response that DynamoDB would return,
     but the response should contain additional attributes that DynamoDB could return.
-    This is only intended to exhaustively test the conversion of the request from client to resource format.
+    This is only intended to exhaustively test the conversion of the request between client and resource formats.
     """
     base = basic_get_item_response(item)
     additional_keys = {
@@ -62,7 +62,7 @@ def exhaustive_query_response(items):
     Get a query response in resource (ddb) format for any items.
     This is not intended to be a real response that DynamoDB would return,
     but the response should contain additional attributes that DynamoDB could return.
-    This is only intended to exhaustively test the conversion of the request from client to resource format.
+    This is only intended to exhaustively test the conversion of the request between client and resource formats.
     """
     base = basic_query_response(items)
     additional_keys = {
@@ -83,7 +83,7 @@ def exhaustive_scan_response(items, keys):
     Get a scan response in resource (ddb) format for any items.
     This is not intended to be a real response that DynamoDB would return,
     but the response should contain additional attributes that DynamoDB could return.
-    This is only intended to exhaustively test the conversion of the request from client to resource format.
+    This is only intended to exhaustively test the conversion of the request between client and resource formats.
     """
     base = basic_scan_response(items, keys)
     additional_keys = {
@@ -105,7 +105,7 @@ def exhaustive_batch_get_item_response(items):
     Get a batch_get_item response in resource (ddb) format for any items.
     This is not intended to be a real response that DynamoDB would return,
     but the response should contain additional attributes that DynamoDB could return.
-    This is only intended to exhaustively test the conversion of the request from client to resource format.
+    This is only intended to exhaustively test the conversion of the request between client and resource formats.
     """
     base = basic_batch_get_item_response(items)
     additional_keys = {
@@ -130,7 +130,7 @@ def exhaustive_batch_write_item_put_response(items):
     Get a batch_write_item response in resource (ddb) format for any items.
     This is not intended to be a real response that DynamoDB would return,
     but the response should contain additional attributes that DynamoDB could return.
-    This is only intended to exhaustively test the conversion of the request from client to resource format.
+    This is only intended to exhaustively test the conversion of the request between client and resource formats.
     """
     base = basic_batch_write_item_put_response(items)
     additional_keys = {

DynamoDbEncryption/runtimes/python/test/unit/internal/test_client_to_resource.py

@@ -1,9 +1,9 @@
 # Copyright Amazon.com Inc. or its affiliates. All Rights Reserved.
 # SPDX-License-Identifier: Apache-2.0
 import pytest
+from boto3.dynamodb.conditions import ConditionExpressionBuilder
 
 from aws_dbesdk_dynamodb.internal.client_to_resource import ClientShapeToResourceShapeConverter
-from boto3.dynamodb.conditions import ConditionExpressionBuilder
 
 from ...items import (
     complex_item_ddb,

DynamoDbEncryption/runtimes/python/tox.ini

@@ -95,7 +95,7 @@ deps =
 basepython = python3
 commands =
     {[testenv:blacken]commands}
-    {[testenv:ruff]commands}
+    {[testenv:ruff]commands} --fix
 
 [testenv:lint-check]
 commands_pre =
@@ -105,4 +105,4 @@ deps =
 basepython = python3
 commands =
     {[testenv:black-check]commands}
-    {[testenv:ruff]commands} --fix
+    {[testenv:ruff]commands}