diff --git a/enrollment-management/app/events/set_status_as_archived.py b/enrollment-management/app/events/set_status_as_archived.py index fd130af..0b9eda2 100644 --- a/enrollment-management/app/events/set_status_as_archived.py +++ b/enrollment-management/app/events/set_status_as_archived.py @@ -20,7 +20,7 @@ def lambda_handler(event: EventBridgeEvent, context: LambdaContext) -> bool: new_image = event.detail['new_image'] now_ = now() - with enrollment_layer.transact_items() as transact: + with enrollment_layer.transact_writer() as transact: transact.update( key=KeyPair(new_image['id'], '0'), update_expr='SET #status = :archived, update_date = :update_date', @@ -41,6 +41,5 @@ def lambda_handler(event: EventBridgeEvent, context: LambdaContext) -> bool: 'create_date': now_, }, ) - transact.write_items() return True diff --git a/enrollment-management/app/events/set_status_as_expired.py b/enrollment-management/app/events/set_status_as_expired.py index 5299bbf..5f31f02 100644 --- a/enrollment-management/app/events/set_status_as_expired.py +++ b/enrollment-management/app/events/set_status_as_expired.py @@ -20,7 +20,7 @@ def lambda_handler(event: EventBridgeEvent, context: LambdaContext) -> bool: new_image = event.detail['new_image'] now_ = now() - with enrollment_layer.transact_items() as transact: + with enrollment_layer.transact_writer() as transact: transact.update( key=KeyPair(new_image['id'], '0'), update_expr='SET #status = :expired, update_date = :update_date', @@ -43,6 +43,4 @@ def lambda_handler(event: EventBridgeEvent, context: LambdaContext) -> bool: }, ) - transact.write_items() - return True diff --git a/http-api/app/konviva.py b/http-api/app/konviva.py index d2e2ae3..dd196e6 100644 --- a/http-api/app/konviva.py +++ b/http-api/app/konviva.py @@ -35,7 +35,7 @@ def token(username: str) -> KonvivaToken: r.raise_for_status() # Because Konviva does not return the proper HTTP status code - if err := glom(r.json(), 'errors.0', default=None): + if err := glom(r.json(), 'errors', default=None): raise KonvivaError(err) return KonvivaToken(**r.json()) diff --git a/http-api/template.yaml b/http-api/template.yaml index a8d9dcc..3f0b7cf 100644 --- a/http-api/template.yaml +++ b/http-api/template.yaml @@ -23,7 +23,7 @@ Globals: Architectures: - x86_64 Layers: - - !Sub arn:aws:lambda:sa-east-1:336641857101:layer:layercake:68 + - !Sub arn:aws:lambda:sa-east-1:336641857101:layer:layercake:72 Environment: Variables: TZ: America/Sao_Paulo diff --git a/http-api/tests/test_konviva.py b/http-api/tests/test_konviva.py index 269aebe..c507f50 100644 --- a/http-api/tests/test_konviva.py +++ b/http-api/tests/test_konviva.py @@ -1,3 +1,5 @@ +import pytest + import konviva from config import KONVIVA_API_URL @@ -9,3 +11,8 @@ def test_konviva_token(): redirect_uri = konviva.redirect_uri(token) assert KONVIVA_API_URL in redirect_uri + + +def test_konviva_token_error(): + with pytest.raises(konviva.KonvivaError): + konviva.token('sergio@doesnotexist.com')