Extended support for snapshot policy for resolver in Maven Projects

Author: fluxxBot

common/build/buildinfoproperties.go

@@ -45,6 +45,9 @@ const Repo = "repo"
 const SnapshotRepo = "snapshotRepo"
 const ReleaseRepo = "releaseRepo"
 
+const DisableSnapshots = "disableSnapshots"
+const SnapshotsUpdatePolicy = "snapshotsUpdatePolicy"
+
 const ServerId = "serverId"
 const Url = "url"
 const Username = "username"
@@ -123,6 +126,8 @@ var mavenConfigMapping = map[string]string{
 	"buildInfoConfig.artifactoryResolutionEnabled":      "buildInfoConfig.artifactoryResolutionEnabled",
 	"resolve.repoKey":                          ResolverPrefix + ReleaseRepo,
 	"resolve.downSnapshotRepoKey":              ResolverPrefix + SnapshotRepo,
+	"resolve.snapshots.disabled":               ResolverPrefix + DisableSnapshots,
+	"resolve.snapshots.updatePolicy":           ResolverPrefix + SnapshotsUpdatePolicy,
 	"publish.repoKey":                          DeployerPrefix + ReleaseRepo,
 	"publish.snapshot.repoKey":                 DeployerPrefix + SnapshotRepo,
 	"publish.includePatterns":                  DeployerPrefix + IncludePatterns,

common/commands/configfile.go

@@ -39,6 +39,8 @@ const (
 	deploymentSnapshotsRepo = "repo-deploy-snapshots"
 	includePatterns         = "include-patterns"
 	excludePatterns         = "exclude-patterns"
+	disableSnapshots        = "disable-snapshots"
+	snapshotsUpdatePolicy   = "snapshots-update-policy"
 
 	// Gradle flags
 	usesPlugin          = "uses-plugin"
@@ -251,14 +253,16 @@ func WithDeployerRepo(repoId string) ConfigOption {
 // Populate Maven related configuration from cli flags
 func (configFile *ConfigFile) populateMavenConfigFromFlags(c *cli.Context) {
 	configFile.Resolver.SnapshotRepo = c.String(resolutionSnapshotsRepo)
+	configFile.Resolver.DisableSnapshots = c.Bool(disableSnapshots)
+	configFile.Resolver.SnapshotsUpdatePolicy = c.String(snapshotsUpdatePolicy)
 	configFile.Resolver.ReleaseRepo = c.String(resolutionReleasesRepo)
 	configFile.Deployer.SnapshotRepo = c.String(deploymentSnapshotsRepo)
 	configFile.Deployer.ReleaseRepo = c.String(deploymentReleasesRepo)
 	configFile.Deployer.IncludePatterns = c.String(includePatterns)
 	configFile.Deployer.ExcludePatterns = c.String(excludePatterns)
 	configFile.UseWrapper = c.Bool(useWrapper)
 	configFile.Interactive = configFile.Interactive && !isAnyFlagSet(c, resolutionSnapshotsRepo, resolutionReleasesRepo,
-		deploymentSnapshotsRepo, deploymentReleasesRepo, includePatterns, excludePatterns)
+		disableSnapshots, snapshotsUpdatePolicy, deploymentSnapshotsRepo, deploymentReleasesRepo, includePatterns, excludePatterns)
 }
 
 func WithResolverSnapshotRepo(repoId string) ConfigOption {

common/commands/flag_test.go

@@ -0,0 +1,46 @@
+package commands
+
+import (
+	"github.com/stretchr/testify/assert"
+	"testing"
+
+	"github.com/urfave/cli"
+)
+
+// This test demonstrates an existing bug in the urfave/cli library.
+// It highlights that both BoolT and Bool flags have their default values set to false.
+// Ideally, the BoolT flag should have a default value of true.
+func TestBoolVsBoolTFlag(t *testing.T) {
+	tests := []struct {
+		name           string
+		args           []string
+		shouldUseBoolT bool
+		expectValue    bool
+	}{
+		{"Resolving flag value using Bool (default false)", []string{"cmd"}, false, false},
+		{"Resolving flag value using Bool (default false)", []string{"cmd"}, true, false},
+	}
+
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			app := &cli.App{
+				Flags: []cli.Flag{
+					&cli.BoolFlag{
+						Name:  "myflag",
+						Usage: "Test boolean flag",
+					},
+				},
+				Action: func(c *cli.Context) error {
+					if tt.shouldUseBoolT {
+						assert.Equal(t, tt.expectValue, c.BoolT("myflag"), "Expected %v, got %v", tt.expectValue, c.BoolT("myflag"))
+					} else {
+						assert.Equal(t, tt.expectValue, c.Bool("myflag"), "Expected %v, got %v", tt.expectValue, c.Bool("myflag"))
+					}
+					return nil
+				},
+			}
+
+			_ = app.Run(tt.args)
+		})
+	}
+}

common/project/projectconfig.go

@@ -96,17 +96,19 @@ func (mre *MissingResolverErr) Error() string {
 }
 
 type Repository struct {
-	Repo             string `yaml:"repo,omitempty"`
-	ServerId         string `yaml:"serverId,omitempty"`
-	SnapshotRepo     string `yaml:"snapshotRepo,omitempty"`
-	ReleaseRepo      string `yaml:"releaseRepo,omitempty"`
-	DeployMavenDesc  bool   `yaml:"deployMavenDescriptors,omitempty"`
-	DeployIvyDesc    bool   `yaml:"deployIvyDescriptors,omitempty"`
-	IvyPattern       string `yaml:"ivyPattern,omitempty"`
-	ArtifactsPattern string `yaml:"artifactPattern,omitempty"`
-	NugetV2          bool   `yaml:"nugetV2,omitempty"`
-	IncludePatterns  string `yaml:"includePatterns,omitempty"`
-	ExcludePatterns  string `yaml:"excludePatterns,omitempty"`
+	Repo                  string `yaml:"repo,omitempty"`
+	ServerId              string `yaml:"serverId,omitempty"`
+	SnapshotRepo          string `yaml:"snapshotRepo,omitempty"`
+	DisableSnapshots      bool   `yaml:"disableSnapshots,omitempty"`
+	SnapshotsUpdatePolicy string `yaml:"snapshotsUpdatePolicy,omitempty"`
+	ReleaseRepo           string `yaml:"releaseRepo,omitempty"`
+	DeployMavenDesc       bool   `yaml:"deployMavenDescriptors,omitempty"`
+	DeployIvyDesc         bool   `yaml:"deployIvyDescriptors,omitempty"`
+	IvyPattern            string `yaml:"ivyPattern,omitempty"`
+	ArtifactsPattern      string `yaml:"artifactPattern,omitempty"`
+	NugetV2               bool   `yaml:"nugetV2,omitempty"`
+	IncludePatterns       string `yaml:"includePatterns,omitempty"`
+	ExcludePatterns       string `yaml:"excludePatterns,omitempty"`
 }
 
 type RepositoryConfig struct {