diff --git a/src/main/scala/org/scoverage/coveralls/CoverallPayloadWriter.scala b/src/main/scala/org/scoverage/coveralls/CoverallPayloadWriter.scala index cb99616..afc1518 100644 --- a/src/main/scala/org/scoverage/coveralls/CoverallPayloadWriter.scala +++ b/src/main/scala/org/scoverage/coveralls/CoverallPayloadWriter.scala @@ -25,7 +25,7 @@ class CoverallPayloadWriter( factory.createGenerator(file, JsonEncoding.UTF8) } - def start() { + def start() = { gen.writeStartObject() def writeOpt(fieldName: String, holder: Option[String]) = @@ -36,11 +36,10 @@ class CoverallPayloadWriter( gen.writeStringField("repo_token", token) case CIServiceToken(token) => gen.writeStringField("repo_token", token) - writeOpt("service_name", service.map(_.name)) case NoTokenNeeded => - writeOpt("service_name", service.map(_.name)) } + writeOpt("service_name", service.map(_.name)) writeOpt("service_job_id", service.flatMap(_.jobId)) writeOpt("service_pull_request", service.flatMap(_.pullRequest)) writeOpt("flag_name", sys.env.get("COVERALLS_FLAG_NAME")) @@ -53,7 +52,7 @@ class CoverallPayloadWriter( gen.writeStartArray() } - private def addGitInfo() { + private def addGitInfo() = { gen.writeFieldName("git") gen.writeStartObject() @@ -86,7 +85,7 @@ class CoverallPayloadWriter( gen.writeEndObject() } - private def addGitRemotes(remotes: Seq[String]) { + private def addGitRemotes(remotes: Seq[String]) = { remotes.foreach(remote => { gen.writeStartObject() gen.writeStringField("name", remote) @@ -95,7 +94,7 @@ class CoverallPayloadWriter( }) } - def addSourceFile(report: SourceFileReport) { + def addSourceFile(report: SourceFileReport) = { val repoRootDirStr = repoRootDir.getCanonicalPath.replace(File.separator, "/") + "/" diff --git a/src/main/scala/org/scoverage/coveralls/CoverallsAuth.scala b/src/main/scala/org/scoverage/coveralls/CoverallsAuth.scala index 156929d..8523ffc 100644 --- a/src/main/scala/org/scoverage/coveralls/CoverallsAuth.scala +++ b/src/main/scala/org/scoverage/coveralls/CoverallsAuth.scala @@ -9,16 +9,18 @@ import io.circe.generic.auto._ */ sealed trait CoverallsAuth -/** Auth strategy where a Coveralls-specific token is used. Works with every CI - * service. +/** Auth strategy where a Coveralls-specific token is used. Works + * with every CI service. */ case class CoverallsRepoToken(token: String) extends CoverallsAuth -/** Auth strategy where a token specific to the CI service is used, such as a - * GitHub token. Works on selected CI services supported by Coveralls. +/** Auth strategy where a token specific to the CI service is used, + * such as a GitHub token. Works on selected CI services supported + * by Coveralls. */ case class CIServiceToken(token: String) extends CoverallsAuth -/** Auth strategy where no token is passed. This seems to work for Travis. +/** Auth strategy where no token is passed. This seems to work + * for Travis. */ case object NoTokenNeeded extends CoverallsAuth diff --git a/src/test/scala/org/scoverage/coveralls/CoverallPayloadWriterTest.scala b/src/test/scala/org/scoverage/coveralls/CoverallPayloadWriterTest.scala index 9d32e49..24798a5 100644 --- a/src/test/scala/org/scoverage/coveralls/CoverallPayloadWriterTest.scala +++ b/src/test/scala/org/scoverage/coveralls/CoverallPayloadWriterTest.scala @@ -75,8 +75,10 @@ class CoverallPayloadWriterTest payloadWriter.start payloadWriter.flush() + println(writer.toString()) + writer.toString should equal( - """{"repo_token":"testRepoToken","service_job_id":"testServiceJob","parallel":false,""" + + """{"repo_token":"testRepoToken","service_name":"my-service","service_job_id":"testServiceJob","parallel":false,""" + expectedGit + ""","source_files":[""" )