microsoft / testfx

MSTest framework and adapter
MIT License
679 stars 246 forks source link

Introduce `[STATestClass]` #3188

Open engyebrahim opened 1 week ago

engyebrahim commented 1 week ago

fix: https://github.com/microsoft/testfx/issues/2688

fforjan commented 5 days ago

@Evangelink by introduce a new subclass, does it mean any code analyzer will have issue because it will not find a TestClass but a STATestClass and need to be made aware of it ?

Evangelink commented 5 days ago

@Evangelink by introduce a new subclass, does it mean any code analyzer will have issue because it will not find a TestClass but a STATestClass and need to be made aware of it ?

For our analyzers we are checking derived from TestClassAttribute so they would remain functional. For third-party analyzers for MSTest, I cannot comment without knowing their logic.