For the tests its better to not use the timeout_commit, and to wait for all the
votes, because otherwise we can end up with timing dependencies in the testing
code which can lead to nondeterministic failures. That was part of the reason
for this change originally.
```
[ERROR] Unable to export dependencies to vendor directory: remove /home/vagrant/go/src/github.com/tendermint/tendermint/vendor/golang.org/x/sys/unix: directory not empty
```
Use 0x-prefixed hex strings in client
server: Decode hex string args
Encode all string args as 0x<hex> without trying to encode as JSON
Added tests for special string arguments
Fix server handling quoted string args
Added string arg handling test cases to bash test script
I am in favor of docker, but people say that running docker containers
inside another container is bad.
Things included in provision:
- docker (latest)
- jq
- curl
- shellcheck
- golang 1.7
- glide
Error #1:
```
Error response from daemon: network with name local_testnet already exists
```
Fixed by stopping and removing local_testnet containers and removing
the network
Error #2:
```
docker: Error response from daemon: Conflict. The name "/test_container_basic" is already in use by container a7cd15d479a964675e7f259de4ed852e7dfef85b447514728f437cd0b980a709. You have to remove (or rename) that container to beable to reuse that name..
```
Fixed by adding `--rm` flag.