spekt / junit.testlogger

JUnit test logger for vstest platform (<= v3.x)
MIT License
77 stars 15 forks source link

4 test fixtures tests are not logged #5

Closed Siphonophora closed 5 years ago

Siphonophora commented 5 years ago

Resolves #4

The acceptance test counts have been corrected from 48 to 52. The four missing tests represent tests with fixture data.

Confirmation this is a correction and not introducing an error:

Output of dotnet test on test\assets\JUnit.Xml.TestLogger.NetCore.Tests:

Test Run Failed.
Total tests: 52
     Passed: 24
     Failed: 14
    Skipped: 6
 Total time: 2.2389 Seconds

Output of dotnet test -t:

The following Tests are available:
    TestA
    TestD
    TestC
    TestB
    TestE
    TestE
    TestData(1,"A")
    TestData(1,"B")
    ... 

Additionally, by renaming the Test() methods to TestA, TestB... I verified that TestE was not present in the log produced by the master branch, but is now present twice per UnitTest folder, matching the list of tests which dotnet discovers.