Skip to content
This repository has been archived by the owner on Jun 20, 2024. It is now read-only.

Commit

Permalink
Removed dependency on jq in test #175.
Browse files Browse the repository at this point in the history
If the machine coordinating the test runs does not have jq, then test #175 fails with:
> test #1 "weave_on test-0.lon1.weave-net report | jq -r '.Router.Targets[] | tostring' | sort" failed:
>     expected "test-1.lon1.weave-net
> test-2.lon1.weave-net"
>     got nothing
This change completely removes the dependency on jq to make the test more robust and avoid potential confusion.
  • Loading branch information
marccarre committed Jan 9, 2017
1 parent 7328c3f commit 4a7e205
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion test/175_target_peer_persistence_3_test.sh
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ assert_targets() {
HOST=$1
shift
EXPECTED=$(for TARGET in $@; do echo $TARGET; done | sort)
assert "weave_on $HOST report | jq -r '.Router.Targets[] | tostring' | sort" "$EXPECTED"
assert "weave_on $HOST report -f '{{range .Router.Targets}}{{.}}{{\"\n\"}}{{end}}' | sort" "$EXPECTED"
}

start_suite "Check Docker restart uses persisted peer list"
Expand Down

0 comments on commit 4a7e205

Please sign in to comment.