RFR: JDK-8240970: Some tests fail when run with JCov

Jonathan Gibbons jonathan.gibbons at oracle.com
Wed Mar 18 17:26:18 UTC 2020


Looks good to me.

When comparing two List<String> to be equal, I recommend using 
ToolBox.checkEqual, which gives more detailed info when the lists are 
not equal.

We might also want to provide, eventually, a variant of ToolBox.grep 
that checked for lines not matching.

-- Jon


On 3/18/20 9:51 AM, Vicente Romero wrote:
> Please review fix for [1] at [2]. The intention of these fix is to be 
> able to have clean jcov results when executing langtools tests. As 
> explained in the JIRA entry, when running tests with jcov, a number of 
> tests fail because some env variables used to run tests under jcov 
> cause the runtime to emit non-suppressible warnings. These warnings 
> can affect tests that check the output to be as expected. There are 
> other issues we found like a dependency jcov has on java.logging which 
> they will be trying to eliminate. This dependency was the case of the 
> failure for test: tools/jdeps/modules/CheckModuleTest.java. The 
> current solution is to ignore that dependency. We should be able to 
> restore the test to its original form once this bug has been fixed in 
> jcov.
>
> Thanks,
> Vicente
>
> [1] https://bugs.openjdk.java.net/browse/JDK-8240970
> [2] http://cr.openjdk.java.net/~vromero/8240970/webrev.00/


More information about the compiler-dev mailing list