Fix SimpleRepository info parsing when the created_on has no miliseconds

Author: machadoit

src/main/scala/com/codacy/client/bitbucket/Repository.scala

@@ -11,7 +11,10 @@ case class Repository(name: String, full_name: String, description: String, scm:
 
 object Repository {
   val dateFormat = "yyyy-MM-dd'T'HH:mm:ss.SSSSSSZZ"
+  val dateFormatWithoutMillis = "yyyy-MM-dd'T'HH:mm:ssZZ"
+
   implicit val jodaDateTimeReads = Reads.jodaDateReads(dateFormat)
+    .orElse(Reads.jodaDateReads(dateFormatWithoutMillis))
 
   implicit val reader: Reads[Repository] = {
     ((__ \ "name").read[String] and

src/main/scala/com/codacy/client/bitbucket/SimpleRepository.scala

@@ -14,7 +14,10 @@ case class SimpleRepository(name: String, description: String, scm: String, crea
 
 object SimpleRepository {
   val dateFormat = "yyyy-MM-dd'T'HH:mm:ss.SSS"
+  val dateFormatWithoutMillis = "yyyy-MM-dd'T'HH:mm:ss"
+
   implicit val jodaDateTimeReads = Reads.jodaDateReads(dateFormat)
+    .orElse(Reads.jodaDateReads(dateFormatWithoutMillis))
 
   implicit val reader: Reads[SimpleRepository] =
     ((__ \ "slug").read[String] and
@@ -27,5 +30,5 @@ object SimpleRepository {
       (__ \ "has_issues").read[Boolean] and
       (__ \ "is_private").read[Boolean] and
       (__ \ "language").read[String]
-      )(SimpleRepository.apply _)
+      ) (SimpleRepository.apply _)
 }

src/test/scala/RemoteRepositorySpecs.scala

@@ -462,4 +462,48 @@ class RemoteRepositorySpecs extends FlatSpec with Matchers {
       repo => repo.length shouldBe 2
     )
   }
+
+  it should "successfully parse a JSON into an array of SimpleRepository that has no milliseconds on the created date" in {
+
+    val input =
+      """[
+        |{
+        |  "scm": "git",
+        |  "has_wiki": false,
+        |  "last_updated": "2016-01-26T21:39:24.485",
+        |  "no_forks": false,
+        |  "created_on": "2015-09-04T20:33:22",
+        |  "owner": "carrots",
+        |  "logo": "https://bitbucket.org/carrots/potatos/avatar/32/?ts=1453840764",
+        |  "email_mailinglist": "",
+        |  "is_mq": false,
+        |  "size": 14544338,
+        |  "read_only": false,
+        |  "fork_of": null,
+        |  "mq_of": null,
+        |  "state": "available",
+        |  "utc_created_on": "2015-09-04 18:37:22+00:00",
+        |  "website": "",
+        |  "description": "",
+        |  "has_issues": false,
+        |  "is_fork": false,
+        |  "slug": "potatos",
+        |  "is_private": true,
+        |  "name": "Carrots and company",
+        |  "language": "",
+        |  "utc_last_updated": "2016-01-26 20:39:24+00:00",
+        |  "no_public_forks": true,
+        |  "creator": null,
+        |  "resource_uri": "/1.0/repositories/carrots/potatos"
+        |}
+        |]
+      """.stripMargin
+    val json = Json.parse(input)
+    val value = json.validate[Seq[SimpleRepository]]
+
+    value.fold(e =>
+      fail(s"$e"),
+      repo => repo.length shouldBe 1
+    )
+  }
 }