mirror of
https://codeberg.org/forgejo/forgejo.git
synced 2024-11-11 04:11:18 +01:00
test: add more workflow event coverage
This commit is contained in:
parent
8322882265
commit
0813126058
1 changed files with 98 additions and 81 deletions
|
@ -16,58 +16,67 @@ import (
|
||||||
|
|
||||||
func TestDetectMatched(t *testing.T) {
|
func TestDetectMatched(t *testing.T) {
|
||||||
testCases := []struct {
|
testCases := []struct {
|
||||||
desc string
|
desc string
|
||||||
commit *git.Commit
|
commit *git.Commit
|
||||||
triggedEvent webhook_module.HookEventType
|
triggeredEvent webhook_module.HookEventType
|
||||||
payload api.Payloader
|
payload api.Payloader
|
||||||
yamlOn string
|
yamlOn string
|
||||||
expected bool
|
expected bool
|
||||||
}{
|
}{
|
||||||
{
|
{
|
||||||
desc: "HookEventCreate(create) matches GithubEventCreate(create)",
|
desc: "HookEventCreate(create) matches GithubEventCreate(create)",
|
||||||
triggedEvent: webhook_module.HookEventCreate,
|
triggeredEvent: webhook_module.HookEventCreate,
|
||||||
payload: nil,
|
payload: nil,
|
||||||
yamlOn: "on: create",
|
yamlOn: "on: create",
|
||||||
expected: true,
|
expected: true,
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
desc: "HookEventIssues(issues) `opened` action matches GithubEventIssues(issues)",
|
desc: "HookEventIssues(issues) `opened` action matches GithubEventIssues(issues)",
|
||||||
triggedEvent: webhook_module.HookEventIssues,
|
triggeredEvent: webhook_module.HookEventIssues,
|
||||||
payload: &api.IssuePayload{Action: api.HookIssueOpened},
|
payload: &api.IssuePayload{Action: api.HookIssueOpened},
|
||||||
yamlOn: "on: issues",
|
yamlOn: "on: issues",
|
||||||
expected: true,
|
expected: true,
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
desc: "HookEventIssues(issues) `milestoned` action matches GithubEventIssues(issues)",
|
desc: "HookEventIssueComment(issue_comment) `created` action matches GithubEventIssueComment(issue_comment)",
|
||||||
triggedEvent: webhook_module.HookEventIssues,
|
triggeredEvent: webhook_module.HookEventIssueComment,
|
||||||
payload: &api.IssuePayload{Action: api.HookIssueMilestoned},
|
payload: &api.IssueCommentPayload{Action: api.HookIssueCommentCreated},
|
||||||
yamlOn: "on: issues",
|
yamlOn: "on:\n issue_comment:\n types: [created]",
|
||||||
expected: true,
|
expected: true,
|
||||||
|
},
|
||||||
|
|
||||||
|
{
|
||||||
|
desc: "HookEventIssues(issues) `milestoned` action matches GithubEventIssues(issues)",
|
||||||
|
triggeredEvent: webhook_module.HookEventIssues,
|
||||||
|
payload: &api.IssuePayload{Action: api.HookIssueMilestoned},
|
||||||
|
yamlOn: "on: issues",
|
||||||
|
expected: true,
|
||||||
|
},
|
||||||
|
|
||||||
|
{
|
||||||
|
desc: "HookEventPullRequestSync(pull_request_sync) matches GithubEventPullRequest(pull_request)",
|
||||||
|
triggeredEvent: webhook_module.HookEventPullRequestSync,
|
||||||
|
payload: &api.PullRequestPayload{Action: api.HookIssueSynchronized},
|
||||||
|
yamlOn: "on: pull_request",
|
||||||
|
expected: true,
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
desc: "HookEventPullRequestSync(pull_request_sync) matches GithubEventPullRequest(pull_request)",
|
desc: "HookEventPullRequest(pull_request) `label_updated` action doesn't match GithubEventPullRequest(pull_request) with no activity type",
|
||||||
triggedEvent: webhook_module.HookEventPullRequestSync,
|
triggeredEvent: webhook_module.HookEventPullRequest,
|
||||||
payload: &api.PullRequestPayload{Action: api.HookIssueSynchronized},
|
payload: &api.PullRequestPayload{Action: api.HookIssueLabelUpdated},
|
||||||
yamlOn: "on: pull_request",
|
yamlOn: "on: pull_request",
|
||||||
expected: true,
|
expected: false,
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
desc: "HookEventPullRequest(pull_request) `label_updated` action doesn't match GithubEventPullRequest(pull_request) with no activity type",
|
desc: "HookEventPullRequest(pull_request) `closed` action doesn't match GithubEventPullRequest(pull_request) with no activity type",
|
||||||
triggedEvent: webhook_module.HookEventPullRequest,
|
triggeredEvent: webhook_module.HookEventPullRequest,
|
||||||
payload: &api.PullRequestPayload{Action: api.HookIssueLabelUpdated},
|
payload: &api.PullRequestPayload{Action: api.HookIssueClosed},
|
||||||
yamlOn: "on: pull_request",
|
yamlOn: "on: pull_request",
|
||||||
expected: false,
|
expected: false,
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
desc: "HookEventPullRequest(pull_request) `closed` action doesn't match GithubEventPullRequest(pull_request) with no activity type",
|
desc: "HookEventPullRequest(pull_request) `closed` action doesn't match GithubEventPullRequest(pull_request) with branches",
|
||||||
triggedEvent: webhook_module.HookEventPullRequest,
|
triggeredEvent: webhook_module.HookEventPullRequest,
|
||||||
payload: &api.PullRequestPayload{Action: api.HookIssueClosed},
|
|
||||||
yamlOn: "on: pull_request",
|
|
||||||
expected: false,
|
|
||||||
},
|
|
||||||
{
|
|
||||||
desc: "HookEventPullRequest(pull_request) `closed` action doesn't match GithubEventPullRequest(pull_request) with branches",
|
|
||||||
triggedEvent: webhook_module.HookEventPullRequest,
|
|
||||||
payload: &api.PullRequestPayload{
|
payload: &api.PullRequestPayload{
|
||||||
Action: api.HookIssueClosed,
|
Action: api.HookIssueClosed,
|
||||||
PullRequest: &api.PullRequest{
|
PullRequest: &api.PullRequest{
|
||||||
|
@ -78,60 +87,68 @@ func TestDetectMatched(t *testing.T) {
|
||||||
expected: false,
|
expected: false,
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
desc: "HookEventPullRequest(pull_request) `label_updated` action matches GithubEventPullRequest(pull_request) with `label` activity type",
|
desc: "HookEventPullRequest(pull_request) `label_updated` action matches GithubEventPullRequest(pull_request) with `label` activity type",
|
||||||
triggedEvent: webhook_module.HookEventPullRequest,
|
triggeredEvent: webhook_module.HookEventPullRequest,
|
||||||
payload: &api.PullRequestPayload{Action: api.HookIssueLabelUpdated},
|
payload: &api.PullRequestPayload{Action: api.HookIssueLabelUpdated},
|
||||||
yamlOn: "on:\n pull_request:\n types: [labeled]",
|
yamlOn: "on:\n pull_request:\n types: [labeled]",
|
||||||
expected: true,
|
expected: true,
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
desc: "HookEventPullRequestReviewComment(pull_request_review_comment) matches GithubEventPullRequestReviewComment(pull_request_review_comment)",
|
desc: "HookEventPullRequestReviewComment(pull_request_review_comment) matches GithubEventPullRequestReviewComment(pull_request_review_comment)",
|
||||||
triggedEvent: webhook_module.HookEventPullRequestReviewComment,
|
triggeredEvent: webhook_module.HookEventPullRequestReviewComment,
|
||||||
payload: &api.PullRequestPayload{Action: api.HookIssueReviewed},
|
payload: &api.PullRequestPayload{Action: api.HookIssueReviewed},
|
||||||
yamlOn: "on:\n pull_request_review_comment:\n types: [created]",
|
yamlOn: "on:\n pull_request_review_comment:\n types: [created]",
|
||||||
expected: true,
|
expected: true,
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
desc: "HookEventPullRequestReviewRejected(pull_request_review_rejected) doesn't match GithubEventPullRequestReview(pull_request_review) with `dismissed` activity type (we don't support `dismissed` at present)",
|
desc: "HookEventPullRequestReviewRejected(pull_request_review_rejected) doesn't match GithubEventPullRequestReview(pull_request_review) with `dismissed` activity type (we don't support `dismissed` at present)",
|
||||||
triggedEvent: webhook_module.HookEventPullRequestReviewRejected,
|
triggeredEvent: webhook_module.HookEventPullRequestReviewRejected,
|
||||||
payload: &api.PullRequestPayload{Action: api.HookIssueReviewed},
|
payload: &api.PullRequestPayload{Action: api.HookIssueReviewed},
|
||||||
yamlOn: "on:\n pull_request_review:\n types: [dismissed]",
|
yamlOn: "on:\n pull_request_review:\n types: [dismissed]",
|
||||||
expected: false,
|
expected: false,
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
desc: "HookEventRelease(release) `published` action matches GithubEventRelease(release) with `published` activity type",
|
desc: "HookEventRelease(release) `published` action matches GithubEventRelease(release) with `published` activity type",
|
||||||
triggedEvent: webhook_module.HookEventRelease,
|
triggeredEvent: webhook_module.HookEventRelease,
|
||||||
payload: &api.ReleasePayload{Action: api.HookReleasePublished},
|
payload: &api.ReleasePayload{Action: api.HookReleasePublished},
|
||||||
yamlOn: "on:\n release:\n types: [published]",
|
yamlOn: "on:\n release:\n types: [published]",
|
||||||
expected: true,
|
expected: true,
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
desc: "HookEventPackage(package) `created` action doesn't match GithubEventRegistryPackage(registry_package) with `updated` activity type",
|
desc: "HookEventRelease(updated) `updated` action matches GithubEventRelease(edited) with `edited` activity type",
|
||||||
triggedEvent: webhook_module.HookEventPackage,
|
triggeredEvent: webhook_module.HookEventRelease,
|
||||||
payload: &api.PackagePayload{Action: api.HookPackageCreated},
|
payload: &api.ReleasePayload{Action: api.HookReleaseUpdated},
|
||||||
yamlOn: "on:\n registry_package:\n types: [updated]",
|
yamlOn: "on:\n release:\n types: [edited]",
|
||||||
expected: false,
|
expected: true,
|
||||||
|
},
|
||||||
|
|
||||||
|
{
|
||||||
|
desc: "HookEventPackage(package) `created` action doesn't match GithubEventRegistryPackage(registry_package) with `updated` activity type",
|
||||||
|
triggeredEvent: webhook_module.HookEventPackage,
|
||||||
|
payload: &api.PackagePayload{Action: api.HookPackageCreated},
|
||||||
|
yamlOn: "on:\n registry_package:\n types: [updated]",
|
||||||
|
expected: false,
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
desc: "HookEventWiki(wiki) matches GithubEventGollum(gollum)",
|
desc: "HookEventWiki(wiki) matches GithubEventGollum(gollum)",
|
||||||
triggedEvent: webhook_module.HookEventWiki,
|
triggeredEvent: webhook_module.HookEventWiki,
|
||||||
payload: nil,
|
payload: nil,
|
||||||
yamlOn: "on: gollum",
|
yamlOn: "on: gollum",
|
||||||
expected: true,
|
expected: true,
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
desc: "HookEventSchedue(schedule) matches GithubEventSchedule(schedule)",
|
desc: "HookEventSchedule(schedule) matches GithubEventSchedule(schedule)",
|
||||||
triggedEvent: webhook_module.HookEventSchedule,
|
triggeredEvent: webhook_module.HookEventSchedule,
|
||||||
payload: nil,
|
payload: nil,
|
||||||
yamlOn: "on: schedule",
|
yamlOn: "on: schedule",
|
||||||
expected: true,
|
expected: true,
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
desc: "HookEventWorkflowDispatch(workflow_dispatch) matches GithubEventWorkflowDispatch(workflow_dispatch)",
|
desc: "HookEventWorkflowDispatch(workflow_dispatch) matches GithubEventWorkflowDispatch(workflow_dispatch)",
|
||||||
triggedEvent: webhook_module.HookEventWorkflowDispatch,
|
triggeredEvent: webhook_module.HookEventWorkflowDispatch,
|
||||||
payload: nil,
|
payload: nil,
|
||||||
yamlOn: "on: workflow_dispatch",
|
yamlOn: "on: workflow_dispatch",
|
||||||
expected: true,
|
expected: true,
|
||||||
},
|
},
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -140,7 +157,7 @@ func TestDetectMatched(t *testing.T) {
|
||||||
evts, err := GetEventsFromContent([]byte(tc.yamlOn))
|
evts, err := GetEventsFromContent([]byte(tc.yamlOn))
|
||||||
require.NoError(t, err)
|
require.NoError(t, err)
|
||||||
assert.Len(t, evts, 1)
|
assert.Len(t, evts, 1)
|
||||||
assert.Equal(t, tc.expected, detectMatched(nil, tc.commit, tc.triggedEvent, tc.payload, evts[0]))
|
assert.Equal(t, tc.expected, detectMatched(nil, tc.commit, tc.triggeredEvent, tc.payload, evts[0]))
|
||||||
})
|
})
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in a new issue