lvm: fix last failing test

resource/lvm/lv/lv.go

@@ -41,6 +41,7 @@ func (r *ResourceLV) Check(resource.Renderer) (resource.TaskStatus, error) {
 		r.needCreate = !ok
 	} else {
 		status.Output = append(status.Output, fmt.Sprintf("group %s not exist, assume that it will be created"))
+		r.needCreate = true
 	}
 
 	status.DevicePath = fmt.Sprintf("/dev/mapper/%s-%s", r.group, r.name)

resource/lvm/lv/lv_test.go

@@ -16,10 +16,10 @@ func TestCreateLogicalVolume(t *testing.T) {
 	volname := "data" // Match with existing name in TESTDATA_VGS, so fool engine to find proper paths, etc
 	// after creation
 	lvm, me := testhelpers.MakeLvmWithMockExec()
+	me.LvsFirstCall = true
 	me.On("Read", "pvs", mock.Anything).Return(testdata.TESTDATA_PVS, nil)
 	me.On("Read", "vgs", mock.Anything).Return(testdata.TESTDATA_VGS, nil)
-	me.On("Read", "lvs", mock.Anything).Return("", nil).Once()
-	me.On("Read", "lvs", mock.Anything).Return(testdata.TESTDATA_VGS, nil)
+	me.On("Read", "lvs", mock.Anything).Return(testdata.TESTDATA_LVS, nil)
 	me.On("Run", "lvcreate", []string{"-n", volname, "-L", "100G", "vg0"}).Return(nil)
 	me.On("Exists", "/dev/mapper/vg0-data").Return(true, nil)
 

resource/lvm/testhelpers/mock.go

@@ -29,7 +29,7 @@ func (e *MockExecutor) RunExitCode(prog string, args []string) (int, error) {
 
 func (me *MockExecutor) Read(prog string, args []string) (string, error) {
 	if me.LvsFirstCall {
-		//        me.LvsFirstCall = false
+		me.LvsFirstCall = false
 		return "", nil
 	}
 	c := me.Called(prog, args)
@@ -57,5 +57,5 @@ func (me *MockExecutor) MkdirAll(path string, perm os.FileMode) error {
 
 func (me *MockExecutor) Exists(path string) (bool, error) {
 	c := me.Called(path)
-	return c.Bool(0), c.Error(0)
+	return c.Bool(0), c.Error(1)
 }