search

microsoft / vstest

Visual Studio Test Platform is the runner and engine that powers test explorer and vstest.console.
MIT License
900 stars 323 forks source link

Add support to DiaSession for Fields #2923

Closed mikeblakeuk closed 3 years ago

mikeblakeuk commented 3 years ago

Steps to reproduce

Using MSpec, we want to remove Cecil but can't use DiaSession as it doesn't support Fields, only Methods .net 452 + netcore2.1

Expected behavior

Remove Mono.Cecil

Actual behavior

DiaSession is closed and support for fields doesn't exist

Diagnostic logs

https://github.com/machine/machine.specifications/issues/436 https://github.com/machine/machine.specifications.runner.visualstudio/blob/master/src/Machine.Specifications.Runner.VisualStudio/Discovery/BuiltIn/SourceCodeLocationFinder.cs#L18

Environment

Windows 10, 16.10 of Microsoft.TestPlatform.ObjectModel

image

mikeblakeuk commented 3 years ago

@mayankbansal018 thoughts?

jakubch1 commented 3 years ago

Opened ticket here: https://developercommunity.visualstudio.com/t/Add-support-to-DiaSession-for-Fields/1445302