-
Notifications
You must be signed in to change notification settings - Fork 177
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
fix: Build green on linux and windows #33
Conversation
Update the readme to include usage and mutator documentation
closes #6 |
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.
I don't like the fact that some unit tests don't run on all OS's. What is necessary to make it work?
@simondel what do you think about this?
- Ubuntu | ||
- Visual Studio 2017 | ||
init: | ||
- ps: git config --global core.autocrlf false |
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.
Can't we add this to a local .gitattributes
file?
src/Stryker.CLI/README.md
Outdated
@@ -0,0 +1,49 @@ | |||
# Stryker.NET Command Line Interface |
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.
Why is this part of "fixing the build"?
}); | ||
metadataReferenceProviderMock.Setup(x => x.CreateFromFile(It.IsAny<string>())) | ||
.Returns(() => null) | ||
.Callback((string reference) => foundReferences.Add(reference)); | ||
|
||
string project = "C:/ProjectFolder/ExampleProject/ExampleProject.Test.csproj"; | ||
string project = @"C:\ProjectFolder\ExampleProject\ExampleProject.Test.csproj".Replace('\\', Path.DirectorySeparatorChar); |
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.
I think it's clearer to use Path.Combine
. This method is tailor made for this action.
} | ||
|
||
[Fact] | ||
[SkippableFact] |
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.
What does this mean? Are these tests only ran on windows? I don't like that.
Is there a clear path to make them run on linux in the near future?
|
||
namespace Stryker.Core.UnitTest.Mocks | ||
{ | ||
public class MockFile |
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.
What's the relationship between this class and MockFileBase
?
@@ -0,0 +1,30 @@ | |||
//using System; |
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.
Remove this file.
@@ -3,6 +3,7 @@ | |||
using Shouldly; |
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.
The name of this file has incorrect casing.
The appveyor build was failing on both linux and windows. I had to update some tests together with the build definition to fix this. On linux some tests will be skipped for now, this will be improved in the future.
Also the readme has been updated with a build badge together with a waffle.io badge.