[FAB-10957] use 0 as bad port for connect fail

sykesm · mastersingh24 · commit a45c3bfb8839 · 2018-07-14T12:48:57.000Z
The test tries to point to a port that has no local listeners to test
behavior when connecting to the orderer fails. Instead of using 9999,
use the invalid port of 0 as no process can listen on that port.

Change-Id: I6f63b80e7798336a5ddf2c58b55c44fc8b180906
Signed-off-by: Matthew Sykes &lt;sykesmat@us.ibm.com&gt;
diff --git a/peer/channel/create_test.go b/peer/channel/create_test.go
@@ -324,8 +324,8 @@ func TestCreateChainWithTimeoutErr(t *testing.T) {
 		assert.Contains(t, err.Error(), "timeout waiting for channel creation")
 	}
 
-	// failure - times out connecting to orderer
-	args = []string{"create", "-c", mockchain, "-o", "localhost:9999", "--connTimeout", "10ms"}
+	// failure - point to bad port and time out connecting to orderer
+	args = []string{"create", "-c", mockchain, "-o", "localhost:0", "--connTimeout", "10ms"}
 	channelCmd.SetArgs(args)
 
 	if err := channelCmd.Execute(); err == nil {

Original file line number	Diff line number	Diff line change
`@@ -324,8 +324,8 @@ func TestCreateChainWithTimeoutErr(t *testing.T) {`
`324`	`324`	`assert.Contains(t, err.Error(), "timeout waiting for channel creation")`
`325`	`325`	`}`
`326`	`326`
`327`		`- // failure - times out connecting to orderer`
`328`		`- args = []string{"create", "-c", mockchain, "-o", "localhost:9999", "--connTimeout", "10ms"}`
	`327`	`+ // failure - point to bad port and time out connecting to orderer`
	`328`	`+ args = []string{"create", "-c", mockchain, "-o", "localhost:0", "--connTimeout", "10ms"}`
`329`	`329`	`channelCmd.SetArgs(args)`
`330`	`330`
`331`	`331`	`if err := channelCmd.Execute(); err == nil {`