summaryrefslogtreecommitdiff
path: root/tests
diff options
context:
space:
mode:
authorJeremy Koritzinsky <jkoritzinsky@gmail.com>2019-01-25 08:37:43 -0800
committerGitHub <noreply@github.com>2019-01-25 08:37:43 -0800
commit8568186a3b918c0239bf6485d56594843b9f544d (patch)
tree608a5ab4ed781611e46934529e33d65ce9143e53 /tests
parente0d680101a93797c9e1a049bf582a874e2cb4113 (diff)
downloadcoreclr-8568186a3b918c0239bf6485d56594843b9f544d.tar.gz
coreclr-8568186a3b918c0239bf6485d56594843b9f544d.tar.bz2
coreclr-8568186a3b918c0239bf6485d56594843b9f544d.zip
Remove reflection-based wraper for AssemblyDependencyResolver. (#22195)
Now that AssemblyDependencyResolver is exposed in CoreFX and has propogated back to CoreCLR, we can remove the reflection-based wrapper for the unit tests for it.
Diffstat (limited to 'tests')
-rw-r--r--tests/src/Loader/AssemblyDependencyResolverTests/AssemblyDependencyResolver.cs59
-rw-r--r--tests/src/Loader/AssemblyDependencyResolverTests/AssemblyDependencyResolverTests.cs1
-rw-r--r--tests/src/Loader/AssemblyDependencyResolverTests/AssemblyDependencyResolverTests.csproj3
-rw-r--r--tests/src/Loader/AssemblyDependencyResolverTests/HostPolicyMock.cs5
-rw-r--r--tests/src/Loader/AssemblyDependencyResolverTests/InvalidHostingTest.cs1
-rw-r--r--tests/src/Loader/AssemblyDependencyResolverTests/NativeDependencyTests.cs1
6 files changed, 6 insertions, 64 deletions
diff --git a/tests/src/Loader/AssemblyDependencyResolverTests/AssemblyDependencyResolver.cs b/tests/src/Loader/AssemblyDependencyResolverTests/AssemblyDependencyResolver.cs
deleted file mode 100644
index 40d7a41f5e..0000000000
--- a/tests/src/Loader/AssemblyDependencyResolverTests/AssemblyDependencyResolver.cs
+++ /dev/null
@@ -1,59 +0,0 @@
-// Licensed to the .NET Foundation under one or more agreements.
-// The .NET Foundation licenses this file to you under the MIT license.
-// See the LICENSE file in the project root for more information.
-using System;
-using System.Reflection;
-
-namespace AssemblyDependencyResolverTests
-{
- /// <summary>
- /// Temporary until the actual public API gets propagated through CoreFX.
- /// </summary>
- public class AssemblyDependencyResolver
- {
- private object _implementation;
- private Type _implementationType;
- private MethodInfo _resolveAssemblyPathInfo;
- private MethodInfo _resolveUnmanagedDllPathInfo;
-
- public AssemblyDependencyResolver(string componentAssemblyPath)
- {
- _implementationType = typeof(object).Assembly.GetType("System.Runtime.Loader.AssemblyDependencyResolver");
- _resolveAssemblyPathInfo = _implementationType.GetMethod("ResolveAssemblyToPath");
- _resolveUnmanagedDllPathInfo = _implementationType.GetMethod("ResolveUnmanagedDllToPath");
-
- try
- {
- _implementation = Activator.CreateInstance(_implementationType, componentAssemblyPath);
- }
- catch (TargetInvocationException tie)
- {
- throw tie.InnerException;
- }
- }
-
- public string ResolveAssemblyToPath(AssemblyName assemblyName)
- {
- try
- {
- return (string)_resolveAssemblyPathInfo.Invoke(_implementation, new object[] { assemblyName });
- }
- catch (TargetInvocationException tie)
- {
- throw tie.InnerException;
- }
- }
-
- public string ResolveUnmanagedDllToPath(string unmanagedDllName)
- {
- try
- {
- return (string)_resolveUnmanagedDllPathInfo.Invoke(_implementation, new object[] { unmanagedDllName });
- }
- catch (TargetInvocationException tie)
- {
- throw tie.InnerException;
- }
- }
- }
-}
diff --git a/tests/src/Loader/AssemblyDependencyResolverTests/AssemblyDependencyResolverTests.cs b/tests/src/Loader/AssemblyDependencyResolverTests/AssemblyDependencyResolverTests.cs
index 883d569f96..9499bee724 100644
--- a/tests/src/Loader/AssemblyDependencyResolverTests/AssemblyDependencyResolverTests.cs
+++ b/tests/src/Loader/AssemblyDependencyResolverTests/AssemblyDependencyResolverTests.cs
@@ -4,6 +4,7 @@
using System;
using System.IO;
using System.Reflection;
+using System.Runtime.Loader;
using Xunit;
namespace AssemblyDependencyResolverTests
diff --git a/tests/src/Loader/AssemblyDependencyResolverTests/AssemblyDependencyResolverTests.csproj b/tests/src/Loader/AssemblyDependencyResolverTests/AssemblyDependencyResolverTests.csproj
index 28bd068fc4..ee13f905a8 100644
--- a/tests/src/Loader/AssemblyDependencyResolverTests/AssemblyDependencyResolverTests.csproj
+++ b/tests/src/Loader/AssemblyDependencyResolverTests/AssemblyDependencyResolverTests.csproj
@@ -13,10 +13,7 @@
<DefineConstants Condition="$(OSGroup) == 'Windows_NT'">WINDOWS</DefineConstants>
<DefineConstants Condition="$(OSGroup) == 'OSX'">OSX</DefineConstants>
</PropertyGroup>
- <PropertyGroup Condition="'$(Configuration)|$(Platform)' == 'Debug|x64'">
- </PropertyGroup>
<ItemGroup>
- <Compile Include="AssemblyDependencyResolver.cs" />
<Compile Include="AssemblyDependencyResolverTests.cs" />
<Compile Include="HostPolicyMock.cs" />
<Compile Include="InvalidHostingTest.cs" />
diff --git a/tests/src/Loader/AssemblyDependencyResolverTests/HostPolicyMock.cs b/tests/src/Loader/AssemblyDependencyResolverTests/HostPolicyMock.cs
index ca4d698576..37cedbc542 100644
--- a/tests/src/Loader/AssemblyDependencyResolverTests/HostPolicyMock.cs
+++ b/tests/src/Loader/AssemblyDependencyResolverTests/HostPolicyMock.cs
@@ -4,6 +4,7 @@
using System;
using System.IO;
using System.Runtime.InteropServices;
+using System.Runtime.Loader;
namespace AssemblyDependencyResolverTests
{
@@ -63,9 +64,9 @@ namespace AssemblyDependencyResolverTests
Path.Combine(testBasePath, hostPolicyFileName),
destinationPath);
- _assemblyDependencyResolverType = typeof(object).Assembly.GetType("System.Runtime.Loader.AssemblyDependencyResolver");
+ _assemblyDependencyResolverType = typeof(AssemblyDependencyResolver);
- // This is needed for marshalling of function pointers to work - requires private access to the CDR unfortunately
+ // This is needed for marshalling of function pointers to work - requires private access to the ADR unfortunately
// Delegate marshalling doesn't support casting delegates to anything but the original type
// so we need to use the original type.
_corehost_error_writer_fnType = _assemblyDependencyResolverType.GetNestedType("corehost_error_writer_fn", System.Reflection.BindingFlags.NonPublic);
diff --git a/tests/src/Loader/AssemblyDependencyResolverTests/InvalidHostingTest.cs b/tests/src/Loader/AssemblyDependencyResolverTests/InvalidHostingTest.cs
index 616b5ee8ac..c51b055e07 100644
--- a/tests/src/Loader/AssemblyDependencyResolverTests/InvalidHostingTest.cs
+++ b/tests/src/Loader/AssemblyDependencyResolverTests/InvalidHostingTest.cs
@@ -3,6 +3,7 @@
// See the LICENSE file in the project root for more information.
using System;
using System.IO;
+using System.Runtime.Loader;
using Xunit;
namespace AssemblyDependencyResolverTests
diff --git a/tests/src/Loader/AssemblyDependencyResolverTests/NativeDependencyTests.cs b/tests/src/Loader/AssemblyDependencyResolverTests/NativeDependencyTests.cs
index e7dcf201bc..43e721e45f 100644
--- a/tests/src/Loader/AssemblyDependencyResolverTests/NativeDependencyTests.cs
+++ b/tests/src/Loader/AssemblyDependencyResolverTests/NativeDependencyTests.cs
@@ -4,6 +4,7 @@
using System;
using System.IO;
using System.Runtime.InteropServices;
+using System.Runtime.Loader;
using Xunit;
namespace AssemblyDependencyResolverTests