Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bump aws-cdk from 2.171.0 to 2.171.1 in /notificationworkerlambda/cdk #1349

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Bump aws-cdk from 2.171.0 to 2.171.1 in /notificationworkerlambda/cdk

5c3cd61
Select commit
Loading
Failed to load commit list.
Sign in for the full log view
Open

Bump aws-cdk from 2.171.0 to 2.171.1 in /notificationworkerlambda/cdk #1349

Bump aws-cdk from 2.171.0 to 2.171.1 in /notificationworkerlambda/cdk
5c3cd61
Select commit
Loading
Failed to load commit list.
GitHub Actions / Tests succeeded Nov 28, 2024 in 0s

35 passed, 0 failed and 0 skipped

Tests passed successfully

Report Passed Failed Skipped Time
notificationworkerlambda/target/test-reports/TEST-com.gu.notifications.worker.delivery.apns.models.payload.ApnsPayloadBuilderSpec.xml 10✅ 2s
notificationworkerlambda/target/test-reports/TEST-com.gu.notifications.worker.delivery.fcm.FcmClientTest.xml 3✅ 4s
notificationworkerlambda/target/test-reports/TEST-com.gu.notifications.worker.delivery.fcm.FcmTransporttJdkImplSpec.xml 4✅ 787ms
notificationworkerlambda/target/test-reports/TEST-com.gu.notifications.worker.delivery.fcm.models.payload.FcmPayloadBuilderSpec.xml 6✅ 70ms
notificationworkerlambda/target/test-reports/TEST-com.gu.notifications.worker.HarvesterRequestHandlerSpec.xml 1✅ 247ms
notificationworkerlambda/target/test-reports/TEST-com.gu.notifications.worker.models.SendingResultsSpec.xml 8✅ 110ms
notificationworkerlambda/target/test-reports/TEST-com.gu.notifications.worker.SenderRequestHandlerSpec.xml 3✅ 7s

✅ notificationworkerlambda/target/test-reports/TEST-com.gu.notifications.worker.delivery.apns.models.payload.ApnsPayloadBuilderSpec.xml

10 tests were completed in 2s with 10 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
com.gu.notifications.worker.delivery.apns.models.payload.ApnsPayloadBuilderSpec 10✅ 2s

✅ com.gu.notifications.worker.delivery.apns.models.payload.ApnsPayloadBuilderSpec

✅ ApnsPayload should::generate correct payload for Breaking News notification
✅ ApnsPayload should::generate correct payload for Breaking News notification with no thumbnail
✅ ApnsPayload should::generate correct payload for Breaking News notification with no image
✅ ApnsPayload should::generate correct payload for Liveblog notifications if blockId exists
✅ ApnsPayload should::generate correct payload for Breaking News notification with no title
✅ ApnsPayload should::generate correct payload for Content notification
✅ ApnsPayload should::generate correct payload for Match status notification
✅ ApnsPayload should::generate correct payload for Newsstand notification
✅ ApnsPayload should::generate correct payload for Edition notification
✅ ApnsPayload should::generate correct payload for US Election notification

✅ notificationworkerlambda/target/test-reports/TEST-com.gu.notifications.worker.delivery.fcm.FcmClientTest.xml

3 tests were completed in 4s with 3 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
com.gu.notifications.worker.delivery.fcm.FcmClientTest 3✅ 4s

✅ com.gu.notifications.worker.delivery.fcm.FcmClientTest

✅ the FcmClient should::Parse successful responses as an FcmDeliverySuccess
✅ the FcmClient should::Parse errors with an invalid token error code as an InvalidToken
✅ the FcmClient should::Parse errors with NOT_FOUND error code and 'Requested entity was not found.' error message as an InvalidToken

✅ notificationworkerlambda/target/test-reports/TEST-com.gu.notifications.worker.delivery.fcm.FcmTransporttJdkImplSpec.xml

4 tests were completed in 787ms with 4 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
com.gu.notifications.worker.delivery.fcm.FcmTransporttJdkImplSpec 4✅ 787ms

✅ com.gu.notifications.worker.delivery.fcm.FcmTransporttJdkImplSpec

✅ FcmTransporttJdkImpl should::parse an invalid token error response correctly from from Firebase API
✅ FcmTransporttJdkImpl should::parse an unexpected error response with general details field from Firebase API
✅ FcmTransporttJdkImpl should::parse an internal error object from the error response without details field from Firebase API
✅ FcmTransporttJdkImpl should::handle unsupported error response from Firebase API gracefully

✅ notificationworkerlambda/target/test-reports/TEST-com.gu.notifications.worker.delivery.fcm.models.payload.FcmPayloadBuilderSpec.xml

6 tests were completed in 70ms with 6 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
com.gu.notifications.worker.delivery.fcm.models.payload.FcmPayloadBuilderSpec 6✅ 70ms

✅ com.gu.notifications.worker.delivery.fcm.models.payload.FcmPayloadBuilderSpec

✅ FcmPayload should::generate correct data for Breaking News notification
✅ FcmPayload should::generate correct data for Breaking News notification with no title
✅ FcmPayload should::generate correct data for Content notification
✅ FcmPayload should::generate correct data for Match Status notification
✅ FcmPayload should::generate correct data for Editions notification
✅ FcmPayload should::generate correct data for US Elections notification

✅ notificationworkerlambda/target/test-reports/TEST-com.gu.notifications.worker.HarvesterRequestHandlerSpec.xml

1 tests were completed in 247ms with 1 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
com.gu.notifications.worker.HarvesterRequestHandlerSpec 1✅ 247ms

✅ com.gu.notifications.worker.HarvesterRequestHandlerSpec

✅ the WorkerRequestHandler should::Queue one multi platform breaking news notification

✅ notificationworkerlambda/target/test-reports/TEST-com.gu.notifications.worker.models.SendingResultsSpec.xml

8 tests were completed in 110ms with 8 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
com.gu.notifications.worker.models.SendingResultsSpec 8✅ 110ms

✅ com.gu.notifications.worker.models.SendingResultsSpec

✅ LatencyMetrics.aggregateForCloudWatch should::Create a single batch of token delivery latencies if there are less than 150 unique values (and the default batch size is used)
✅ LatencyMetrics.aggregateForCloudWatch should::Create multiple batches of token delivery latencies if the number of items exceeds the batch size
✅ LatencyMetrics.aggregateForCloudWatch should::Create multiple batches of 150 by default
✅ LatencyMetrics.audienceSizeBucket should::Categorise a small audience correctly
✅ LatencyMetrics.audienceSizeBucket should::Categorise a medium audience correctly
✅ LatencyMetrics.audienceSizeBucket should::Categorise a large audience correctly
✅ LatencyMetrics.audienceSizeBucket should::Categorise an extra large audience correctly
✅ LatencyMetrics.audienceSizeBucket should::Mark a missing audience as unknown

✅ notificationworkerlambda/target/test-reports/TEST-com.gu.notifications.worker.SenderRequestHandlerSpec.xml

3 tests were completed in 7s with 3 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
com.gu.notifications.worker.SenderRequestHandlerSpec 3✅ 7s

✅ com.gu.notifications.worker.SenderRequestHandlerSpec

✅ the SenderRequestHandler should::Send one notification
✅ the SenderRequestHandler should::Clean invalid tokens
✅ the SenderRequestHandler should::Count dry runs