test: add unit test for func storage/volume/VolumePath

Signed-off-by: btlcmr0702 <[email protected]>

storage/volume/core.go

@@ -385,7 +385,6 @@ func (c *Core) DetachVolume(id types.VolumeID, extra map[string]string) (*types.
 	if err != nil {
 		return nil, err
 	}
-
 	ctx := driver.Contexts()
 
 	// merge extra to volume spec extra.

storage/volume/core_test.go

@@ -176,17 +176,16 @@ func TestListVolumes(t *testing.T) {
 }
 
 func TestListVolumeName(t *testing.T) {
-	// TODO
-	driverName := "fake_driver4"
+	driverName := "my_fake"
 	dir, err := ioutil.TempDir("", "TestGetVolume")
 	if err != nil {
 		t.Fatal(err)
 	}
 	defer os.RemoveAll(dir)
 
-	core, err := createVolumeCore(dir)
-	if err != nil {
-		t.Fatal(err)
+	core, err_cv := createVolumeCore(dir)
+	if err_cv != nil {
+		t.Fatal("create volume core error")
 	}
 
 	driver.Register(driver.NewFakeDriver(driverName))
@@ -199,12 +198,16 @@ func TestListVolumeName(t *testing.T) {
 		volid := types.VolumeID{Name: volName, Driver: driverName}
 		v, err := core.CreateVolume(volid)
 		if err != nil {
-			t.Fatalf("create volume error: %v", err)
+			t.Fatalf("create volume fail: %v", err)
 		}
 		volmap[volName] = v
 	}
 
-	volarray, err := core.ListVolumes(nil)
+	volarray, err_lv := core.ListVolumes(nil)
+	if err_lv != nil {
+		t.Fatalf("list volumes fail")
+	}
+
 	for k := 0; k < len(volarray); k++ {
 		vol := volarray[k]
 		_, found := volmap[vol.Name]
@@ -214,15 +217,15 @@ func TestListVolumeName(t *testing.T) {
 	}
 	//add unit test for listVolumeName
 	var volNames []string
-        volNames, err = core.ListVolumeName(nil)
-	if err != nil{
-		t.Fatalf("list volume name function failed!")
+	volNames, err = core.ListVolumeName(nil)
+	if err != nil {
+		t.Fatalf("list volume name function fail!")
 	}
-	for j:= 0; j < len(volNames); j++ {
+	for j := 0; j < len(volNames); j++ {
 		_, found := volmap[volNames[j]]
 		if found == false {
 			t.Fatalf("list volumes name %s not found", volNames[j])
-		}	
+		}
 	}
 
 }