-
Notifications
You must be signed in to change notification settings - Fork 26
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adding framework and assembly tokens for test output path #49
Conversation
src/NUnit.Xml.TestLogger.TestAdapter/NUnit.Xml.TestLogger.TestAdapter.csproj
Outdated
Show resolved
Hide resolved
OK I just added a small test for the log file tokens. Tried to be minimally invasive in existing tests without reinventing the wheel. |
{ | ||
string[] expectedResultsFiles = new string[] | ||
{ | ||
Path.Combine(DotnetTestFixture.RootDirectory, "NUnit.Xml.TestLogger.NetMulti.Tests.NetFramework46.test-results.xml"), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tests are failing in unix because these file names need to be case sensitive (caps NET).
NUnit.Xml.TestLogger.NetMulti.Tests.NETFramework46.test-results.xml
NUnit.Xml.TestLogger.NetMulti.Tests.NETCoreApp20.test-results.xml
🎆 🚀 Thank you! Tests are failing on unix due to a minor error, added a commit fixing it. Will merge after CI is green! |
When can I consume these changes via a package on nuget @codito? |
Just pushed an update! https://www.nuget.org/packages/NunitXml.TestLogger/2.1.41 |
For issue #24