Skip to content
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

Fixing tests #472

Closed
wants to merge 2 commits into from
Closed

Fixing tests #472

wants to merge 2 commits into from

Conversation

lajones
Copy link
Contributor

@lajones lajones commented Jul 30, 2014

@anpete Start server from temp dir so doesn't interfere with git clean. Also add retry functionality to ensure connection is available.

@anpete
Copy link
Contributor

anpete commented Jul 30, 2014

:shipit:

…it clean. Also add retry functionality to ensure connection is available.
@lajones
Copy link
Contributor Author

lajones commented Jul 30, 2014

Merged with # 233e497

@lajones lajones closed this Jul 30, 2014
@lajones lajones deleted the lajones-140730_FixTestFailures_01 branch July 30, 2014 20:42
@lajones lajones reopened this Jul 30, 2014
var tempPath = Environment.GetEnvironmentVariable("TMP");
if (tempPath == null)
{
throw new Exception("User does not have a TMP environment variable defined");
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Have updated this to look for TEMP as well as TMP - just in case. Now I'll only throw if can't find either of them.

@lajones
Copy link
Contributor Author

lajones commented Jul 30, 2014

Committed with # 41b04ff

@lajones lajones closed this Aug 8, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants