Wrap exceptions from array constructors in TargetInvocationException (#69440)

Fixes #69336

Author: jkotas

src/coreclr/nativeaot/System.Private.Reflection.Core/src/System/Reflection/Runtime/MethodInfos/CustomMethodInvoker.cs

@@ -42,7 +42,7 @@ protected sealed override object Invoke(object thisObject, object[] arguments, B
             {
                 result = _action(thisObject, convertedArguments, _thisType);
             }
-            catch (Exception e) when (wrapInTargetInvocationException && ((_options & InvokerOptions.DontWrapException) == 0))
+            catch (Exception e) when (wrapInTargetInvocationException)
             {
                 throw new TargetInvocationException(e);
             }

src/coreclr/nativeaot/System.Private.Reflection.Core/src/System/Reflection/Runtime/MethodInfos/InvokerOptions.cs

@@ -8,6 +8,5 @@ internal enum InvokerOptions
     {
         None = 0x00000000,
         AllowNullThis = 0x00000001,     // Don't raise an exception if the "thisObject" parameter to Invoker is null.
-        DontWrapException = 0x00000002, // Don't wrap target exceptions in TargetInvocationException.
     }
 }

src/coreclr/nativeaot/System.Private.Reflection.Core/src/System/Reflection/Runtime/TypeInfos/RuntimeArrayTypeInfo.cs

@@ -71,7 +71,7 @@ internal sealed override IEnumerable<RuntimeConstructorInfo> SyntheticConstructo
                         SyntheticMethodId.ArrayCtor,
                         arrayType,
                         ctorParameters,
-                        InvokerOptions.AllowNullThis | InvokerOptions.DontWrapException,
+                        InvokerOptions.AllowNullThis,
                         delegate (object _this, object[] args, Type thisType)
                         {
                             int[] lengths = new int[rank];
@@ -109,7 +109,7 @@ internal sealed override IEnumerable<RuntimeConstructorInfo> SyntheticConstructo
                             SyntheticMethodId.ArrayCtorJagged + parameterCount,
                             arrayType,
                             ctorParameters,
-                            InvokerOptions.AllowNullThis | InvokerOptions.DontWrapException,
+                            InvokerOptions.AllowNullThis,
                             delegate (object _this, object[] args, Type thisType)
                             {
                                 int[] lengths = new int[args.Length];
@@ -145,7 +145,7 @@ internal sealed override IEnumerable<RuntimeConstructorInfo> SyntheticConstructo
                         SyntheticMethodId.ArrayMultiDimCtor,
                         arrayType,
                         ctorParameters,
-                        InvokerOptions.AllowNullThis | InvokerOptions.DontWrapException,
+                        InvokerOptions.AllowNullThis,
                         delegate (object _this, object[] args, Type thisType)
                         {
                             int[] lengths = new int[rank];

src/libraries/System.Reflection/tests/ConstructorInfoTests.cs

@@ -115,7 +115,6 @@ public void Invoke_OneDimensionalArray()
 
         [Fact]
         [ActiveIssue("https://github.com/dotnet/runtime/issues/67457", TestRuntimes.Mono)]
-        [ActiveIssue("https://github.com/dotnet/runtime/issues/69336", typeof(PlatformDetection), nameof(PlatformDetection.IsNativeAot))]
         public void Invoke_OneDimensionalArray_NegativeLengths_ThrowsOverflowException()
         {
             ConstructorInfo[] constructors = GetConstructors(typeof(object[]));