chore: regression tests

Author: denis256

test/integration_regressions_test.go

@@ -217,61 +217,6 @@ func TestDependencyEmptyConfigPath_ReportsError(t *testing.T) {
 	}
 }
 
-// TestDisabledDependencyEmptyConfigPath_NoCycleError tests that disabled dependencies with empty
-// config_path values do not cause cycle detection errors during discovery.
-// This is a regression test for issue #4977 where setting enabled = false on a dependency
-// with an empty config_path ("") was still causing terragrunt to throw cycle errors.
-//
-// The expected behavior is that disabled dependencies should be completely ignored during
-// dependency graph construction and cycle detection, regardless of their config_path value.
-//
-// See: https://github.com/gruntwork-io/terragrunt/issues/4977
-func TestDisabledDependencyEmptyConfigPath_NoCycleError(t *testing.T) {
-	t.Parallel()
-
-	helpers.CleanupTerraformFolder(t, testFixtureDisabledDependencyEmptyConfigPath)
-	tmpEnvPath := helpers.CopyEnvironment(t, testFixtureDisabledDependencyEmptyConfigPath)
-	rootPath := util.JoinPath(tmpEnvPath, testFixtureDisabledDependencyEmptyConfigPath)
-	helpers.CreateGitRepo(t, rootPath)
-
-	// Test 1: Run against unit-b which has disabled dependency with empty config_path
-	// This should NOT fail with cycle detection or empty config_path errors
-	unitBPath := util.JoinPath(rootPath, "unit-b")
-	stdout, stderr, err := helpers.RunTerragruntCommandWithOutput(
-		t,
-		"terragrunt plan --non-interactive --working-dir "+unitBPath,
-	)
-
-	// The command should succeed - disabled dependencies with empty paths should be ignored
-	require.NoError(t, err, "plan should succeed when disabled dependency has empty config_path")
-
-	// Should not see cycle detection errors
-	combinedOutput := stdout + stderr
-	assert.NotContains(t, combinedOutput, "cycle",
-		"Should not see cycle detection errors for disabled dependencies")
-	assert.NotContains(t, combinedOutput, "Cycle detected",
-		"Should not see 'Cycle detected' error")
-
-	// Should not see empty config_path errors since the dependency is disabled
-	assert.NotContains(t, combinedOutput, "has empty config_path",
-		"Should not see empty config_path error for disabled dependency")
-
-	// Test 2: Also test run --all from the root to ensure discovery doesn't fail
-	_, runAllStderr, runAllErr := helpers.RunTerragruntCommandWithOutput(
-		t,
-		"terragrunt run --all plan --non-interactive --working-dir "+rootPath,
-	)
-
-	// run --all should also succeed
-	require.NoError(t, runAllErr, "run --all plan should succeed")
-
-	// Should not see cycle or graph-related errors during discovery
-	assert.NotContains(t, runAllStderr, "cycle",
-		"run --all should not see cycle errors")
-	assert.NotContains(t, runAllStderr, "dependency graph",
-		"run --all should not see dependency graph errors")
-}
-
 // TestExposedIncludeWithDeprecatedInputsSyntax tests that deprecated dependency.*.inputs.* syntax
 // is properly detected even when used in an included config with expose = true.
 // This is a regression test for a bug introduced in v0.91.1 where the partial parse path
@@ -427,3 +372,50 @@ func TestSensitiveValues(t *testing.T) {
 	assert.Equal(t, "25", passwordLengthStr,
 		"Password length should match dev password")
 }
+
+// TestDisabledDependencyEmptyConfigPath_NoCycleError tests that disabled dependencies with empty
+// config_path values do not cause cycle detection errors during discovery.
+// This is a regression test for issue #4977 where setting enabled = false on a dependency
+// with an empty config_path ("") was still causing terragrunt to throw cycle errors.
+//
+// The expected behavior is that disabled dependencies should be completely ignored during
+// dependency graph construction and cycle detection, regardless of their config_path value.
+//
+// See: https://github.com/gruntwork-io/terragrunt/issues/4977
+func TestDisabledDependencyEmptyConfigPath_NoCycleError(t *testing.T) {
+	t.Parallel()
+
+	helpers.CleanupTerraformFolder(t, testFixtureDisabledDependencyEmptyConfigPath)
+	tmpEnvPath := helpers.CopyEnvironment(t, testFixtureDisabledDependencyEmptyConfigPath)
+	rootPath := util.JoinPath(tmpEnvPath, testFixtureDisabledDependencyEmptyConfigPath)
+	helpers.CreateGitRepo(t, rootPath)
+
+	unitBPath := util.JoinPath(rootPath, "unit-b")
+	stdout, stderr, err := helpers.RunTerragruntCommandWithOutput(
+		t,
+		"terragrunt plan --non-interactive --working-dir "+unitBPath,
+	)
+
+	require.NoError(t, err, "plan should succeed when disabled dependency has empty config_path")
+
+	combinedOutput := stdout + stderr
+	assert.NotContains(t, combinedOutput, "cycle",
+		"Should not see cycle detection errors for disabled dependencies")
+	assert.NotContains(t, combinedOutput, "Cycle detected",
+		"Should not see 'Cycle detected' error")
+
+	assert.NotContains(t, combinedOutput, "has empty config_path",
+		"Should not see empty config_path error for disabled dependency")
+
+	_, runAllStderr, runAllErr := helpers.RunTerragruntCommandWithOutput(
+		t,
+		"terragrunt run --all plan --non-interactive --working-dir "+rootPath,
+	)
+
+	require.NoError(t, runAllErr, "run --all plan should succeed")
+
+	assert.NotContains(t, runAllStderr, "cycle",
+		"run --all should not see cycle errors")
+	assert.NotContains(t, runAllStderr, "dependency graph",
+		"run --all should not see dependency graph errors")
+}