RFR (xs) 8132003: Update javax/management regression test for Verona (versioning)

[Iris Clark]

Hi.

Please review changes to resolve the following bug:

8132003: Update javax/management regression test for Verona (versioning)
Bug: https://bugs.openjdk.java.net/browse/JDK-8132003

The regression test javax/management/remote/mandatory/notif/NotSerializableNotifTest.java fails in Verona [0,1] builds because it assumes that the system property java.verison will always begin with "1.".  Verona drops this part of the version string for JDK 9 and later.

The test attempts to discern whether it is being run against JDK 1.4 or earlier.  Since I believe that this test will never be run against JDK 1.4, I expect that we can entirely remove the conditional.  Here's the required diff:

--- a/test/javax/management/remote/mandatory/notif/NotSerializableNotifTest.java
        Mon Jul 20 11:01:24 2015 -0700
+++ b/test/javax/management/remote/mandatory/notif/NotSerializableNotifTest.java
        Mon Jul 20 19:31:54 2015 -0700
@@ -24,7 +24,7 @@
 /*
  * @test
  * @summary Tests to send a not serializable notification.
- * @bug 5022196
+ * @bug 5022196 8132003
  * @author Shanliang JIANG
  * @modules java.management
  * @run clean NotSerializableNotifTest
@@ -53,22 +53,13 @@
     private static final MBeanServer mbeanServer = MBeanServerFactory.createMBeanServer();
     private static ObjectName emitter;

-    private static String[] protocols;
+    private static String[] protocols = new String[] {"rmi", "iiop", "jmxmp"};

     private static final int sentNotifs = 10;

     public static void main(String[] args) throws Exception {
         System.out.println(">>> Test to send a not serializable notification");

-        // IIOP fails on JDK1.4, see 5034318
-        final String v = System.getProperty("java.version");
-        float f = Float.parseFloat(v.substring(0, 3));
-        if (f<1.5) {
-            protocols = new String[] {"rmi", "jmxmp"};
-        } else {
-            protocols = new String[] {"rmi", "iiop", "jmxmp"};
-        }
-
         emitter = new ObjectName("Default:name=NotificationEmitter");
         mbeanServer.registerMBean(new NotificationEmitter(), emitter);

If you believe that we need to keep the conditional, I can submit an alternate diff.  Let me know what you'd prefer.  Please also let me know if there are any JMX-specific development processes I should be aware of (e.g. required number of Reviewers, testing requirements, etc.).

After review, the changeset will be pushed to verona/stage [2]. The changeset will go to jdk9/* when Verona is complete later this summer.

Thanks,
iris

[0] http://openjdk.java.net/projects/verona
[1] http://openjdk.java.net/jeps/223
[2] http://hg.openjdk.java.net/verona/stage