Documenting changes done to copied test method due to inflexibility / visibility of methods / constructors.

Author: mjmbischoff

x-pack/plugin/esql/src/test/java/org/elasticsearch/xpack/esql/expression/function/scalar/multivalue/MvContainsTests.java

@@ -286,7 +286,11 @@ private static void bytesRefs(List<TestCaseSupplier> suppliers) {
         }));
     }
 
-    // Adjusted from static method anyNullIsNull in {@code AbstractScalarFunctionTestCase#}
+    // Adjusted from static method anyNullIsNull in {@code AbstractFunctionTestCase#}
+    // - changed logic to expect a Boolean as an outcome and alternative evaluators (IsNullEvaluator, ConstantTrue)
+    // - constructor TestCase that's used has default access which we can't access, using public constructor variant as a replacement
+    // - Added prefix to generated tests for my sanity.
+    // - changed construction of new lists by copying them and updating the entries where necessary instead of regenerating
     protected static List<TestCaseSupplier> anyNullIsNull(
         List<TestCaseSupplier> testCaseSuppliers,
         ExpectedType expectedType,