From 0178e842bab8b5b77cfcd4f39b6b4f0909b807f2 Mon Sep 17 00:00:00 2001
From: RobertGHippo <robert.gammon@education.gov.uk>
Date: Wed, 3 Jul 2024 19:07:49 +0100
Subject: [PATCH] Consistency of error-log format

---
 .../ContentfulContentFilterService.cs         |  2 +-
 .../ContentfulContentFilterServiceTests.cs    | 37 +++++++++++++++++++
 2 files changed, 38 insertions(+), 1 deletion(-)

diff --git a/src/Dfe.EarlyYearsQualification.Content/Services/ContentfulContentFilterService.cs b/src/Dfe.EarlyYearsQualification.Content/Services/ContentfulContentFilterService.cs
index 14b55d88..2db4ec9e 100644
--- a/src/Dfe.EarlyYearsQualification.Content/Services/ContentfulContentFilterService.cs
+++ b/src/Dfe.EarlyYearsQualification.Content/Services/ContentfulContentFilterService.cs
@@ -141,7 +141,7 @@ private List<Qualification> FilterQualificationsByDate(int startDateMonth, int s
         var splitQualificationDate = qualificationDate.Split('-');
         if (splitQualificationDate.Length != 2)
         {
-            logger.LogError("Found qualification date {QualificationDate} with unexpected format", qualificationDate);
+            logger.LogError("Qualification date {QualificationDate} has unexpected format", qualificationDate);
             return (false, 0, 0);
         }
 
diff --git a/tests/Dfe.EarlyYearsQualification.UnitTests/Services/ContentfulContentFilterServiceTests.cs b/tests/Dfe.EarlyYearsQualification.UnitTests/Services/ContentfulContentFilterServiceTests.cs
index 632ab03e..d27c45c9 100644
--- a/tests/Dfe.EarlyYearsQualification.UnitTests/Services/ContentfulContentFilterServiceTests.cs
+++ b/tests/Dfe.EarlyYearsQualification.UnitTests/Services/ContentfulContentFilterServiceTests.cs
@@ -370,6 +370,43 @@ public async Task GetFilteredQualifications_ContentfulClientThrowsException_Retu
         filteredQualifications.Should().BeEmpty();
     }
 
+    [TestMethod]
+    public async Task GetFilteredQualifications_DataContainsInvalidDateFormat_LogsError()
+    {
+        var results = new ContentfulCollection<Qualification>
+                      {
+                          Items = new[]
+                                  {
+                                      new Qualification(
+                                                        "EYQ-123",
+                                                        "test",
+                                                        "NCFE",
+                                                        4,
+                                                        "Sep15", // We expect Mmm-yy, e.g. "Sep-15"
+                                                        "Aug-19",
+                                                        "abc/123/987",
+                                                        "requirements")
+                                  }
+                      };
+
+        var mockContentfulClient = new Mock<IContentfulClient>();
+        mockContentfulClient.Setup(x => x.GetEntries(
+                                                     It.IsAny<QueryBuilder<Qualification>>(),
+                                                     It.IsAny<CancellationToken>()))
+                            .ReturnsAsync(results);
+
+        var mockQueryBuilder = new MockQueryBuilder();
+        var mockLogger = new Mock<ILogger<ContentfulContentFilterService>>();
+        var filterService = new ContentfulContentFilterService(mockContentfulClient.Object, mockLogger.Object)
+                            {
+                                QueryBuilder = mockQueryBuilder
+                            };
+
+        await filterService.GetFilteredQualifications(4, 5, 2016);
+
+        mockLogger.VerifyError("Qualification date Sep15 has unexpected format");
+    }
+
     [TestMethod]
     public async Task GetFilteredQualifications_DataContainsInvalidMonth_LogsError()
     {