Merge pull request #51388 from alrs/fix-scaleio-swallowed-error

Automatic merge from submit-queue (batch tested with PRs 51174, 51363, 51087, 51382, 51388) Fix swallowed error in scaleio package tests **What this PR does / why we need it**: Fixes a dropped error in the tests of the scaleio package. **Release note**: ```release-note NONE ```

Merge pull request #51388 from alrs/fix-scaleio-swallowed-error
84d9778f · Kubernetes Submit Queue · GitHub · 4b713551 · 7ee91d6d · 84d9778f
Commit 84d9778f authored Aug 26, 2017 by Kubernetes Submit Queue Committed by GitHub Aug 26, 2017
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 1 deletion

sio_util_test.go pkg/volume/scaleio/sio_util_test.go +4 -1

No files found.
--- a/pkg/volume/scaleio/sio_util_test.go
+++ b/pkg/volume/scaleio/sio_util_test.go
@@ -212,10 +212,13 @@ func TestUtilLoadConfig(t *testing.T) {
 	configFile := path.Join(tmpDir, sioConfigFileName)

 	if err := saveConfig(configFile, config); err != nil {
-		t.Fatal("failed while saving data", err)
+		t.Fatalf("failed to save configFile %s error:%v", configFile, err)
 	}

 	dataRcvd, err := loadConfig(configFile)
+	if err != nil {
+		t.Fatalf("failed to load configFile %s error:%v", configFile, err)
+	}
 	if dataRcvd[confKey.gateway] != config[confKey.gateway] ||
 		dataRcvd[confKey.system] != config[confKey.system] {
 		t.Fatal("loaded config data not matching saved config data")