Tests failing on master from VS 2017 Test Explorer about vts HOT 8 CLOSED

I updated to 15.8.2 and still had 4 unit tests failing. Then lmalenfant suggested that I revert my laptop to English. I had set it to French to use our WPF GUI in french. When I rebooted in English and ran the unit tests, all pass now.

from vts.

Hi dcuccia,

I am very sorry you're experiencing this! Typically when our code is pushed to Github all unit tests pass. I am using VS 2017 (15.8.1) as well and am finding 4 errors. I thought possibly it was due to your using the Text Explorer (which uses Test Runner I think), but I ran the tests using the Text Explorer and Nunit and I get the same 4 errors in both:

I'm not sure why you're experiencing more errors. I too ran BuildTestRelease.bat and all worked fine so I was relieved for that. The errors I'm finding have to do with reading the SpectralDatabase and in particular reading in the data from the file Spectroscopy/Resources/Spectra.txt. It reads the header of the file successfully but on the first line of data it reads with the code:
var valEntry = convert ? Convert.ToDouble(row[i+1]).ConvertCoefficient(chromophoreList[i].AbsorptionCoefficientUnit, chromophoreList[i].MolarUnit) : Convert.ToDouble(row[i+1]);

the code excepts with an error saying the data is not in the correct format. It could be due to the removal of Automapper from our solution. I tried adding the package back in, but this did not solve the problem.

I'll keep debugging my 4 errors and hopefully by solving them, we might have a clue about the 70 additional errors you're encountering.

from vts.

I just pulled a clean clone and added the Microsoft.NET.Test.Sdk NuGet package back in because it was still being used. All unit tests pass when I run them from both Resharper and Test Explorer.

The only difference I can see is that my version of Visual Studio is 15.6.7, I will update to 15.8.2 and see if this changes anything.

from vts.

from vts.

Just curious as to whether dcuccia was able to reproduce the failing unit tests on another machine.

from vts.

If we are not informed of way to reproduce this issue, we plan to close it in a week. Thanks.

from vts.

from vts.

Thanks for letting me know. Good to hear there isn't a problem.

from vts.