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
IntegrationTests\UpdateVerb #193
base: release/v0.5.0
Are you sure you want to change the base?
Conversation
Build Failure :( |
How do you rerun the tests? |
@powellm123 I believe you just have to push new code. There may be a way to go the appveyor page and tell it to re-run a build. But I'm not sure. |
Adding some logging to see if we can figure out why this fails on AppVeyor.
|
||
namespace Lib1 | ||
{ | ||
public class StringUtils |
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.
Is this indentation change intentional? If this is the change, I would prefer something else. This kind of indentation stuff is a pet peeve.
@@ -30,21 +57,47 @@ public void InvalidVerb_Returns_InvalidCommand() | |||
[Test] | |||
public void CloneVerb_ShouldCloneAllRepos() |
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.
This is all in the HelpPageScenario, which should focus only on Help page work flows. You should create a new Scenario file for Cloning.
Assert.IsTrue(lib2Exists); | ||
|
||
[Test] | ||
public void UpdateVerb_CloneReposChangeLib1_ShouldUpdateArtifacts() |
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.
This is all in the HelpPageScenario, which should focus only on Help page work flows. You should create a new Scenario file for Updating.
Adding Integration test that tests the Update Verb