test/p2p: name client conts so we dont need to rm them because circle

blockchain/reactor.go

@@ -231,6 +231,7 @@ FOR_LOOP:
 					break SYNC_LOOP
 				} else {
 					bcR.pool.PopRequest()
+					// TODO: use ApplyBlock instead of Exec/Commit/SetAppHash/Save
 					err := bcR.state.ExecBlock(bcR.evsw, bcR.proxyAppConn, first, firstPartsHeader)
 					if err != nil {
 						// TODO This is bad, are we zombie?
@@ -245,7 +246,7 @@ FOR_LOOP:
 						PanicQ(Fmt("Failed to commit block at application: %v", res))
 					}
 					bcR.store.SaveBlock(first, firstParts, second.LastCommit)
-					bcR.state.AppHash = res.Data // ...
+					bcR.state.AppHash = res.Data
 					bcR.state.Save()
 				}
 			}

test/p2p/a.sh

@@ -1,8 +0,0 @@
-#! /bin/bash
-
-h1=10
-h2=0
-while [ $h2 -lt $(($h1+3)) ]; do
-	echo "$h2"
-	h2=$(($h2+1))
-done

test/p2p/client.sh

@@ -3,7 +3,8 @@ set -eu
 
 DOCKER_IMAGE=$1
 NETWORK_NAME=$2
-CMD=$3
+ID=$3
+CMD=$4
 
 echo "starting test client container with CMD=$CMD"
 # run the test container on the local network
@@ -11,8 +12,7 @@ docker run -t \
 	-v $GOPATH/src/github.com/tendermint/tendermint/test/p2p/:/go/src/github.com/tendermint/tendermint/test/p2p \
 	--net=$NETWORK_NAME \
 	--ip=$(test/p2p/ip.sh "-1") \
-	--name test_container \
+	--name test_container_$ID \
 	--entrypoint bash \
 	$DOCKER_IMAGE $CMD
 
-docker rm -vf test_container

test/p2p/test.sh

@@ -4,11 +4,13 @@ set -eu
 DOCKER_IMAGE=$1
 NETWORK_NAME=local_testnet
 
+cd $GOPATH/src/github.com/tendermint/tendermint
+
 # start the testnet on a local network
 bash test/p2p/local_testnet.sh $DOCKER_IMAGE $NETWORK_NAME
 
 # test atomic broadcast
-bash test/p2p/client.sh $DOCKER_IMAGE $NETWORK_NAME test/p2p/atomic_broadcast/test.sh
+bash test/p2p/client.sh $DOCKER_IMAGE $NETWORK_NAME ab test/p2p/atomic_broadcast/test.sh
 
 # test fast sync (from current state of network)
 # run it on each of them
@@ -17,7 +19,9 @@ for i in `seq 1 $N`; do
 	echo "Testing fasysync on node $i"
 
 	# kill peer 
+	set +e # circle sigh :(
 	docker rm -vf local_testnet_$i
+	set -e 
 
 	# restart peer - should have an empty blockchain
 	SEEDS="$(test/p2p/ip.sh 1):46656"
@@ -26,7 +30,7 @@ for i in `seq 1 $N`; do
 	done
 	bash test/p2p/peer.sh $DOCKER_IMAGE $NETWORK_NAME $i $SEEDS
 
-	bash test/p2p/client.sh $DOCKER_IMAGE $NETWORK_NAME "test/p2p/fast_sync/test.sh $i"
+	bash test/p2p/client.sh $DOCKER_IMAGE $NETWORK_NAME fs_$i "test/p2p/fast_sync/test.sh $i"
 done
 echo ""
 echo "PASS"