Skip to content

Commit

Permalink
test: fix mock
Browse files Browse the repository at this point in the history
  • Loading branch information
raffis committed Aug 3, 2023
1 parent 6b10d29 commit 9adabdb
Showing 1 changed file with 2 additions and 12 deletions.
14 changes: 2 additions & 12 deletions tests/vcmock/vcmock.go
Original file line number Diff line number Diff line change
Expand Up @@ -109,18 +109,8 @@ func (vc *VersionController) IsPullRequestApprovedByMe(ctx context.Context, pull
return false, nil
}

// ApprovePullRequest approves a pull request
func (vc *VersionController) ApprovePullRequest(ctx context.Context, pullReq scm.PullRequest, comment string) error {
return nil
}

// RejectPullRequest requests changes
func (vc *VersionController) RejectPullRequest(ctx context.Context, pullReq scm.PullRequest, comment string) error {
return nil
}

// CommentPullRequest leaves a comment
func (vc *VersionController) CommentPullRequest(ctx context.Context, pullReq scm.PullRequest, comment string) error {
// ReviewPullRequest reviews a pull request
func (vc *VersionController) ReviewPullRequest(ctx context.Context, pullReq scm.PullRequest, action scm.Review, comment string) error {

Check failure on line 113 in tests/vcmock/vcmock.go

View workflow job for this annotation

GitHub Actions / golangci-lint

[golangci] reported by reviewdog 🐶 unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive) Raw Output: tests/vcmock/vcmock.go:113:48: unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive) func (vc *VersionController) ReviewPullRequest(ctx context.Context, pullReq scm.PullRequest, action scm.Review, comment string) error { ^
return nil
}

Expand Down

0 comments on commit 9adabdb

Please sign in to comment.