Added more verbose error message when calling method using reflection and target type does not match passed type (#98129)

* Added more verbose error message when calling method using reflection and target type does not match passed type (fixes #97022)

* Fixed resource string as per PR comment

* Fixed the test related to a resource string change

* Modified test to have a contains check rather than validating the English string.

* Modifications to AOT implementation for validating target type when a method is invoked using reflection, inline with other PR changes

Author: AndyBevan

src/coreclr/nativeaot/System.Private.CoreLib/src/Internal/Reflection/Core/Execution/MethodBaseInvoker.cs

@@ -57,7 +57,7 @@ protected static void ValidateThis(object thisObject, RuntimeTypeHandle declarin
                 throw new TargetException(SR.RFLCT_Targ_StatMethReqTarg);
 
             if (!RuntimeAugments.IsAssignable(thisObject, declaringTypeHandle))
-                throw new TargetException(SR.RFLCT_Targ_ITargMismatch);
+                throw new TargetException(SR.Format(SR.RFLCT_Targ_ITargMismatch_WithType, declaringTypeHandle.GetRuntimeTypeInfoForRuntimeTypeHandle().Name, thisObject.GetType().Name));
         }
     }
 }

src/libraries/System.Private.CoreLib/src/Resources/Strings.resx

@@ -3344,6 +3344,9 @@
   <data name="RFLCT_Targ_ITargMismatch" xml:space="preserve">
     <value>Object does not match target type.</value>
   </data>
+  <data name="RFLCT_Targ_ITargMismatch_WithType" xml:space="preserve">
+    <value>Object type {0} does not match target type {1}.</value>
+  </data>	  
   <data name="RFLCT_Targ_StatFldReqTarg" xml:space="preserve">
     <value>Non-static field requires a target.</value>
   </data>
@@ -4307,4 +4310,4 @@
   <data name="WasmThreads_BlockingWaitNotSupportedOnJSInterop" xml:space="preserve">
     <value>Blocking wait is not supported on the JS interop threads.</value>
   </data>
-</root>
+</root>

src/libraries/System.Private.CoreLib/src/System/Reflection/MethodInvokerCommon.cs

@@ -96,7 +96,7 @@ internal static void ValidateInvokeTarget(object? target, MethodBase method)
 
             if (!method.DeclaringType!.IsInstanceOfType(target))
             {
-                throw new TargetException(SR.RFLCT_Targ_ITargMismatch);
+                throw new TargetException(SR.Format(SR.RFLCT_Targ_ITargMismatch_WithType, method.DeclaringType.Name, target.GetType().Name));
             }
         }
 

src/libraries/System.Runtime/tests/System.Reflection.Tests/DefaultBinderTests.cs

@@ -107,6 +107,18 @@ public static void DefaultBinderNamedParametersSkippedAndOutOfOrderTest()
             Assert.Equal("MethodMoreParameters", method.Name);
         }
 
+        [Fact]
+        public void InvokeWithIncorrectTargetTypeThrowsCorrectException()
+        {
+            Type t = typeof(Sample);
+            object incorrectInstance = Activator.CreateInstance(t);
+            MethodInvoker invoker = MethodInvoker.Create(typeof(Test).GetMethod(nameof(Test.TestMethod)));
+
+            TargetException ex = Assert.Throws<TargetException>(() => invoker.Invoke(obj: incorrectInstance, "NotAnInt"));
+            Assert.Contains(nameof(Test), ex.Message);
+            Assert.Contains(nameof(Sample), ex.Message);
+        }
+
         [Fact]
         public static void InvokeWithNamedParameters1st2ndTest()
         {