test: Change test to validate we cannot access DirectBuffer

Author: torito

memory/memory-core/src/main/java/org/apache/arrow/memory/util/MemoryUtil.java

@@ -87,13 +87,14 @@ public Object run() {
       try {
         addressField.setAccessible(true);
         maybeOffset = UNSAFE.objectFieldOffset(addressField);
-      } catch (InaccessibleObjectException e){
+      } catch (InaccessibleObjectException e) {
         maybeOffset = -1;
-        logger.debug("Cannot access the address field of java.nio.Buffer. DirectBuffer operations wont be available", e);
+        logger.debug(
+            "Cannot access the address field of java.nio.Buffer. DirectBuffer operations wont be available",
+            e);
       }
       BYTE_BUFFER_ADDRESS_OFFSET = maybeOffset;
 
-
       Constructor<?> directBufferConstructor;
       long address = -1;
       final ByteBuffer direct = ByteBuffer.allocateDirect(1);
@@ -168,7 +169,7 @@ public Object run() {
    * @return address of the underlying memory.
    */
   public static long getByteBufferAddress(ByteBuffer buf) {
-    if(BYTE_BUFFER_ADDRESS_OFFSET != -1) {
+    if (BYTE_BUFFER_ADDRESS_OFFSET != -1) {
       return UNSAFE.getLong(buf, BYTE_BUFFER_ADDRESS_OFFSET);
     }
     throw new UnsupportedOperationException(

memory/memory-core/src/test/java/org/apache/arrow/memory/TestOpens.java

@@ -20,6 +20,7 @@
 import static org.junit.jupiter.api.Assertions.assertTrue;
 import static org.junit.jupiter.api.condition.JRE.JAVA_16;
 
+import org.apache.arrow.memory.util.MemoryUtil;
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.condition.EnabledForJreRange;
 
@@ -34,18 +35,15 @@ public void testMemoryUtilFailsLoudly() {
     Throwable e =
         assertThrows(
             Throwable.class,
-            () -> {
-              BufferAllocator allocator = new RootAllocator();
-              allocator.close();
-            });
+            () -> MemoryUtil.directBuffer(0, 10));
     boolean found = false;
     while (e != null) {
-      e = e.getCause();
-      if (e instanceof RuntimeException
-          && e.getMessage().contains("Failed to initialize MemoryUtil")) {
+      if (e instanceof UnsupportedOperationException
+          && e.getMessage().contains("java.nio.DirectByteBuffer.<init>(long, int) not available")) {
         found = true;
         break;
       }
+      e = e.getCause();
     }
     assertTrue(found, "Expected exception was not thrown");
   }