feat: rename flag to force-bump-patch-version and adapt tests accordingly

Author: timoknapp

pkg/config/config.go

@@ -33,7 +33,7 @@ type Config struct {
 	Dry                             bool
 	AllowInitialDevelopmentVersions bool
 	AllowNoChanges                  bool
-	ForceBumpPatch                  bool
+	ForceBumpPatchVersion           bool
 	MaintainedVersion               string
 	PrependChangelog                bool
 	DownloadPlugins                 bool
@@ -123,7 +123,7 @@ func NewConfig(cmd *cobra.Command) (*Config, error) {
 		Dry:                             mustGetBool(cmd, "dry"),
 		AllowInitialDevelopmentVersions: mustGetBool(cmd, "allow-initial-development-versions"),
 		AllowNoChanges:                  mustGetBool(cmd, "allow-no-changes"),
-		ForceBumpPatch:                  mustGetBool(cmd, "force-bump-patch"),
+		ForceBumpPatchVersion:           mustGetBool(cmd, "force-bump-patch-version"),
 		MaintainedVersion:               viper.GetString("maintainedVersion"),
 		PrependChangelog:                mustGetBool(cmd, "prepend-changelog"),
 		DownloadPlugins:                 mustGetBool(cmd, "download-plugins"),

pkg/semrel/semrel.go

@@ -71,5 +71,5 @@ func applyChange(rawVersion string, rawChange *Change, allowInitialDevelopmentVe
 }
 
 func GetNewVersion(conf *config.Config, commits []*Commit, latestRelease *Release) string {
-	return applyChange(latestRelease.Version, calculateChange(commits, latestRelease), conf.AllowInitialDevelopmentVersions, conf.ForceBumpPatch)
+	return applyChange(latestRelease.Version, calculateChange(commits, latestRelease), conf.AllowInitialDevelopmentVersions, conf.ForceBumpPatchVersion)
 }

pkg/semrel/semrel_test.go

@@ -39,28 +39,30 @@ func TestApplyChange(t *testing.T) {
 		change                          *Change
 		expectedVersion                 string
 		allowInitialDevelopmentVersions bool
+		forceBumpPatchVersion           bool
 	}{
 		// No Previous releases
-		{"0.0.0", NoChange, "1.0.0", false},
-		{"0.0.0", PatchChange, "1.0.0", false},
-		{"0.0.0", MinorChange, "1.0.0", false},
-		{"0.0.0", MajorChange, "1.0.0", false},
+		{"0.0.0", NoChange, "1.0.0", false, false},
+		{"0.0.0", PatchChange, "1.0.0", false, false},
+		{"0.0.0", MinorChange, "1.0.0", false, false},
+		{"0.0.0", MajorChange, "1.0.0", false, false},
 
-		{"0.0.0", NoChange, "0.1.0", true},
-		{"0.0.0", PatchChange, "0.1.0", true},
-		{"0.0.0", MinorChange, "0.1.0", true},
-		{"0.0.0", MajorChange, "1.0.0", true},
+		{"0.0.0", NoChange, "0.1.0", true, false},
+		{"0.0.0", PatchChange, "0.1.0", true, false},
+		{"0.0.0", MinorChange, "0.1.0", true, false},
+		{"0.0.0", MajorChange, "1.0.0", true, false},
 
-		{"1.0.0", NoChange, "", false},
-		{"1.0.0", PatchChange, "1.0.1", false},
-		{"1.0.0", MinorChange, "1.1.0", false},
-		{"1.0.0", MajorChange, "2.0.0", false},
-		{"0.1.0", NoChange, "1.0.0", false},
-		{"0.1.0", NoChange, "", true},
+		{"1.0.0", NoChange, "", false, false},
+		{"1.0.0", PatchChange, "1.0.1", false, true},
+		{"1.0.0", PatchChange, "1.0.1", false, false},
+		{"1.0.0", MinorChange, "1.1.0", false, false},
+		{"1.0.0", MajorChange, "2.0.0", false, false},
+		{"0.1.0", NoChange, "1.0.0", false, false},
+		{"0.1.0", NoChange, "", true, false},
 
-		{"2.0.0-beta", MajorChange, "2.0.0-beta.1", false},
-		{"2.0.0-beta.2", MajorChange, "2.0.0-beta.3", false},
-		{"2.0.0-beta.1.1", MajorChange, "2.0.0-beta.2", false},
+		{"2.0.0-beta", MajorChange, "2.0.0-beta.1", false, false},
+		{"2.0.0-beta.2", MajorChange, "2.0.0-beta.3", false, false},
+		{"2.0.0-beta.1.1", MajorChange, "2.0.0-beta.2", false, false},
 	}
 
 	for _, tc := range testCases {
@@ -71,7 +73,7 @@ func TestApplyChange(t *testing.T) {
 				t.Errorf("failed to create version: %v", err)
 			}
 
-			actual := applyChange(current.String(), tc.change, tc.allowInitialDevelopmentVersions)
+			actual := applyChange(current.String(), tc.change, tc.allowInitialDevelopmentVersions, tc.forceBumpPatchVersion)
 
 			// Handle no new version case
 			if actual != "" && tc.expectedVersion != "" {