fix: Addressing lints

Author: yhakbar

internal/awshelper/policy_test.go

@@ -92,15 +92,15 @@ func TestAwsUnmarshalActionResourceList(t *testing.T) {
 	assert.NotNil(t, bucketPolicy.Statement[0].Resource)
 
 	switch actions := bucketPolicy.Statement[0].Action.(type) {
-	case []interface{}:
+	case []any:
 		assert.Len(t, actions, 11)
 		assert.Contains(t, actions, "s3:ListJobs")
 	default:
 		assert.Fail(t, "Expected []string type for Action")
 	}
 
 	switch resource := bucketPolicy.Statement[0].Resource.(type) {
-	case []interface{}:
+	case []any:
 		assert.Len(t, resource, 2)
 		assert.Contains(t, resource, "arn:aws:s3:*:666:job/*")
 	default:

internal/runner/configstack/runner.go

@@ -664,7 +664,7 @@ func (runner *Runner) acquireCredentials(ctx context.Context, l log.Logger, opts
 }
 
 // nolint:unparam
-func (runner *Runner) partialParseConfig(ctx context.Context, parseCtx *config.ParsingContext, l log.Logger, terragruntConfigPath string, includeConfig *config.IncludeConfig, howThisUnitWasFound string) (*config.TerragruntConfig, error) {
+func (runner *Runner) partialParseConfig(_ context.Context, parseCtx *config.ParsingContext, l log.Logger, terragruntConfigPath string, includeConfig *config.IncludeConfig, howThisUnitWasFound string) (*config.TerragruntConfig, error) {
 	terragruntConfig, err := config.PartialParseConfigFile( //nolint:contextcheck
 		parseCtx,
 		l,

internal/runner/runner.go

@@ -2,6 +2,7 @@
 package runner
 
 import (
+	"maps"
 	"context"
 	"path/filepath"
 	"slices"
@@ -41,9 +42,7 @@ func FindWhereWorkingDirIsIncluded(ctx context.Context, l log.Logger, opts *opti
 	pathsToCheck := discoverPathsToCheck(ctx, l, opts, terragruntConfig)
 
 	for _, dir := range pathsToCheck {
-		for k, v := range findMatchingUnitsInPath(ctx, l, dir, opts, terragruntConfig) {
-			matchedModulesMap[k] = v
-		}
+		maps.Copy(matchedModulesMap, findMatchingUnitsInPath(ctx, l, dir, opts, terragruntConfig))
 	}
 
 	var matchedModules = make(common.Units, 0, len(matchedModulesMap))

test/helpers/aws.go

@@ -13,7 +13,7 @@ import (
 func DeleteS3BucketWithRetry(t *testing.T, awsRegion string, bucketName string) {
 	t.Helper()
 
-	for i := 0; i < 3; i++ {
+	for range 3 {
 		err := DeleteS3Bucket(t, awsRegion, bucketName)
 		if err == nil {
 			return

test/integration_aws_test.go

@@ -1299,7 +1299,7 @@ func TestAwsDependencyOutputSameOutputConcurrencyRegression(t *testing.T) {
 		require.NoError(t, err)
 	}
 
-	for i := 0; i < 3; i++ {
+	for range 3 {
 		tt()
 		// We need to bust the output cache that stores the dependency outputs so that the second run pulls the outputs.
 		// This is only a problem during testing, where the process is shared across terragrunt runs.
@@ -1400,7 +1400,7 @@ func TestAwsParallelStateInit(t *testing.T) {
 	t.Parallel()
 
 	tmpEnvPath := t.TempDir()
-	for i := 0; i < 20; i++ {
+	for i := range 20 {
 		err := util.CopyFolderContents(logger.CreateLogger(), testFixtureParallelStateInit, tmpEnvPath, ".terragrunt-test", nil, nil)
 		require.NoError(t, err)
 		err = os.Rename(

test/integration_find_test.go

@@ -166,7 +166,7 @@ func TestFindDAGWithMixedDependencies(t *testing.T) {
 }
 
 // jsonStringsEqual compares two JSON strings for equivalence, ignoring the order of nested arrays.
-func jsonStringsEqual(t *testing.T, expected, actual string, msgAndArgs ...interface{}) bool {
+func jsonStringsEqual(t *testing.T, expected, actual string, msgAndArgs ...any) bool {
 	t.Helper()
 
 	patch, err := jsondiff.CompareJSON([]byte(expected), []byte(actual), jsondiff.Equivalent())

test/integration_runner_pool_test.go

@@ -67,7 +67,7 @@ func TestRunnerPoolTerragruntDestroyOrder(t *testing.T) {
 	// Parse the destruction order from stdout
 	var destroyOrder []string
 	re := regexp.MustCompile(`Hello, Module ([A-Za-z]+)`)
-	for _, line := range strings.Split(stdout, "\n") {
+	for line := range strings.SplitSeq(stdout, "\n") {
 		if match := re.FindStringSubmatch(line); match != nil {
 			destroyOrder = append(destroyOrder, "module-"+strings.ToLower(match[1]))
 		}

test/integration_serial_aws_test.go

@@ -162,7 +162,7 @@ func testRemoteFixtureParallelism(t *testing.T, parallelism int, numberOfModules
 
 	// copy the template `numberOfModules` times into the app
 	tmpEnvPath := t.TempDir()
-	for i := 0; i < numberOfModules; i++ {
+	for i := range numberOfModules {
 		err := util.CopyFolderContents(createLogger(), testFixtureParallelism, tmpEnvPath, ".terragrunt-test", nil, nil)
 		if err != nil {
 			return "", 0, err

test/integration_sops_kms_test.go

@@ -36,13 +36,13 @@ func TestAwsSopsDecryptedKMSCorrectly(t *testing.T) {
 	outputs := map[string]helpers.TerraformOutput{}
 	require.NoError(t, json.Unmarshal(stdout.Bytes(), &outputs))
 
-	assert.Equal(t, []interface{}{true, false}, outputs["json_bool_array"].Value)
-	assert.Equal(t, []interface{}{"example_value1", "example_value2"}, outputs["json_string_array"].Value)
+	assert.Equal(t, []any{true, false}, outputs["json_bool_array"].Value)
+	assert.Equal(t, []any{"example_value1", "example_value2"}, outputs["json_string_array"].Value)
 	assert.InEpsilon(t, 1234.56789, outputs["json_number"].Value, 0.0001)
 	assert.Equal(t, "example_value", outputs["json_string"].Value)
 	assert.Equal(t, "Welcome to SOPS! Edit this file as you please!", outputs["json_hello"].Value)
-	assert.Equal(t, []interface{}{true, false}, outputs["yaml_bool_array"].Value)
-	assert.Equal(t, []interface{}{"example_value1", "example_value2"}, outputs["yaml_string_array"].Value)
+	assert.Equal(t, []any{true, false}, outputs["yaml_bool_array"].Value)
+	assert.Equal(t, []any{"example_value1", "example_value2"}, outputs["yaml_string_array"].Value)
 	assert.InEpsilon(t, 1234.5679, outputs["yaml_number"].Value, 0.0001)
 	assert.Equal(t, "example_value", outputs["yaml_string"].Value)
 	assert.Equal(t, "Welcome to SOPS! Edit this file as you please!", outputs["yaml_hello"].Value)