[Bug] Fix tests not found in dotnet core projects

Author: robertcoltheart

Source/Machine.VSTestAdapter/Discovery/BuiltIn/SourceCodeLocationFinder.cs

@@ -1,13 +1,12 @@
 using System;
-using System.Collections.Generic;
 using System.Linq;
 using System.Reflection;
 using Mono.Cecil;
 using Mono.Cecil.Cil;
 
 namespace Machine.VSTestAdapter.Discovery.BuiltIn
 {
-    public class SourceCodeLocationFinder
+    public class SourceCodeLocationFinder : IDisposable
     {
         private readonly Lazy<AssemblyDefinition> assemblyDefinition;
 
@@ -36,11 +35,15 @@ private AssemblyDefinition LoadAssembly(string assemblyFilePath)
             });
         }
 
+        public void Dispose()
+        {
+            Assembly?.Dispose();
+        }
+
         private AssemblyDefinition Assembly {
             get { return assemblyDefinition.Value; }
         }
 
-
         private string HandleNestedTypeName(string type)
         {
             return type.Replace("+", "/");
@@ -63,23 +66,18 @@ private SourceCodeLocationInfo GetFieldLocationCore(TypeDefinition type, string
             if (!constructorDefinition.HasBody)
                 return null;
 
-#if NETCOREAPP1_1
             if (constructorDefinition.DebugInformation == null)
                 return null;
-#endif
 
             Instruction instruction = constructorDefinition.Body.Instructions
-                .Where(x => x.Operand != null &&
+                .SingleOrDefault(x => x.Operand != null &&
                             x.Operand.GetType().IsAssignableFrom(typeof(FieldDefinition)) &&
-                            ((MemberReference)x.Operand).Name == fieldFullName).SingleOrDefault();
+                            ((MemberReference)x.Operand).Name == fieldFullName);
 
             while (instruction != null)
             {
-#if NETCOREAPP1_1
                 SequencePoint sequencePoint = constructorDefinition.DebugInformation?.GetSequencePoint(instruction);
-#else
-                SequencePoint sequencePoint = instruction.SequencePoint;
-#endif
+
                 if (sequencePoint != null && !IsHidden(sequencePoint))
                 {
                     return new SourceCodeLocationInfo()
@@ -97,13 +95,7 @@ private SourceCodeLocationInfo GetFieldLocationCore(TypeDefinition type, string
 
         private bool IsHidden(SequencePoint sequencePoint)
         {
-
-#if NETCOREAPP1_1
             return sequencePoint.IsHidden;
-#else
-            const int lineNumberIndicatingHiddenLine = 0xfeefee;
-            return sequencePoint.StartLine == lineNumberIndicatingHiddenLine;
-#endif
         }
     }
 

Source/Machine.VSTestAdapter/Discovery/BuiltIn/TestDiscoverer.cs

@@ -24,9 +24,10 @@ public IEnumerable<MSpecTestCase> DiscoverTests(string assemblyPath)
             Assembly assembly = AssemblyHelper.Load(assemblyPath);
             IEnumerable<Context> contexts = assemblyExplorer.FindContextsIn(assembly);
 
-            SourceCodeLocationFinder locationFinder = new SourceCodeLocationFinder(assemblyPath);
-
-            return contexts.SelectMany(context => CreateTestCase(context, locationFinder)).ToList();
+            using (SourceCodeLocationFinder locationFinder = new SourceCodeLocationFinder(assemblyPath))
+            {
+                return contexts.SelectMany(context => CreateTestCase(context, locationFinder)).ToList();
+            }
         }
 
         private IEnumerable<MSpecTestCase> CreateTestCase(Context context, SourceCodeLocationFinder locationFinder)

Source/Machine.VSTestAdapter/Machine.VSTestAdapter.csproj

@@ -12,7 +12,7 @@
     <Product>Machine.Specifications.Runner.VisualStudio</Product>
     <AssemblyTitle>Machine.Specifications.Runner.VisualStudio</AssemblyTitle>
     <PackageId>Machine.Specifications.Runner.VisualStudio </PackageId>
-    <Version>1.0.0</Version>
+    <Version>2.0.0</Version>
     <AssemblyVersion>1.0.0.0</AssemblyVersion>
     <Authors>https://github.com/machine/machine.vstestadapter</Authors>
     <PackageProjectUrl>https://github.com/machine/machine.vstestadapter</PackageProjectUrl>
@@ -25,26 +25,25 @@
   <PropertyGroup Condition="'$(TargetFramework)'=='netcoreapp1.1'">
     <DefineConstants>NETSTANDARD</DefineConstants>
   </PropertyGroup>
-
-  <ItemGroup Condition="'$(TargetFramework)'==$(_TargetDotNetCore)">
+  
+  <ItemGroup>
     <PackageReference Include="Mono.Cecil" Version="[0.10-*, 0.11)" />
-    <PackageReference Include="Microsoft.TestPlatform.ObjectModel" Version="[15.0.0,16)">
-      <PrivateAssets>All</PrivateAssets> <!-- Only a reference during build/dev time -->
-    </PackageReference>
     <PackageReference Include="Machine.Specifications" Version="[0.11.0, 1.0)">
       <PrivateAssets>All</PrivateAssets>
       <!-- Only a reference during build/dev time -->
     </PackageReference>
   </ItemGroup>
+
+  <ItemGroup Condition="'$(TargetFramework)'==$(_TargetDotNetCore)">
+    <PackageReference Include="Microsoft.TestPlatform.ObjectModel" Version="[15.0.0,16)">
+      <PrivateAssets>All</PrivateAssets> <!-- Only a reference during build/dev time -->
+    </PackageReference>
+  </ItemGroup>
 
   <ItemGroup Condition="'$(TargetFramework)'=='$(_TargetDotNetFramework)'">
-    <PackageReference Include="Mono.Cecil" Version="0.9.6.1" />
     <PackageReference Include="Microsoft.TestPlatform.ObjectModel" Version="[11.0.0,16.0.0)">
       <PrivateAssets>All</PrivateAssets> <!-- Only a reference during build/dev time -->
     </PackageReference>
-    <PackageReference Include="Machine.Specifications" Version="[0.11.0, 1.0)">
-      <PrivateAssets>All</PrivateAssets> <!-- Only a reference during build/dev time -->
-    </PackageReference>
   </ItemGroup>
 
   <ItemGroup>