microsoft / testfx

MSTest framework and adapter
MIT License
767 stars 259 forks source link

IntelliSense support for MSTest.Sdk properties #4118

Open Youssef1313 opened 17 hours ago

Youssef1313 commented 17 hours ago

Currently, there doesn't seem to be any IntelliSense support in csproj. I think it's useful to have such support.

I'm seeing all MSBuild IntelliSense defined here (even for Razor).

So, we could just PR to MSBuild to add IntelliSense for MSTest.Sdk.

However, it will be a better idea if we can maintain the xsd ourselves and have it shipped inside MSTest.Sdk NuGet package. This will allow us to always have the right IntelliSense depending on the actual MSTest.Sdk version that the user is using! But I don't see a way MSBuild would allow it, which is probably Razor is on MSBuild side as well.

List of properties to have IntelliSense for:

Youssef1313 commented 17 hours ago

@rainersigwald Can you advise please if there is a way to provide our own xsd? Otherwise, are you okay with updating CommonTypes.xsd in MSBuild repo?

Evangelink commented 13 hours ago

I would also consider EnableMSTestRunner (bool) and all these properties https://github.com/microsoft/testfx/blob/main/src/Adapter/MSTest.TestAdapter/build/net/MSTest.TestAdapter.targets#L5-L8

@Youssef1313 I'll let you edit the lsit above

rainersigwald commented 1 hour ago

Unfortunately there's not currently a way to provide your own XSD--but we'd be happy to accept a PR to add these to CommonTypes.xsd.