Update Windows unit tests

cloudfoundry-attic / consul-release

This is a BOSH release for consul.

Apache License 2.0

10 stars 30 forks source link

Update Windows unit tests #50

Closed sunjayBhatia closed 7 years ago

sunjayBhatia commented 7 years ago

We updated the unit tests with the following:

skip testing running consul as a daemon on Windows since we run it as a foreground process
clean file paths so they do not break cross platform

The tests currently pass on Windows and *nix

cfdreddbot commented 7 years ago

Hey sunjayBhatia!

Thanks for submitting this pull request! I'm here to inform the recipients of the pull request that you and the commit authors have already signed the CLA.

cf-gitbot commented 7 years ago

We have created an issue in Pivotal Tracker to manage this:

https://www.pivotaltracker.com/story/show/132693763

The labels on this github issue will be updated when the story is started.