Code review changes

gaurav1086 · gaurav1086 · commit c41932e9195f · 2020-04-20T23:05:05.000-04:00
diff --git a/pkg/volume/awsebs/aws_ebs_block_test.go b/pkg/volume/awsebs/aws_ebs_block_test.go
@@ -68,7 +68,7 @@ func TestGetVolumeSpecFromGlobalMapPath(t *testing.T) {
 	// Good Path
 	spec, err := plug.(*awsElasticBlockStorePlugin).getVolumeSpecFromGlobalMapPath("myVolume", expectedGlobalPath)
 	if spec == nil || err != nil {
-		t.Fatalf("Failed to get spec from GlobalMapPath: %v", err)
+		t.Errorf("Invalid volumeMode from GlobalMapPath spec: %v - %v", &specMode, block)
 	}
 	if spec.PersistentVolume.Name != "myVolume" {
 		t.Errorf("Invalid PV name from GlobalMapPath spec: %s", spec.PersistentVolume.Name)

Original file line number	Diff line number	Diff line change
`@@ -68,7 +68,7 @@ func TestGetVolumeSpecFromGlobalMapPath(t *testing.T) {`
`68`	`68`	`// Good Path`
`69`	`69`	`spec, err := plug.(*awsElasticBlockStorePlugin).getVolumeSpecFromGlobalMapPath("myVolume", expectedGlobalPath)`
`70`	`70`	`if spec == nil \|\| err != nil {`
`71`		`- t.Fatalf("Failed to get spec from GlobalMapPath: %v", err)`
	`71`	`+ t.Errorf("Invalid volumeMode from GlobalMapPath spec: %v - %v", &specMode, block)`
`72`	`72`	`}`
`73`	`73`	`if spec.PersistentVolume.Name != "myVolume" {`
`74`	`74`	`t.Errorf("Invalid PV name from GlobalMapPath spec: %s", spec.PersistentVolume.Name)`