On 04/01/2016 15:05, Jaroslav Bachorik wrote:
:

Not sure. With this change in place the test will not even try to detect the situation when JMX is not bundled with JDK.

There is another test (jdk/test/javax/management/remote/mandatory/version/ImplVersionTest.java) with the same wording in the summary but not even trying to detest the standalone jmxrmi.jar.

If you insist on changing the wording I would prefer changing both of these tests in one go, perhaps as a separate CR?
I just wanted to make the point that with the change then the @summary on the test doesn't make sense and it's the opportunity to align it with what the test does.

Doing the other test as as separate issue is no problem.

-Alan

Reply via email to