postgresql-testing: On failure, logs are no longer an Option. (#5190)

`logs` used to be an `Option[Seq[String]]`. They were changed to
`Seq[String]` with a default of `Seq.empty`, but the code to print them
still assumed an `Option`, which means they were printing as
"ArrayBuffer(line 1, line 2, line 3)" instead of, well, nicely.

CHANGELOG_BEGIN
CHANGELOG_END
This commit is contained in:
Samir Talwar 2020-03-26 09:09:24 +01:00 committed by GitHub
parent c9e2f7929b
commit 1e13b8679e
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -209,7 +209,7 @@ object PostgresAround {
command.mkString("\n"),
s"\nSTDOUT:\n$stdout",
s"\nSTDERR:\n$stderr",
logs.map(lines => s"\nLogs:\n${lines.mkString("\n")}"),
if (logs.isEmpty) "\nLogs: <none>" else s"\nLogs:\n${logs.mkString("\n")}",
).mkString("\n"),
cause,
)