chore: poetry update and minor fixes (#788)

---------

Co-authored-by: Ran Isenberg <[email protected]>

Author: ran-isenberg

.pre-commit-config.yaml

@@ -26,7 +26,7 @@ repos:
         exclude: "^(?!helpers/)"
   - repo: https://github.com/astral-sh/ruff-pre-commit
     # Ruff version.
-    rev: v0.1.8
+    rev: v0.1.13
     hooks:
       # Run the Ruff linter.
       - id: ruff

cdk/service/configuration/configuration_construct.py

@@ -1,7 +1,7 @@
 from pathlib import Path
 
 import aws_cdk.aws_appconfig_alpha as appconfig
-from aws_cdk import Duration, RemovalPolicy
+from aws_cdk import Duration
 from constructs import Construct
 
 from cdk.service.configuration.schema import FeatureFlagsConfiguration
@@ -32,17 +32,13 @@ def __init__(self, scope: Construct, id_: str, environment: str, service_name: s
             name=self.app_name,
         )
 
-        self.config_app.apply_removal_policy(RemovalPolicy.DESTROY)
-
         self.config_env = appconfig.Environment(
             self,
             id=f'{id_}env',
             application=self.config_app,
             name=environment,
         )
 
-        self.config_env.apply_removal_policy(RemovalPolicy.DESTROY)
-
         # zero minutes, zero bake, 100 growth all at once
         self.config_dep_strategy = appconfig.DeploymentStrategy(
             self,
@@ -54,8 +50,6 @@ def __init__(self, scope: Construct, id_: str, environment: str, service_name: s
             ),
         )
 
-        self.config_dep_strategy.apply_removal_policy(RemovalPolicy.DESTROY)
-
         self.config = appconfig.HostedConfiguration(
             self,
             f'{id_}version',
@@ -67,9 +61,6 @@ def __init__(self, scope: Construct, id_: str, environment: str, service_name: s
             deploy_to=[self.config_env],
         )
 
-        # workaround until https://github.com/aws/aws-cdk/issues/26804 is resolved
-        self.config.node.default_child.apply_removal_policy(RemovalPolicy.DESTROY)  # type: ignore
-
     def _get_and_validate_configuration(self, environment: str) -> str:
         current = Path(__file__).parent
         conf_filepath = current / (f'json/{environment}_configuration.json')

package-lock.json

@@ -1,5 +1,5 @@
 {
     "dependencies": {
-        "aws-cdk": "2.115.0"
+        "aws-cdk": "2.121.1"
     }
 }

package.json

@@ -1,5 +1,5 @@
 import uuid
-from datetime import datetime
+from datetime import datetime, timezone
 
 import boto3
 from botocore.exceptions import ClientError
@@ -27,7 +27,7 @@ def _get_db_handler(self, table_name: str) -> Table:
         return dynamodb.Table(table_name)
 
     def _get_unix_time(self) -> int:
-        return int(datetime.utcnow().timestamp())
+        return int(datetime.now(timezone.utc).timestamp())
 
     @tracer.capture_method(capture_response=False)
     def create_order_in_db(self, customer_name: str, order_item_count: int) -> Order:

poetry.lock

@@ -1,5 +1,5 @@
 import json
-from datetime import datetime
+from datetime import datetime, timezone
 from http import HTTPStatus
 from typing import Any
 
@@ -70,7 +70,7 @@ def test_handler_200_ok(mocker, table_name: str):
     assert 'Item' in response
     assert response['Item']['name'] == customer_name
     assert response['Item']['item_count'] == order_item_count
-    now = int(datetime.utcnow().timestamp())
+    now = int(datetime.now(timezone.utc).timestamp())
     assert now - int(response['Item']['created_at']) <= 60  # assume item was created in last minute, check that utc time calc is correct
 
 

service/dal/dynamo_dal_handler.py

@@ -3,8 +3,6 @@
 
 from service.models.input import CreateOrderRequest
 
-# ... potentially more imports based on your project ...
-
 
 def test_invalid_name():
     # Given: An empty customer_name