fix: handle primitive arrays in 'in' operator and .length field access

[chenjunwenhao]

Summary

Fix ClassCastException when the in operator or .length field access encounters primitive arrays (byte[], int[], char[], double[], etc.) returned by Java methods or passed via execution context.

Problem

Two locations in the codebase cast arrays directly to Object[], which crashes for primitive arrays:

1. BaseBinaryOperator.inOperator() (line 316):

   Object[] rightArray = (Object[])rightOperand;  // crashes for byte[], int[], etc.

2. ReflectLoader.loadField() (line 85):

   return new DataValue(((Object[])bean).length);  // crashes for byte[], int[], etc.

While QLExpress's new int[]{} syntax creates Integer[] internally (via BuiltInTypesSet), primitive arrays from Java method returns (e.g., String.getBytes() → byte[], String.toCharArray() → char[]) or context variables trigger this crash.

Reproduction:

Express4Runner runner = new Express4Runner();
// Crashes with: ClassCastException: class [B cannot be cast to class [Ljava.lang.Object;
runner.execute("65 in \"ABC\".getBytes()", Collections.emptyMap(), QLOptions.DEFAULT_OPTIONS);
runner.execute("bytes = \"hello\".getBytes(); return bytes.length;", Collections.emptyMap(), QLOptions.DEFAULT_OPTIONS);

Fix

Use java.lang.reflect.Array.getLength() and Array.get() instead of Object[] cast. This is consistent with other array-handling code in the codebase (ForEachInstruction, IndexInstruction, SliceInstruction, SpreadGetFieldInstruction, etc.), all of which correctly use java.lang.reflect.Array for array operations.

Tests

Added two test suite files:

• testsuite/java/operator/in_primitive_array.ql — tests in and not_in with byte[] from String.getBytes()
• testsuite/java/property/primitive_array_length.ql — tests .length with byte[] from String.getBytes() and char[] from String.toCharArray()

Verified that:

• Tests fail with ClassCastException before the fix
• Tests pass after the fix