fix error code

Author: aracho1

mobile-save-for-later/src/main/scala/com/gu/sfl/controller/SaveArticlesController.scala

@@ -26,11 +26,10 @@ class SaveArticlesController(updateSavedArticles: UpdateSavedArticles)(implicit
           case Failure(t) => {
             val headersWithoutAuth = lambdaRequest.headers.filter{ case (k,v) => headersToKeep.contains(k.toLowerCase)}
             logger.warn(s"Could not read value: $json \nWith headers: $headersWithoutAuth" )
+            Future { LambdaResponse(StatusCodes.badRequest, Some("Could not parse request body")) }
           }
-
-          case _ => ()
+          case _ => futureSave(triedSavedArticles, lambdaRequest.headers)
         }
-        futureSave(triedSavedArticles, lambdaRequest.headers)
       case LambdaRequest(None,  _) =>
         Future { LambdaResponse(StatusCodes.badRequest, Some("Expected a json body")) }
     }

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

@@ -0,0 +1,54 @@
+package com.gu.sfl.controller
+
+import com.gu.sfl.exception.SaveForLaterError
+import com.gu.sfl.lambda.LambdaRequest
+import com.gu.sfl.model.SavedArticles
+import com.gu.sfl.savedarticles.UpdateSavedArticles
+import com.gu.sfl.util.StatusCodes
+import org.specs2.mock.Mockito
+import org.specs2.mutable.Specification
+import org.specs2.specification.Scope
+
+import scala.concurrent.duration.Duration
+import scala.concurrent.{Await, ExecutionContext, Future}
+
+class SaveArticlesControllerSpec extends Specification with Mockito {
+
+  "SaveArticlesController" should {
+
+    "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}]}"""
+      val response = Await.result(controller(LambdaRequest(Some(json))), Duration.Inf)
+
+      response.statusCode mustEqual StatusCodes.badRequest
+      there were no(updateSavedArticles).save(any[Map[String, String]](), any[SavedArticles]())
+    }
+
+    "return a 400 when the request body is not valid json" in new Setup {
+      val response = Await.result(controller(LambdaRequest(Some("not json"))), Duration.Inf)
+
+      response.statusCode mustEqual StatusCodes.badRequest
+    }
+
+    "return a 400 when there is no request body" in new Setup {
+      val response = Await.result(controller(LambdaRequest(None)), Duration.Inf)
+
+      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)
+  }
+}