Fix tests using invalid ID

estesp · estesp · commit b769cce9042e · 2018-04-04T15:32:02.000-04:00
These tests are using their name as ID, but subtests add a forward slash
connected to the parent test, and slash (/) is an invalid character for
container IDs.

Signed-off-by: Phil Estes &lt;estesp@linux.vnet.ibm.com&gt;
diff --git a/container_linux_test.go b/container_linux_test.go
@@ -493,7 +493,7 @@ func testContainerUser(t *testing.T, userstr, expectedOutput string) {
 	var (
 		image       Image
 		ctx, cancel = testContext()
-		id          = t.Name()
+		id          = strings.Replace(t.Name(), "/", "_", -1)
 	)
 	defer cancel()
 

Original file line number	Diff line number	Diff line change
`@@ -493,7 +493,7 @@ func testContainerUser(t *testing.T, userstr, expectedOutput string) {`
`493`	`493`	`var (`
`494`	`494`	`image Image`
`495`	`495`	`ctx, cancel = testContext()`
`496`		`- id = t.Name()`
	`496`	`+ id = strings.Replace(t.Name(), "/", "_", -1)`
`497`	`497`	`)`
`498`	`498`	`defer cancel()`
`499`	`499`