search

jcansdale / TestDriven.Net-Issues

Issue tracking for TestDriven.Net
https://github.com/jcansdale/TestDriven.Net-Issues/issues
24 stars 2 forks source link

"Extension Already installed" #142

Closed sm-g closed 5 years ago

sm-g commented 5 years ago

In Tools>Extensions and Updates I see that there is new version of extension: CurrentVersion 4.1, NewVersion 4.3.18025. After i update it, I got "Extension Already installed".

VS: 15.8.7

jcansdale commented 5 years ago

This scenario is a little tricky. There will be test runners installed by the EXE/MSI, and test runners that come bundled with the VSIX. For example, the EXE/MSI install includes multiple NUnit test runners for legacy versions, but the VSIX only includes a test runner for NUnit 3+. I'll need to make sure that people don't lose functionality if they install the VSIX version over the top of an EXE/MSI install (or if it automatically updates).

jcansdale commented 5 years ago

Hi @sm-g,

Here is a new VSIX installer that can be installed over the top of a VSIX install: TestDriven.VSPackage_InstallOverMsi.zip

This includes a fix to prevent test runners from the VSIX and MSI from being considered twice (the version installed by the VSIX will be given priority). This is from a different PR branch to the ValueTuple fix, so don't expect tuples to work on this version.

You can see this fix working if you attempt to execute a class that doesn't use a known test framework. Previously you would have seen this: image

In this version you should only see the warning message once. Could you give it a try and let me know if you notice any problems?

Thanks, Jamie.

sm-g commented 5 years ago

Sorry for late response. I could not reproduce "a class that doesn't use a known test framework", so I just uninstall MSI and use VS "Extensions and updates" to install VSIX (just missed that in was two independent extensions).