Skip to content

Commit

Permalink
Merge pull request #51 from hudl/UpdateReadme
Browse files Browse the repository at this point in the history
Update README.md
  • Loading branch information
damtur authored Sep 22, 2016
2 parents 3c21b9d + 9002c40 commit 9e19e95
Showing 1 changed file with 9 additions and 5 deletions.
14 changes: 9 additions & 5 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -11,16 +11,20 @@ c.GetApps() // returns a map[String]fargo.Application
# Testing

Tests can be executed using docker container. See the below section to build and start
all the required containers. Once fargo container is running:
all the required containers. Once the Eureka containers are running, and fargo image is built then you can run the command as follows:

1. attach shell `docker attach fargo`
2. Checkout your test branch
3. execute `go test -v ./...`
Run:
```
docker run --rm -v "$PWD":/go/src/github.com/hudl/fargo -w /go/src/github.com/hudl/fargo hudloss/fargo:master go test -v ./...
```
Note: If you are running bash for Windows add `MSYS_NO_PATHCONV=1 ` at the beginning.

The tests may need to be run a couple of times to allow changes to propagate
between the two eureka servers. If the tests are failing, try running them again
between the two Eureka servers. If the tests are failing, try running them again
approximately 30 seconds later.

If you are adding new packages to godep you may want to update the `hudloss/fargo` image first.

# Known Issues

Until [this PR](https://github.com/mitchellh/vagrant/pull/2742) is in an
Expand Down

0 comments on commit 9e19e95

Please sign in to comment.