Skip to content

Commit

Permalink
Merge pull request #58 from dell/ut-patch-1
Browse files Browse the repository at this point in the history
Update mockhelper.go
  • Loading branch information
adarsh-dell authored Sep 19, 2024
2 parents c9c4a0b + 6aef873 commit da605b2
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
3 changes: 1 addition & 2 deletions internal/mockhelper/mockhelper.go
Original file line number Diff line number Diff line change
Expand Up @@ -195,8 +195,7 @@ func (mh *MockHelper) FilePathGlobOK(m *wrp.MockLimitedFilepath) *gomock.Call {

// OSExecCommandContextCall mocks implementation of CommandContext method from LimitedOSExec interface
func (mh *MockHelper) OSExecCommandContextCall(m *wrp.MockLimitedOSExec) *gomock.Call {
return m.EXPECT().CommandContext(gomock.Any(),
mh.OSEXECCommandContextName, mh.OSEXECCommandContextArgs)
return m.EXPECT().CommandContext(gomock.Any(), gomock.Any(), gomock.Any())
}

// OSExecCommandContextOK mocks returning success from OSExecCommandContextCall
Expand Down
2 changes: 2 additions & 0 deletions pkg/powerpath/powerpath_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -203,6 +203,8 @@ busy: False
stateSetter: func(fields ppFields) {
_, cmdMock := mocks.OSExecCommandContextOK(fields.osexec)
mocks.OSExecCmdOK(cmdMock)
_, cmdMock2 := mocks.OSExecCommandContextOK(fields.osexec)

Check failure on line 206 in pkg/powerpath/powerpath_test.go

View workflow job for this annotation

GitHub Actions / golangci-lint

File is not `gofumpt`-ed (gofumpt)
mocks.OSExecCmdOK(cmdMock2)
},
args: defaultArgs,
want: true,
Expand Down

0 comments on commit da605b2

Please sign in to comment.