sanity: avoid nil access for connection

pohly · pohly · commit 4d03b5ec7720 · 2019-03-20T20:03:58.000+01:00
The assumption that sc.conn always gets set turned out to be
false. The reason probably was running with such a high number of
Ginkgo nodes that some nodes never had to execute any test.
diff --git a/pkg/sanity/sanity.go b/pkg/sanity/sanity.go
@@ -139,7 +139,9 @@ func Test(t *testing.T, reqConfig *Config) {
 		specReporters = append(specReporters, junitReporter)
 	}
 	RunSpecsWithDefaultAndCustomReporters(t, "CSI Driver Test Suite", specReporters)
-	sc.Conn.Close()
+	if sc.Conn != nil {
+		sc.Conn.Close()
+	}
 }
 
 func GinkgoTest(reqConfig *Config) {

Original file line number	Diff line number	Diff line change
`@@ -139,7 +139,9 @@ func Test(t testing.T, reqConfig Config) {`
`139`	`139`	`specReporters = append(specReporters, junitReporter)`
`140`	`140`	`}`
`141`	`141`	`RunSpecsWithDefaultAndCustomReporters(t, "CSI Driver Test Suite", specReporters)`
`142`		`- sc.Conn.Close()`
	`142`	`+ if sc.Conn != nil {`
	`143`	`+ sc.Conn.Close()`
	`144`	`+ }`
`143`	`145`	`}`
`144`	`146`
`145`	`147`	`func GinkgoTest(reqConfig *Config) {`