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

fix: Improve Redis error handling #877

Merged
merged 2 commits into from
Nov 13, 2024

Merge branch 'development' into fix/redis/add-error-handling

87b9d5d
Select commit
Loading
Failed to load commit list.
Sign in for the full log view
Merged

fix: Improve Redis error handling #877

Merge branch 'development' into fix/redis/add-error-handling
87b9d5d
Select commit
Loading
Failed to load commit list.
GitHub Actions / DotNET Tests succeeded Nov 13, 2024 in 0s

642 passed, 0 failed and 0 skipped

Tests passed successfully

Report Passed Failed Skipped Time
tests/Dfe.PlanTech.Application.UnitTests/TestResults/test-results.trx 108✅ 7s
tests/Dfe.PlanTech.Domain.UnitTests/TestResults/test-results.trx 14✅ 3s
tests/Dfe.PlanTech.Infrastructure.Contentful.UnitTests/TestResults/test-results.trx 72✅ 5s
tests/Dfe.PlanTech.Infrastructure.Redis.UnitTests/TestResults/test-results.trx 60✅ 11s
tests/Dfe.PlanTech.Infrastructure.ServiceBus.UnitTests/TestResults/test-results.trx 16✅ 7s
tests/Dfe.PlanTech.Infrastructure.SignIn.UnitTests/TestResults/test-results.trx 45✅ 5s
tests/Dfe.PlanTech.UnitTests.Shared/TestResults/test-results.trx 702ms
tests/Dfe.PlanTech.Web.UnitTests/TestResults/test-results.trx 327✅ 11s

✅ tests/Dfe.PlanTech.Application.UnitTests/TestResults/test-results.trx

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

Test suite Passed Failed Skipped Time
Dfe.PlanTech.Application.Cookie.Service.Tests.CookiesCleanerTests 1✅ 65ms
Dfe.PlanTech.Application.UnitTests.Content.Commands.WriteCmsWebhookToQueueCommandTests 3✅ 17ms
Dfe.PlanTech.Application.UnitTests.Content.Queries.GetEntityFromContentfulQueryTests 1✅ 7ms
Dfe.PlanTech.Application.UnitTests.Content.Queries.GetNavigationQueryTests 2✅ 17ms
Dfe.PlanTech.Application.UnitTests.Content.Queries.GetPageQueryTests 3✅ 42ms
Dfe.PlanTech.Application.UnitTests.Content.Queries.GetSubTopicRecommendationQueryTests 9✅ 137ms
Dfe.PlanTech.Application.UnitTests.Cookie.Service.CookieServiceTests 9✅ 100ms
Dfe.PlanTech.Application.UnitTests.Core.UrlHistoryTests 2✅ 7ms
Dfe.PlanTech.Application.UnitTests.Establishments.Commands.CreateEstablishmentCommandTests 3✅ 31ms
Dfe.PlanTech.Application.UnitTests.Establishments.EstablishmentTests 11✅ 174ms
Dfe.PlanTech.Application.UnitTests.Establishments.Queries.GetEstablishmentIdQueryTests 3✅ 26ms
Dfe.PlanTech.Application.UnitTests.Persistence.Commands.WebhookMessageProcessorTests 2✅ 118ms
Dfe.PlanTech.Application.UnitTests.Persistence.GetEntitiesOptionsTests 3✅ 3ms
Dfe.PlanTech.Application.UnitTests.Questionnaire.Queries.GetNextUnansweredQuestionQueryTests 4✅ 229ms
Dfe.PlanTech.Application.UnitTests.Questionnaire.Queries.GetSectionQueryTests 2✅ 237ms
Dfe.PlanTech.Application.UnitTests.QuestionnaireCacherTests 2✅ 26ms
Dfe.PlanTech.Application.UnitTests.Submissions.CalculateMaturityCommandTests 2✅ 44ms
Dfe.PlanTech.Application.UnitTests.Submissions.Commands.ProcessSubmissionResponsesDtoCommandTests 2✅ 31ms
Dfe.PlanTech.Application.UnitTests.Submissions.Commands.SubmitAnswerCommandTests 4✅ 56ms
Dfe.PlanTech.Application.UnitTests.Submissions.Queries.CheckAnswersOrNextQuestionCheckerTests 7✅ 83ms
Dfe.PlanTech.Application.UnitTests.Submissions.Queries.GetLatestResponsesQueryTests 5✅ 322ms
Dfe.PlanTech.Application.UnitTests.Submissions.Queries.GetSubmissionStatusesQueryTests 5✅ 219ms
Dfe.PlanTech.Application.UnitTests.Submissions.Queries.SectionCompleteStatusCheckerTests 4✅ 244ms
Dfe.PlanTech.Application.UnitTests.Submissions.Queries.SectionNotStartedStatusCheckerTests 4✅ 2ms
Dfe.PlanTech.Application.UnitTests.Submissions.Queries.SubmissionStatusProcessorTests 3✅ 87ms
Dfe.PlanTech.Application.UnitTests.TemplateTest 1✅ 0ms
Dfe.PlanTech.Application.UnitTests.Users.Commands.CreateUserCommandTests 3✅ 34ms
Dfe.PlanTech.Application.UnitTests.Users.Commands.RecordUserSigninCommandTests 5✅ 35ms
Dfe.PlanTech.Application.UnitTests.Users.Queries.GetUserIdQueryTests 3✅ 16ms

✅ tests/Dfe.PlanTech.Domain.UnitTests/TestResults/test-results.trx

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

Test suite Passed Failed Skipped Time
Dfe.PlanTech.Domain.UnitTests.Helpers.DateTimeFormatterTests 7✅ 389ms
Dfe.PlanTech.Domain.UnitTests.Helpers.TimeZoneHelpersTests 3✅ 380ms
Dfe.PlanTech.Domain.UnitTests.LockOwnerUnitTests 1✅ 1ms
Dfe.PlanTech.Domain.UnitTests.RandomNumberTests 3✅ 13ms

✅ tests/Dfe.PlanTech.Infrastructure.Contentful.UnitTests/TestResults/test-results.trx

72 tests were completed in 5s with 72 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
Dfe.PlanTech.Infrastructure.Contentful.UnitTests.Content.Renderers.Models.PartRenderers.HeadingRendererTests 19✅ 9ms
Dfe.PlanTech.Infrastructure.Contentful.UnitTests.Content.Renderers.Models.PartRenderers.HyperlinkRendererTests 6✅ 3ms
Dfe.PlanTech.Infrastructure.Contentful.UnitTests.Content.Renderers.Models.PartRenderers.ListItemRendererTests 3✅ 1ms
Dfe.PlanTech.Infrastructure.Contentful.UnitTests.Content.Renderers.Models.PartRenderers.OrderedListRendererTests 3✅ 1ms
Dfe.PlanTech.Infrastructure.Contentful.UnitTests.Content.Renderers.Models.PartRenderers.ParagraphRendererTests 4✅ 1ms
Dfe.PlanTech.Infrastructure.Contentful.UnitTests.Content.Renderers.Models.PartRenderers.TableRendererTests 2✅ 28ms
Dfe.PlanTech.Infrastructure.Contentful.UnitTests.Content.Renderers.Models.PartRenderers.TextRendererTests 5✅ 5ms
Dfe.PlanTech.Infrastructure.Contentful.UnitTests.Content.Renderers.Models.PartRenderers.UnorderedListRendererTests 3✅ 2ms
Dfe.PlanTech.Infrastructure.Contentful.UnitTests.Content.Renderers.Models.RichTextRendererTests 2✅ 88ms
Dfe.PlanTech.Infrastructure.Contentful.UnitTests.Persistence.ContentfulRepositoryTests 10✅ 125ms
Dfe.PlanTech.Infrastructure.Contentful.UnitTests.Persistence.EntityResolverTests 3✅ 30ms
Dfe.PlanTech.Infrastructure.Contentful.UnitTests.QueryBuilderByContentTypeTests 4✅ 2ms
Dfe.PlanTech.Infrastructure.Contentful.UnitTests.QueryBuilderSelectTests 2✅ 2ms
Dfe.PlanTech.Infrastructure.Contentful.UnitTests.QueryBuilderWithQueryTests 4✅ 20ms
Dfe.PlanTech.Infrastructure.Contentful.UnitTests.Serializers.DependencyInjectionContractResolverTests 2✅ 102ms

✅ tests/Dfe.PlanTech.Infrastructure.Redis.UnitTests/TestResults/test-results.trx

60 tests were completed in 11s with 60 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
Dfe.PlanTech.Infrastructure.Redis.UnitTests.ContentComponentJsonExtensionsTests 1✅ 610ms
Dfe.PlanTech.Infrastructure.Redis.UnitTests.GuardTests 21✅ 33ms
Dfe.PlanTech.Infrastructure.Redis.UnitTests.GZipRedisValueCompressorTests 8✅ 87ms
Dfe.PlanTech.Infrastructure.Redis.UnitTests.RedisCacheTests 19✅ 930ms
Dfe.PlanTech.Infrastructure.Redis.UnitTests.RedisLockProviderUnitTests 11✅ 5s

✅ tests/Dfe.PlanTech.Infrastructure.ServiceBus.UnitTests/TestResults/test-results.trx

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

Test suite Passed Failed Skipped Time
Dfe.PlanTech.Infrastructure.ServiceBus.UnitTests.ContentfulServiceBusProcessorTests 6✅ 67ms
Dfe.PlanTech.Infrastructure.ServiceBus.UnitTests.QueueWriterTests 2✅ 280ms
Dfe.PlanTech.Infrastructure.ServiceBus.UnitTests.ServiceBusResultProcessorTests 6✅ 296ms
Dfe.PlanTech.Infrastructure.ServiceBus.UnitTests.Utils.MessageRetryHandlerTests 2✅ 224ms

✅ tests/Dfe.PlanTech.Infrastructure.SignIn.UnitTests/TestResults/test-results.trx

45 tests were completed in 5s with 45 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
Dfe.PlanTech.Infrastructure.SignIns.UnitTests.DfeOpenIdConnectEventsTests 37✅ 236ms
Dfe.PlanTech.Infrastructure.SignIns.UnitTests.UserClaimsExtensionsTests 8✅ 72ms

✅ tests/Dfe.PlanTech.UnitTests.Shared/TestResults/test-results.trx

No tests found

✅ tests/Dfe.PlanTech.Web.UnitTests/TestResults/test-results.trx

327 tests were completed in 11s with 327 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
Dfe.ContentSupport.Web.Tests.Controllers.ContentCacheControllerTests 1✅ 2ms
Dfe.ContentSupport.Web.Tests.Controllers.ContentControllerTests 6✅ 111ms
Dfe.PlanTech.Web.UnitTests.Authorisation.ApiKeyAuthorisationFilterTests 6✅ 13ms
Dfe.PlanTech.Web.UnitTests.Authorisation.PageModelAuthorisationPolicyTests 9✅ 56ms
Dfe.PlanTech.Web.UnitTests.Authorisation.SignedRequestAuthorisationPolicyTests 5✅ 18ms
Dfe.PlanTech.Web.UnitTests.BaseTaskListTagHelperTests 1✅ 0ms
Dfe.PlanTech.Web.UnitTests.Binders.PageModelBinderTests 3✅ 172ms
Dfe.PlanTech.Web.UnitTests.Content.ContentfulServiceTests 1✅ 75ms
Dfe.PlanTech.Web.UnitTests.Content.ContentServiceTests 15✅ 148ms
Dfe.PlanTech.Web.UnitTests.Content.LayoutServiceTests 15✅ 6ms
Dfe.PlanTech.Web.UnitTests.Content.ModelMapperTests 8✅ 9ms
Dfe.PlanTech.Web.UnitTests.Content.StubContentfulClientTests 1✅ 127ms
Dfe.PlanTech.Web.UnitTests.Controllers.AuthControllerTests 1✅ 5ms
Dfe.PlanTech.Web.UnitTests.Controllers.CheckAnswersControllerTests 12✅ 212ms
Dfe.PlanTech.Web.UnitTests.Controllers.CmsControllerTests 3✅ 19ms
Dfe.PlanTech.Web.UnitTests.Controllers.CookieControllerTests 13✅ 26ms
Dfe.PlanTech.Web.UnitTests.Controllers.PagesControllerTests 7✅ 73ms
Dfe.PlanTech.Web.UnitTests.Controllers.QuestionsControllerTests 13✅ 70ms
Dfe.PlanTech.Web.UnitTests.Controllers.RecommendationsControllerTests 12✅ 27ms
Dfe.PlanTech.Web.UnitTests.Exceptions.ExceptionTests 6✅ 1ms
Dfe.PlanTech.Web.UnitTests.Extensions.WebApplicationBuilderExtensionsTests 3✅ 119ms
Dfe.PlanTech.Web.UnitTests.GetQuestionBySlugRouterTests 15✅ 36ms
Dfe.PlanTech.Web.UnitTests.Helpers.CacherTests 9✅ 25ms
Dfe.PlanTech.Web.UnitTests.Helpers.ComponentViewsFactoryTests 2✅ 21ms
Dfe.PlanTech.Web.UnitTests.Helpers.CustomTelemetryInitializerTests 2✅ 73ms
Dfe.PlanTech.Web.UnitTests.Helpers.GtmServiceTests 4✅ 2ms
Dfe.PlanTech.Web.UnitTests.Helpers.HeaderExtensionsTests 4✅ 1ms
Dfe.PlanTech.Web.UnitTests.Helpers.LogInvalidModelStateTests 1✅ 181ms
Dfe.PlanTech.Web.UnitTests.Helpers.RichTextMarkExtensionsTests 3✅ 5ms
Dfe.PlanTech.Web.UnitTests.Helpers.RouteDataExtensionsTests 4✅ 6ms
Dfe.PlanTech.Web.UnitTests.Helpers.UserHelperTests 3✅ 87ms
Dfe.PlanTech.Web.UnitTests.Middleware.HeadRequestMiddlewareTests 2✅ 26ms
Dfe.PlanTech.Web.UnitTests.Middleware.SecurityHeadersMiddlewareTests 1✅ 175ms
Dfe.PlanTech.Web.UnitTests.Middleware.ServiceExceptionHandlerMiddlewareTests 5✅ 7ms
Dfe.PlanTech.Web.UnitTests.Middleware.UserAuthorisationMiddlewareResultHandlerTests 6✅ 31ms
Dfe.PlanTech.Web.UnitTests.Middleware.UserJourneyMissingContentExceptionHandlerTests 1✅ 33ms
Dfe.PlanTech.Web.UnitTests.Middleware.UserOrganisationAuthorisationHandlerTests 5✅ 115ms
Dfe.PlanTech.Web.UnitTests.Models.ComponentModelTests 20✅ 19ms
Dfe.PlanTech.Web.UnitTests.Models.Content.Mapped.Custom.CustomAccordionTests 1✅ 88ms
Dfe.PlanTech.Web.UnitTests.Models.Content.Mapped.Custom.CustomAttachmentTests 1✅ 3ms
Dfe.PlanTech.Web.UnitTests.Models.Content.Mapped.Custom.CustomCardTests 1✅ 34ms
Dfe.PlanTech.Web.UnitTests.Models.Content.Mapped.Custom.CustomGridContainerTests 1✅ 5ms
Dfe.PlanTech.Web.UnitTests.Models.Content.Mapped.Standard.CsTextTests 2✅ 3ms
Dfe.PlanTech.Web.UnitTests.Models.Content.Mapped.Standard.EmbeddedAssetTests 3✅ 7ms
Dfe.PlanTech.Web.UnitTests.Models.Content.Mapped.Standard.EmbeddedEntryTests 1✅ 1ms
Dfe.PlanTech.Web.UnitTests.Models.Content.Mapped.Standard.HyperlinkTests 2✅ 4ms
Dfe.PlanTech.Web.UnitTests.Models.EstablishmentDtoTests 8✅ 12ms
Dfe.PlanTech.Web.UnitTests.Routing.CheckAnswersRouterTests 6✅ 112ms
Dfe.PlanTech.Web.UnitTests.Routing.GetRecommendationRouterTests 29✅ 52ms
Dfe.PlanTech.Web.UnitTests.TagHelpers.FooterLinkTagHelperTests 3✅ 7ms
Dfe.PlanTech.Web.UnitTests.TagHelpers.HeaderComponentTagHelperTests 9✅ 17ms
Dfe.PlanTech.Web.UnitTests.TagHelpers.RichTextTagHelperTests 2✅ 8ms
Dfe.PlanTech.Web.UnitTests.TagHelpers.TaskListTagTagHelperTests 4✅ 6ms
Dfe.PlanTech.Web.UnitTests.TagHelpers.WarningComponentTagHelperTests 1✅ 23ms
Dfe.PlanTech.Web.UnitTests.TaskListItemNameTagHelperTests 1✅ 1ms
Dfe.PlanTech.Web.UnitTests.TaskListItemTagHelperTests 1✅ 0ms
Dfe.PlanTech.Web.UnitTests.TaskListTagHelperTests 1✅ 0ms
Dfe.PlanTech.Web.UnitTests.ViewComponents.CategorySectionViewComponentTests 13✅ 151ms
Dfe.PlanTech.Web.UnitTests.ViewComponents.FooterLinksComponentTests 2✅ 15ms
Dfe.PlanTech.Web.UnitTests.ViewComponents.NotificationBannerTests 3✅ 1ms
Dfe.PlanTech.Web.UnitTests.ViewComponents.WarningComponentTests 4✅ 3ms