Skip to content

Commit

Permalink
[FLINK-24995][tests] Migrate tests to ConsoleReporter#finish
Browse files Browse the repository at this point in the history
'ConsoleReporter#finish' behaves just like '#printSummary', but is also supported in later versions.
Additionally, we now fail the test if the compiler reported any warnings/errors, as without it the summary business didn't provide a benefit in the first place.
  • Loading branch information
zentol committed Jun 13, 2022
1 parent f5c6abb commit 1800ab7
Showing 1 changed file with 4 additions and 1 deletion.
Expand Up @@ -189,6 +189,9 @@ object EnumValueSerializerCompatibilityTest {

run.compile(List(file.getAbsolutePath))

reporter.printSummary()
if (reporter.hasWarnings || reporter.hasErrors) {
reporter.finish()
fail("Scala compiler reported warnings or errors")
}
}
}

0 comments on commit 1800ab7

Please sign in to comment.