Merge branch 'develop' into feature/coding_standardsfeature/coding_standards

1 files changed, 66 insertions, 11 deletions

diff --git a/java/test/TestH5VL.java b/java/test/TestH5VL.java
index 0397be1..4253d20 100644
--- a/java/test/TestH5VL.java
+++ b/java/test/TestH5VL.java
@@ -1,12 +1,11 @@
 /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * *
  * Copyright by The HDF Group.                                               *
- * Copyright by the Board of Trustees of the University of Illinois.         *
  * All rights reserved.                                                      *
  *                                                                           *
  * This file is part of HDF5.  The full HDF5 copyright notice, including     *
  * terms governing use, modification, and redistribution, is contained in    *
  * the COPYING file, which can be found at the root of the source code       *
- * distribution tree, or in https://support.hdfgroup.org/ftp/HDF5/releases.  *
+ * distribution tree, or in https://www.hdfgroup.org/licenses.               *
  * If you do not have access to either file, you may request a copy from     *
  * help@hdfgroup.org.                                                        *
  * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */
@@ -55,29 +54,77 @@ public class TestH5VL {
         try {
             boolean is_registered;
 
-            is_registered = H5.H5VLis_connector_registered(HDF5Constants.H5VL_NATIVE_NAME);
-            assertTrue("H5.H5VLis_connector_registered H5VL_NATIVE_NAME", is_registered);
+            is_registered = H5.H5VLis_connector_registered_by_name(HDF5Constants.H5VL_NATIVE_NAME);
+            assertTrue("H5.H5VLis_connector_registered_by_name H5VL_NATIVE_NAME", is_registered);
 
-            is_registered = H5.H5VLis_connector_registered("FAKE_VOL_NAME");
-            assertFalse("H5.H5VLis_connector_registered FAKE_VOL_NAME", is_registered);
+            is_registered = H5.H5VLis_connector_registered_by_name("FAKE_VOL_NAME");
+            assertFalse("H5.H5VLis_connector_registered_by_name FAKE_VOL_NAME", is_registered);
+
+            is_registered = H5.H5VLis_connector_registered_by_value(HDF5Constants.H5VL_NATIVE_VALUE);
+            assertTrue("H5.H5VLis_connector_registered_by_value H5VL_NATIVE_VALUE", is_registered);
         }
         catch (Throwable err) {
             err.printStackTrace();
-            fail("H5.H5VLis_connector_registered " + err);
+            fail("testH5VLnative_init(): " + err);
         }
     }
 
     @Test
     public void testH5VLget_connector_id() {
+        String H5_FILE = "testFvl.h5";
+
+        long H5fid = H5.H5Fcreate(H5_FILE, HDF5Constants.H5F_ACC_TRUNC,
+                HDF5Constants.H5P_DEFAULT, HDF5Constants.H5P_DEFAULT);
+
         try {
-            long native_id = H5.H5VLget_connector_id(HDF5Constants.H5VL_NATIVE_NAME);
-            assertTrue("H5.H5VLget_connector_id H5VL_NATIVE_NAME", native_id >= 0);
-            assertEquals(HDF5Constants.H5VL_NATIVE, native_id);
+            long native_id = H5.H5VLget_connector_id(H5fid);
+            assertTrue("H5.H5VLget_connector_id", native_id >= 0);
+
+            /*
+             * If HDF5_VOL_CONNECTOR is set, this might not be the
+             * native connector. Only check for the native connector
+             * if this isn't set.
+             */
+            String connector = System.getenv("HDF5_VOL_CONNECTOR");
+            if (connector == null)
+                assertEquals(HDF5Constants.H5VL_NATIVE, native_id);
         }
         catch (Throwable err) {
             err.printStackTrace();
             fail("H5.H5VLget_connector_id " + err);
         }
+        finally {
+            if (H5fid > 0) {
+                try {H5.H5Fclose(H5fid);} catch (Exception ex) {}
+            }
+            _deleteFile(H5_FILE);
+        }
+    }
+
+    @Test
+    public void testH5VLget_connector_id_by_name() {
+        try {
+            long native_id = H5.H5VLget_connector_id_by_name(HDF5Constants.H5VL_NATIVE_NAME);
+            assertTrue("H5.H5VLget_connector_id_by_name H5VL_NATIVE_NAME", native_id >= 0);
+            assertEquals(HDF5Constants.H5VL_NATIVE, native_id);
+        }
+        catch (Throwable err) {
+            err.printStackTrace();
+            fail("H5.H5VLget_connector_id_by_name " + err);
+        }
+    }
+
+    @Test
+    public void testH5VLget_connector_id_by_value() {
+        try {
+            long native_id = H5.H5VLget_connector_id_by_value(HDF5Constants.H5VL_NATIVE_VALUE);
+            assertTrue("H5.H5VLget_connector_id_by_value H5VL_NATIVE_VALUE", native_id >= 0);
+            assertEquals(HDF5Constants.H5VL_NATIVE, native_id);
+        }
+        catch (Throwable err) {
+            err.printStackTrace();
+            fail("H5.H5VLget_connector_id_by_value " + err);
+        }
     }
 
     @Test
@@ -90,7 +137,15 @@ public class TestH5VL {
 
         try {
             String native_name = H5.H5VLget_connector_name(H5fid);
-            assertTrue("H5.H5VLget_connector_name H5VL_NATIVE", native_name.compareToIgnoreCase(HDF5Constants.H5VL_NATIVE_NAME)==0);
+
+            /*
+             * If HDF5_VOL_CONNECTOR is set, this might not be the
+             * native connector. Only check for the native connector
+             * if this isn't set.
+             */
+            String connector = System.getenv("HDF5_VOL_CONNECTOR");
+            if (connector == null)
+                assertTrue("H5.H5VLget_connector_name H5VL_NATIVE", native_name.compareToIgnoreCase(HDF5Constants.H5VL_NATIVE_NAME)==0);
         }
         catch (Throwable err) {
             err.printStackTrace();