mirror of
https://github.com/thegeeklab/git-sv.git
synced 2024-11-23 14:50:39 +00:00
2d2de1a5e0
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Robert Kaussow <mail@thegeeklab.de>
964 lines
23 KiB
Go
964 lines
23 KiB
Go
package sv
|
|
|
|
import (
|
|
"reflect"
|
|
"testing"
|
|
)
|
|
|
|
var ccfg = CommitMessageConfig{
|
|
Types: []string{"feat", "fix"},
|
|
Scope: CommitMessageScopeConfig{},
|
|
Footer: map[string]CommitMessageFooterConfig{
|
|
"issue": {Key: "jira", KeySynonyms: []string{"Jira"}},
|
|
"refs": {Key: "Refs", UseHash: true},
|
|
},
|
|
Issue: CommitMessageIssueConfig{Regex: "[A-Z]+-[0-9]+"},
|
|
}
|
|
|
|
var ccfgHash = CommitMessageConfig{
|
|
Types: []string{"feat", "fix"},
|
|
Scope: CommitMessageScopeConfig{},
|
|
Footer: map[string]CommitMessageFooterConfig{
|
|
"issue": {Key: "jira", KeySynonyms: []string{"Jira"}, UseHash: true},
|
|
"refs": {Key: "Refs", UseHash: true},
|
|
},
|
|
Issue: CommitMessageIssueConfig{Regex: "[A-Z]+-[0-9]+"},
|
|
}
|
|
|
|
var ccfgGitIssue = CommitMessageConfig{
|
|
Types: []string{"feat", "fix"},
|
|
Scope: CommitMessageScopeConfig{},
|
|
Footer: map[string]CommitMessageFooterConfig{
|
|
"issue": {Key: "issue", KeySynonyms: []string{"Issue"}, UseHash: false, AddValuePrefix: "#"},
|
|
},
|
|
Issue: CommitMessageIssueConfig{Regex: "#?[0-9]+"},
|
|
}
|
|
|
|
var ccfgEmptyIssue = CommitMessageConfig{
|
|
Types: []string{"feat", "fix"},
|
|
Scope: CommitMessageScopeConfig{},
|
|
Footer: map[string]CommitMessageFooterConfig{
|
|
"issue": {},
|
|
},
|
|
Issue: CommitMessageIssueConfig{Regex: "[A-Z]+-[0-9]+"},
|
|
}
|
|
|
|
var ccfgWithScope = CommitMessageConfig{
|
|
Types: []string{"feat", "fix"},
|
|
Scope: CommitMessageScopeConfig{Values: []string{"", "scope"}},
|
|
Footer: map[string]CommitMessageFooterConfig{
|
|
"issue": {Key: "jira", KeySynonyms: []string{"Jira"}},
|
|
"refs": {Key: "Refs", UseHash: true},
|
|
},
|
|
Issue: CommitMessageIssueConfig{Regex: "[A-Z]+-[0-9]+"},
|
|
}
|
|
|
|
func newBranchCfg(skipDetached bool) BranchesConfig {
|
|
return BranchesConfig{
|
|
Prefix: "([a-z]+\\/)?",
|
|
Suffix: "(-.*)?",
|
|
Skip: []string{"develop", "master"},
|
|
SkipDetached: &skipDetached,
|
|
}
|
|
}
|
|
|
|
func newCommitMessageCfg(headerSelector string) CommitMessageConfig {
|
|
return CommitMessageConfig{
|
|
Types: []string{"feat", "fix"},
|
|
Scope: CommitMessageScopeConfig{Values: []string{"", "scope"}},
|
|
Footer: map[string]CommitMessageFooterConfig{
|
|
"issue": {Key: "jira", KeySynonyms: []string{"Jira"}},
|
|
"refs": {Key: "Refs", UseHash: true},
|
|
},
|
|
Issue: CommitMessageIssueConfig{Regex: "[A-Z]+-[0-9]+"},
|
|
HeaderSelector: headerSelector,
|
|
}
|
|
}
|
|
|
|
// messages samples start.
|
|
var fullMessage = `fix: correct minor typos in code
|
|
|
|
see the issue for details
|
|
|
|
on typos fixed.
|
|
|
|
Reviewed-by: Z
|
|
Refs #133`
|
|
|
|
var fullMessageWithJira = `fix: correct minor typos in code
|
|
|
|
see the issue for details
|
|
|
|
on typos fixed.
|
|
|
|
Reviewed-by: Z
|
|
Refs #133
|
|
jira: JIRA-456`
|
|
|
|
var fullMessageRefs = `fix: correct minor typos in code
|
|
|
|
see the issue for details
|
|
|
|
on typos fixed.
|
|
|
|
Refs #133`
|
|
|
|
var subjectAndBodyMessage = `fix: correct minor typos in code
|
|
|
|
see the issue for details
|
|
|
|
on typos fixed.`
|
|
|
|
var subjectAndFooterMessage = `refactor!: drop support for Node 6
|
|
|
|
BREAKING CHANGE: refactor to use JavaScript features not available in Node 6.`
|
|
|
|
// multiline samples end
|
|
|
|
func TestBaseMessageProcessor_SkipBranch(t *testing.T) {
|
|
tests := []struct {
|
|
name string
|
|
bcfg BranchesConfig
|
|
branch string
|
|
detached bool
|
|
want bool
|
|
}{
|
|
{"normal branch", newBranchCfg(false), "JIRA-123", false, false},
|
|
{"dont ignore detached branch", newBranchCfg(false), "JIRA-123", true, false},
|
|
{"ignore branch on skip list", newBranchCfg(false), "master", false, true},
|
|
{"ignore detached branch", newBranchCfg(true), "JIRA-123", true, true},
|
|
{"null skip detached", BranchesConfig{Skip: []string{}}, "JIRA-123", true, false},
|
|
}
|
|
for _, tt := range tests {
|
|
t.Run(tt.name, func(t *testing.T) {
|
|
p := NewMessageProcessor(ccfg, tt.bcfg)
|
|
if got := p.SkipBranch(tt.branch, tt.detached); got != tt.want {
|
|
t.Errorf("BaseMessageProcessor.SkipBranch() = %v, want %v", got, tt.want)
|
|
}
|
|
})
|
|
}
|
|
}
|
|
|
|
func TestBaseMessageProcessor_Validate(t *testing.T) {
|
|
tests := []struct {
|
|
name string
|
|
cfg CommitMessageConfig
|
|
message string
|
|
wantErr bool
|
|
}{
|
|
{
|
|
"single line valid message",
|
|
ccfg,
|
|
"feat: add something", false,
|
|
},
|
|
{
|
|
"single line valid message with scope",
|
|
ccfg,
|
|
"feat(scope): add something", false,
|
|
},
|
|
{"single line valid scope from list", ccfgWithScope, "feat(scope): add something", false},
|
|
{"single line invalid scope from list", ccfgWithScope, "feat(invalid): add something", true},
|
|
{
|
|
"single line invalid type message",
|
|
ccfg,
|
|
"something: add something", true,
|
|
},
|
|
{
|
|
"single line invalid type message",
|
|
ccfg,
|
|
"feat?: add something", true,
|
|
},
|
|
|
|
{
|
|
"multi line valid message",
|
|
ccfg,
|
|
`feat: add something
|
|
|
|
team: x`, false,
|
|
},
|
|
|
|
{
|
|
"multi line invalid message",
|
|
ccfg,
|
|
`feat add something
|
|
|
|
team: x`, true,
|
|
},
|
|
|
|
{
|
|
"support ! for breaking change",
|
|
ccfg,
|
|
"feat!: add something", false,
|
|
},
|
|
{
|
|
"support ! with scope for breaking change",
|
|
ccfg,
|
|
"feat(scope)!: add something", false,
|
|
},
|
|
}
|
|
for _, tt := range tests {
|
|
t.Run(tt.name, func(t *testing.T) {
|
|
p := NewMessageProcessor(tt.cfg, newBranchCfg(false))
|
|
if err := p.Validate(tt.message); (err != nil) != tt.wantErr {
|
|
t.Errorf("BaseMessageProcessor.Validate() error = %v, wantErr %v", err, tt.wantErr)
|
|
}
|
|
})
|
|
}
|
|
}
|
|
|
|
func TestBaseMessageProcessor_ValidateType(t *testing.T) {
|
|
tests := []struct {
|
|
name string
|
|
cfg CommitMessageConfig
|
|
ctype string
|
|
wantErr bool
|
|
}{
|
|
{
|
|
"valid type",
|
|
ccfg,
|
|
"feat", false,
|
|
},
|
|
{
|
|
"invalid type",
|
|
ccfg,
|
|
"aaa", true,
|
|
},
|
|
{
|
|
"empty type",
|
|
ccfg,
|
|
"", true,
|
|
},
|
|
}
|
|
for _, tt := range tests {
|
|
t.Run(tt.name, func(t *testing.T) {
|
|
p := NewMessageProcessor(tt.cfg, newBranchCfg(false))
|
|
if err := p.ValidateType(tt.ctype); (err != nil) != tt.wantErr {
|
|
t.Errorf("BaseMessageProcessor.ValidateType() error = %v, wantErr %v", err, tt.wantErr)
|
|
}
|
|
})
|
|
}
|
|
}
|
|
|
|
func TestBaseMessageProcessor_ValidateScope(t *testing.T) {
|
|
tests := []struct {
|
|
name string
|
|
cfg CommitMessageConfig
|
|
scope string
|
|
wantErr bool
|
|
}{
|
|
{
|
|
"any scope",
|
|
ccfg,
|
|
"aaa", false,
|
|
},
|
|
{"valid scope with scope list", ccfgWithScope, "scope", false},
|
|
{"invalid scope with scope list", ccfgWithScope, "aaa", true},
|
|
}
|
|
for _, tt := range tests {
|
|
t.Run(tt.name, func(t *testing.T) {
|
|
p := NewMessageProcessor(tt.cfg, newBranchCfg(false))
|
|
if err := p.ValidateScope(tt.scope); (err != nil) != tt.wantErr {
|
|
t.Errorf("BaseMessageProcessor.ValidateScope() error = %v, wantErr %v", err, tt.wantErr)
|
|
}
|
|
})
|
|
}
|
|
}
|
|
|
|
func TestBaseMessageProcessor_ValidateDescription(t *testing.T) {
|
|
tests := []struct {
|
|
name string
|
|
cfg CommitMessageConfig
|
|
description string
|
|
wantErr bool
|
|
}{
|
|
{
|
|
"empty description",
|
|
ccfg,
|
|
"", true,
|
|
},
|
|
{
|
|
"sigle letter description",
|
|
ccfg,
|
|
"a", false,
|
|
},
|
|
{
|
|
"number description",
|
|
ccfg,
|
|
"1", true,
|
|
},
|
|
{
|
|
"valid description",
|
|
ccfg,
|
|
"add some feature", false,
|
|
},
|
|
{
|
|
"invalid capital letter description",
|
|
ccfg,
|
|
"Add some feature", true,
|
|
},
|
|
}
|
|
for _, tt := range tests {
|
|
t.Run(tt.name, func(t *testing.T) {
|
|
p := NewMessageProcessor(tt.cfg, newBranchCfg(false))
|
|
if err := p.ValidateDescription(tt.description); (err != nil) != tt.wantErr {
|
|
t.Errorf("BaseMessageProcessor.ValidateDescription() error = %v, wantErr %v", err, tt.wantErr)
|
|
}
|
|
})
|
|
}
|
|
}
|
|
|
|
func TestBaseMessageProcessor_Enhance(t *testing.T) {
|
|
tests := []struct {
|
|
name string
|
|
cfg CommitMessageConfig
|
|
branch string
|
|
message string
|
|
want string
|
|
wantErr bool
|
|
}{
|
|
{
|
|
"issue on branch name",
|
|
ccfg,
|
|
"JIRA-123", "fix: fix something", "\njira: JIRA-123", false,
|
|
},
|
|
{
|
|
"issue on branch name with description",
|
|
ccfg,
|
|
"JIRA-123-some-description", "fix: fix something", "\njira: JIRA-123", false,
|
|
},
|
|
{
|
|
"issue on branch name with prefix",
|
|
ccfg,
|
|
"feature/JIRA-123", "fix: fix something", "\njira: JIRA-123", false,
|
|
},
|
|
{
|
|
"with footer",
|
|
ccfg,
|
|
"JIRA-123", fullMessage, "jira: JIRA-123", false,
|
|
},
|
|
{
|
|
"with issue on footer",
|
|
ccfg,
|
|
"JIRA-123", fullMessageWithJira, "", false,
|
|
},
|
|
{
|
|
"issue on branch name with prefix and description",
|
|
ccfg,
|
|
"feature/JIRA-123-some-description", "fix: fix something", "\njira: JIRA-123", false,
|
|
},
|
|
{
|
|
"no issue on branch name",
|
|
ccfg,
|
|
"branch", "fix: fix something", "", true,
|
|
},
|
|
{
|
|
"unexpected branch name",
|
|
ccfg,
|
|
"feature /JIRA-123", "fix: fix something", "", true,
|
|
},
|
|
{
|
|
"issue on branch name using hash",
|
|
ccfgHash,
|
|
"JIRA-123-some-description", "fix: fix something", "\njira #JIRA-123", false,
|
|
},
|
|
{
|
|
"numeric issue on branch name",
|
|
ccfgGitIssue,
|
|
"#13", "fix: fix something", "\nissue: #13", false,
|
|
},
|
|
{
|
|
"numeric issue on branch name without hash",
|
|
ccfgGitIssue,
|
|
"13", "fix: fix something", "\nissue: #13", false,
|
|
},
|
|
{
|
|
"numeric issue on branch name with description without hash",
|
|
ccfgGitIssue,
|
|
"13-some-fix", "fix: fix something", "\nissue: #13", false,
|
|
},
|
|
}
|
|
for _, tt := range tests {
|
|
t.Run(tt.name, func(t *testing.T) {
|
|
got, err := NewMessageProcessor(tt.cfg, newBranchCfg(false)).Enhance(tt.branch, tt.message)
|
|
if (err != nil) != tt.wantErr {
|
|
t.Errorf("BaseMessageProcessor.Enhance() error = %v, wantErr %v", err, tt.wantErr)
|
|
|
|
return
|
|
}
|
|
|
|
if got != tt.want {
|
|
t.Errorf("BaseMessageProcessor.Enhance() = %v, want %v", got, tt.want)
|
|
}
|
|
})
|
|
}
|
|
}
|
|
|
|
func TestBaseMessageProcessor_IssueID(t *testing.T) {
|
|
p := NewMessageProcessor(ccfg, newBranchCfg(false))
|
|
|
|
tests := []struct {
|
|
name string
|
|
branch string
|
|
want string
|
|
wantErr bool
|
|
}{
|
|
{"simple branch", "JIRA-123", "JIRA-123", false},
|
|
{"branch with prefix", "feature/JIRA-123", "JIRA-123", false},
|
|
{"branch with prefix and posfix", "feature/JIRA-123-some-description", "JIRA-123", false},
|
|
{"branch not found", "feature/wrong123-some-description", "", false},
|
|
{"empty branch", "", "", false},
|
|
{"unexpected branch name", "feature /JIRA-123", "", false},
|
|
}
|
|
for _, tt := range tests {
|
|
t.Run(tt.name, func(t *testing.T) {
|
|
got, err := p.IssueID(tt.branch)
|
|
if (err != nil) != tt.wantErr {
|
|
t.Errorf("BaseMessageProcessor.IssueID() error = %v, wantErr %v", err, tt.wantErr)
|
|
|
|
return
|
|
}
|
|
|
|
if got != tt.want {
|
|
t.Errorf("BaseMessageProcessor.IssueID() = %v, want %v", got, tt.want)
|
|
}
|
|
})
|
|
}
|
|
}
|
|
|
|
const (
|
|
multilineBody = `a
|
|
b
|
|
c`
|
|
fullFooter = `BREAKING CHANGE: breaks
|
|
jira: JIRA-123`
|
|
)
|
|
|
|
func Test_hasIssueID(t *testing.T) {
|
|
cfgColon := CommitMessageFooterConfig{Key: "jira"}
|
|
cfgHash := CommitMessageFooterConfig{Key: "jira", UseHash: true}
|
|
cfgEmpty := CommitMessageFooterConfig{}
|
|
|
|
tests := []struct {
|
|
name string
|
|
message string
|
|
issueCfg CommitMessageFooterConfig
|
|
want bool
|
|
}{
|
|
{"single line without issue", "feat: something", cfgColon, false},
|
|
{"multi line without issue", `feat: something
|
|
|
|
yay`, cfgColon, false},
|
|
{"multi line without jira issue", `feat: something
|
|
|
|
jira1: JIRA-123`, cfgColon, false},
|
|
{"multi line with issue", `feat: something
|
|
|
|
jira: JIRA-123`, cfgColon, true},
|
|
{"multi line with issue and hash", `feat: something
|
|
|
|
jira #JIRA-123`, cfgHash, true},
|
|
{"empty config", `feat: something
|
|
|
|
jira #JIRA-123`, cfgEmpty, false},
|
|
}
|
|
for _, tt := range tests {
|
|
t.Run(tt.name, func(t *testing.T) {
|
|
if got := hasIssueID(tt.message, tt.issueCfg); got != tt.want {
|
|
t.Errorf("hasIssueID() = %v, want %v", got, tt.want)
|
|
}
|
|
})
|
|
}
|
|
}
|
|
|
|
func Test_hasFooter(t *testing.T) {
|
|
tests := []struct {
|
|
name string
|
|
message string
|
|
want bool
|
|
}{
|
|
{"simple message", "feat: add something", false},
|
|
{"full messsage", fullMessage, true},
|
|
{"full messsage with refs", fullMessageRefs, true},
|
|
{"subject and footer message", subjectAndFooterMessage, true},
|
|
{"subject and body message", subjectAndBodyMessage, false},
|
|
}
|
|
for _, tt := range tests {
|
|
t.Run(tt.name, func(t *testing.T) {
|
|
if got := hasFooter(tt.message); got != tt.want {
|
|
t.Errorf("hasFooter() = %v, want %v", got, tt.want)
|
|
}
|
|
})
|
|
}
|
|
}
|
|
|
|
// conventional commit tests
|
|
|
|
var completeBody = `some descriptions
|
|
|
|
jira: JIRA-123
|
|
BREAKING CHANGE: this change breaks everything`
|
|
|
|
var (
|
|
bodyWithCarriage = "some description\r\nmore description\r\n\r\njira: JIRA-123\r"
|
|
expectedBodyWithCarriage = "some description\nmore description\n\njira: JIRA-123"
|
|
)
|
|
|
|
var issueOnlyBody = `some descriptions
|
|
|
|
jira: JIRA-456`
|
|
|
|
var issueSynonymsBody = `some descriptions
|
|
|
|
Jira: JIRA-789`
|
|
|
|
var hashMetadataBody = `some descriptions
|
|
|
|
Jira: JIRA-999
|
|
Refs #123`
|
|
|
|
func TestBaseMessageProcessor_Parse(t *testing.T) {
|
|
tests := []struct {
|
|
name string
|
|
cfg CommitMessageConfig
|
|
subject string
|
|
body string
|
|
want CommitMessage
|
|
}{
|
|
{
|
|
"simple message",
|
|
ccfg,
|
|
"feat: something awesome", "",
|
|
CommitMessage{
|
|
Type: "feat",
|
|
Scope: "",
|
|
Description: "something awesome",
|
|
Body: "",
|
|
IsBreakingChange: false,
|
|
Metadata: map[string]string{},
|
|
},
|
|
},
|
|
{
|
|
"message with scope",
|
|
ccfg,
|
|
"feat(scope): something awesome", "",
|
|
CommitMessage{
|
|
Type: "feat",
|
|
Scope: "scope",
|
|
Description: "something awesome",
|
|
Body: "",
|
|
IsBreakingChange: false,
|
|
Metadata: map[string]string{},
|
|
},
|
|
},
|
|
{
|
|
"unmapped type",
|
|
ccfg,
|
|
"unkn: something unknown", "",
|
|
CommitMessage{
|
|
Type: "unkn",
|
|
Scope: "",
|
|
Description: "something unknown",
|
|
Body: "",
|
|
IsBreakingChange: false,
|
|
Metadata: map[string]string{},
|
|
},
|
|
},
|
|
{
|
|
"jira and breaking change metadata",
|
|
ccfg,
|
|
"feat: something new", completeBody,
|
|
CommitMessage{
|
|
Type: "feat",
|
|
Scope: "",
|
|
Description: "something new",
|
|
Body: completeBody,
|
|
IsBreakingChange: true,
|
|
Metadata: map[string]string{
|
|
IssueMetadataKey: "JIRA-123",
|
|
BreakingChangeMetadataKey: "this change breaks everything",
|
|
},
|
|
},
|
|
},
|
|
{
|
|
"jira only metadata",
|
|
ccfg,
|
|
"feat: something new", issueOnlyBody,
|
|
CommitMessage{
|
|
Type: "feat",
|
|
Scope: "",
|
|
Description: "something new",
|
|
Body: issueOnlyBody,
|
|
IsBreakingChange: false,
|
|
Metadata: map[string]string{IssueMetadataKey: "JIRA-456"},
|
|
},
|
|
},
|
|
{
|
|
"jira synonyms metadata",
|
|
ccfg,
|
|
"feat: something new", issueSynonymsBody,
|
|
CommitMessage{
|
|
Type: "feat",
|
|
Scope: "",
|
|
Description: "something new",
|
|
Body: issueSynonymsBody,
|
|
IsBreakingChange: false,
|
|
Metadata: map[string]string{IssueMetadataKey: "JIRA-789"},
|
|
},
|
|
},
|
|
{
|
|
"breaking change with empty body",
|
|
ccfg,
|
|
"feat!: something new", "",
|
|
CommitMessage{
|
|
Type: "feat",
|
|
Scope: "",
|
|
Description: "something new",
|
|
Body: "",
|
|
IsBreakingChange: true,
|
|
Metadata: map[string]string{
|
|
BreakingChangeMetadataKey: "something new",
|
|
},
|
|
},
|
|
},
|
|
{
|
|
"hash metadata",
|
|
ccfg,
|
|
"feat: something new", hashMetadataBody,
|
|
CommitMessage{
|
|
Type: "feat",
|
|
Scope: "",
|
|
Description: "something new",
|
|
Body: hashMetadataBody,
|
|
IsBreakingChange: false,
|
|
Metadata: map[string]string{IssueMetadataKey: "JIRA-999", "refs": "#123"},
|
|
},
|
|
},
|
|
{
|
|
"empty issue cfg",
|
|
ccfgEmptyIssue,
|
|
"feat: something new", hashMetadataBody,
|
|
CommitMessage{
|
|
Type: "feat",
|
|
Scope: "",
|
|
Description: "something new",
|
|
Body: hashMetadataBody,
|
|
IsBreakingChange: false,
|
|
Metadata: map[string]string{},
|
|
},
|
|
},
|
|
{
|
|
"carriage return on body",
|
|
ccfg,
|
|
"feat: something new", bodyWithCarriage,
|
|
CommitMessage{
|
|
Type: "feat",
|
|
Scope: "",
|
|
Description: "something new",
|
|
Body: expectedBodyWithCarriage,
|
|
IsBreakingChange: false,
|
|
Metadata: map[string]string{IssueMetadataKey: "JIRA-123"},
|
|
},
|
|
},
|
|
}
|
|
for _, tt := range tests {
|
|
t.Run(tt.name, func(t *testing.T) {
|
|
if got, err := NewMessageProcessor(
|
|
tt.cfg, newBranchCfg(false),
|
|
).Parse(tt.subject, tt.body); !reflect.DeepEqual(got, tt.want) && err == nil {
|
|
t.Errorf("BaseMessageProcessor.Parse() = [%+v], want [%+v]", got, tt.want)
|
|
}
|
|
})
|
|
}
|
|
}
|
|
|
|
func TestBaseMessageProcessor_Format(t *testing.T) {
|
|
tests := []struct {
|
|
name string
|
|
cfg CommitMessageConfig
|
|
msg CommitMessage
|
|
wantHeader string
|
|
wantBody string
|
|
wantFooter string
|
|
}{
|
|
{
|
|
"simple message",
|
|
ccfg,
|
|
NewCommitMessage("feat", "", "something", "", "", ""),
|
|
"feat: something",
|
|
"",
|
|
"",
|
|
},
|
|
{
|
|
"with issue",
|
|
ccfg,
|
|
NewCommitMessage("feat", "", "something", "", "JIRA-123", ""),
|
|
"feat: something",
|
|
"",
|
|
"jira: JIRA-123",
|
|
},
|
|
{
|
|
"with issue using hash",
|
|
ccfgHash,
|
|
NewCommitMessage("feat", "", "something", "", "JIRA-123", ""),
|
|
"feat: something",
|
|
"",
|
|
"jira #JIRA-123",
|
|
},
|
|
{
|
|
"with issue using double hash",
|
|
ccfgHash,
|
|
NewCommitMessage("feat", "", "something", "", "#JIRA-123", ""),
|
|
"feat: something",
|
|
"",
|
|
"jira #JIRA-123",
|
|
},
|
|
{
|
|
"with breaking change",
|
|
ccfg,
|
|
NewCommitMessage("feat", "", "something", "", "", "breaks"),
|
|
"feat: something",
|
|
"",
|
|
"BREAKING CHANGE: breaks",
|
|
},
|
|
{
|
|
"with scope",
|
|
ccfg,
|
|
NewCommitMessage("feat", "scope", "something", "", "", ""),
|
|
"feat(scope): something",
|
|
"",
|
|
"",
|
|
},
|
|
{
|
|
"with body",
|
|
ccfg,
|
|
NewCommitMessage("feat", "", "something", "body", "", ""),
|
|
"feat: something",
|
|
"body",
|
|
"",
|
|
},
|
|
{
|
|
"with multiline body",
|
|
ccfg,
|
|
NewCommitMessage("feat", "", "something", multilineBody, "", ""),
|
|
"feat: something",
|
|
multilineBody,
|
|
"",
|
|
},
|
|
{
|
|
"full message",
|
|
ccfg,
|
|
NewCommitMessage("feat", "scope", "something", multilineBody, "JIRA-123", "breaks"),
|
|
"feat(scope): something",
|
|
multilineBody,
|
|
fullFooter,
|
|
},
|
|
{
|
|
"config without issue key",
|
|
ccfgEmptyIssue,
|
|
NewCommitMessage("feat", "", "something", "", "JIRA-123", ""),
|
|
"feat: something",
|
|
"",
|
|
"",
|
|
},
|
|
{
|
|
"with issue and issue prefix",
|
|
ccfgGitIssue,
|
|
NewCommitMessage("feat", "", "something", "", "123", ""),
|
|
"feat: something",
|
|
"",
|
|
"issue: #123",
|
|
},
|
|
{
|
|
"with #issue and issue prefix",
|
|
ccfgGitIssue,
|
|
NewCommitMessage("feat", "", "something", "", "#123", ""),
|
|
"feat: something",
|
|
"",
|
|
"issue: #123",
|
|
},
|
|
}
|
|
for _, tt := range tests {
|
|
t.Run(tt.name, func(t *testing.T) {
|
|
got, got1, got2 := NewMessageProcessor(tt.cfg, newBranchCfg(false)).Format(tt.msg)
|
|
if got != tt.wantHeader {
|
|
t.Errorf("BaseMessageProcessor.Format() header got = %v, want %v", got, tt.wantHeader)
|
|
}
|
|
|
|
if got1 != tt.wantBody {
|
|
t.Errorf("BaseMessageProcessor.Format() body got = %v, want %v", got1, tt.wantBody)
|
|
}
|
|
|
|
if got2 != tt.wantFooter {
|
|
t.Errorf("BaseMessageProcessor.Format() footer got = %v, want %v", got2, tt.wantFooter)
|
|
}
|
|
})
|
|
}
|
|
}
|
|
|
|
var expectedBodyFullMessage = `
|
|
see the issue for details
|
|
|
|
on typos fixed.
|
|
|
|
Reviewed-by: Z
|
|
Refs #133`
|
|
|
|
func Test_splitCommitMessageContent(t *testing.T) {
|
|
tests := []struct {
|
|
name string
|
|
content string
|
|
wantSubject string
|
|
wantBody string
|
|
}{
|
|
{"single line commit", "feat: something", "feat: something", ""},
|
|
{"multi line commit", fullMessage, "fix: correct minor typos in code", expectedBodyFullMessage},
|
|
}
|
|
for _, tt := range tests {
|
|
t.Run(tt.name, func(t *testing.T) {
|
|
got, got1 := splitCommitMessageContent(tt.content)
|
|
if got != tt.wantSubject {
|
|
t.Errorf("splitCommitMessageContent() subject got = %v, want %v", got, tt.wantSubject)
|
|
}
|
|
|
|
if got1 != tt.wantBody {
|
|
t.Errorf("splitCommitMessageContent() body got1 = [%v], want [%v]", got1, tt.wantBody)
|
|
}
|
|
})
|
|
}
|
|
}
|
|
|
|
func Test_parseSubjectMessage(t *testing.T) {
|
|
tests := []struct {
|
|
name string
|
|
message string
|
|
wantType string
|
|
wantScope string
|
|
wantDescription string
|
|
wantHasBreakingChange bool
|
|
}{
|
|
{"valid commit", "feat: something", "feat", "", "something", false},
|
|
{"valid commit with scope", "feat(scope): something", "feat", "scope", "something", false},
|
|
{"valid commit with breaking change", "feat(scope)!: something", "feat", "scope", "something", true},
|
|
{"missing description", "feat: ", "feat", "", "", false},
|
|
}
|
|
for _, tt := range tests {
|
|
t.Run(tt.name, func(t *testing.T) {
|
|
ctype, scope, description, hasBreakingChange := parseSubjectMessage(tt.message)
|
|
if ctype != tt.wantType {
|
|
t.Errorf("parseSubjectMessage() type got = %v, want %v", ctype, tt.wantType)
|
|
}
|
|
|
|
if scope != tt.wantScope {
|
|
t.Errorf("parseSubjectMessage() scope got = %v, want %v", scope, tt.wantScope)
|
|
}
|
|
|
|
if description != tt.wantDescription {
|
|
t.Errorf("parseSubjectMessage() description got = %v, want %v", description, tt.wantDescription)
|
|
}
|
|
|
|
if hasBreakingChange != tt.wantHasBreakingChange {
|
|
t.Errorf("parseSubjectMessage() hasBreakingChange got = %v, want %v", hasBreakingChange, tt.wantHasBreakingChange)
|
|
}
|
|
})
|
|
}
|
|
}
|
|
|
|
func Test_prepareHeader(t *testing.T) {
|
|
tests := []struct {
|
|
name string
|
|
headerSelector string
|
|
commitHeader string
|
|
wantHeader string
|
|
wantError bool
|
|
}{
|
|
{
|
|
"conventional without selector",
|
|
"",
|
|
"feat: something",
|
|
"feat: something",
|
|
false,
|
|
},
|
|
{
|
|
"conventional with scope without selector",
|
|
"",
|
|
"feat(scope): something",
|
|
"feat(scope): something",
|
|
false,
|
|
},
|
|
{
|
|
"non-conventional without selector",
|
|
"",
|
|
"something", "something",
|
|
false,
|
|
},
|
|
{
|
|
"matching conventional with selector with group",
|
|
"Merged PR (\\d+): (?P<header>.*)",
|
|
"Merged PR 123: feat: something",
|
|
"feat: something",
|
|
false,
|
|
},
|
|
{
|
|
"matching non-conventional with selector with group",
|
|
"Merged PR (\\d+): (?P<header>.*)",
|
|
"Merged PR 123: something",
|
|
"something",
|
|
false,
|
|
},
|
|
{
|
|
"matching non-conventional with selector without group",
|
|
"Merged PR (\\d+): (.*)",
|
|
"Merged PR 123: something",
|
|
"",
|
|
true,
|
|
},
|
|
{
|
|
"non-matching non-conventional with selector with group",
|
|
"Merged PR (\\d+): (?P<header>.*)",
|
|
"something",
|
|
"",
|
|
true,
|
|
},
|
|
{
|
|
"matching non-conventional with invalid regex",
|
|
"Merged PR (\\d+): (<header>.*)",
|
|
"Merged PR 123: something",
|
|
"",
|
|
true,
|
|
},
|
|
}
|
|
for _, tt := range tests {
|
|
t.Run(tt.name, func(t *testing.T) {
|
|
msgProcessor := NewMessageProcessor(newCommitMessageCfg(tt.headerSelector), newBranchCfg(false))
|
|
header, err := msgProcessor.prepareHeader(tt.commitHeader)
|
|
|
|
if tt.wantError && err == nil {
|
|
t.Errorf("prepareHeader() err got = %v, want not nil", err)
|
|
}
|
|
|
|
if header != tt.wantHeader {
|
|
t.Errorf("prepareHeader() header got = %v, want %v", header, tt.wantHeader)
|
|
}
|
|
})
|
|
}
|
|
}
|
|
|
|
func Test_removeCarriage(t *testing.T) {
|
|
tests := []struct {
|
|
name string
|
|
commit string
|
|
want string
|
|
}{
|
|
{"normal string", "normal string", "normal string"},
|
|
{"break line", "normal\nstring", "normal\nstring"},
|
|
{"carriage return", "normal\r\nstring", "normal\nstring"},
|
|
}
|
|
for _, tt := range tests {
|
|
t.Run(tt.name, func(t *testing.T) {
|
|
if got := removeCarriage(tt.commit); got != tt.want {
|
|
t.Errorf("removeCarriage() = %v, want %v", got, tt.want)
|
|
}
|
|
})
|
|
}
|
|
}
|