update test

Author: aracho1

mobile-save-for-later/src/test/scala/com/gu/sfl/controller/SaveArticlesControllerSpec.scala

@@ -15,9 +15,19 @@ import scala.concurrent.{Await, ExecutionContext, Future}
 class SaveArticlesControllerSpec extends Specification with Mockito {
 
   "SaveArticlesController" should {
+    "save the articles when the date is in the expected format" in new Setup {
+      val validDateTimeString = "2026-07-16T10:15:30Z"
+      val json = s"""{"version":"1","articles":[{"id":"id/1","shortUrl":"p/1","date": $validDateTimeString,"read":false}]}"""
+      updateSavedArticles.save(any[Map[String, String]](), any[SavedArticles]()) returns Future.successful(Left(mock[SaveForLaterError]))
+
+      Await.result(controller(LambdaRequest(Some(json))), Duration.Inf)
 
-    "return a 400 when the date on an article is not in the expected format" in new Setup {
-      val json = """{"version":"1","articles":[{"id":"id/1","shortUrl":"p/1","date":"2026-07-16T10:15:30.123Z","read":false}]}"""
+      there was one(updateSavedArticles).save(any[Map[String, String]](), any[SavedArticles]())
+    }
+
+    "return a 400 when the date is not in the expected format" in new Setup {
+      val invalidDateTimeString = "Fri Jan 01 08:00:01 GMT+08:00 2010"
+      val json = s"""{"version":"1","articles":[{"id":"id/1","shortUrl":"p/1","date": $invalidDateTimeString,"read":false}]}"""
       val response = Await.result(controller(LambdaRequest(Some(json))), Duration.Inf)
 
       response.statusCode mustEqual StatusCodes.badRequest
@@ -35,20 +45,11 @@ class SaveArticlesControllerSpec extends Specification with Mockito {
 
       response.statusCode mustEqual StatusCodes.badRequest
     }
-
-    "attempt to save the articles when the date is in the expected format" in new Setup {
-      val json = """{"version":"1","articles":[{"id":"id/1","shortUrl":"p/1","date":"2026-07-16T10:15:30Z","read":false}]}"""
-      updateSavedArticles.save(any[Map[String, String]](), any[SavedArticles]()) returns Future.successful(Left(mock[SaveForLaterError]))
-
-      Await.result(controller(LambdaRequest(Some(json))), Duration.Inf)
-
-      there was one(updateSavedArticles).save(any[Map[String, String]](), any[SavedArticles]())
-    }
   }
 
   trait Setup extends Scope {
     implicit val executionContext: ExecutionContext = scala.concurrent.ExecutionContext.global
     val updateSavedArticles = mock[UpdateSavedArticles]
     val controller = new SaveArticlesController(updateSavedArticles)
   }
-}
+}