Skip to content

Commit 201e2c2

Browse files
committed
chore: Removing experiment gate on filter tests
1 parent 90f5f14 commit 201e2c2

11 files changed

+138
-429
lines changed

test/integration_docs_test.go

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -300,10 +300,6 @@ func TestStacksWithLocalStateFileStructure(t *testing.T) {
300300
func TestFilterDocumentationExamples(t *testing.T) {
301301
t.Parallel()
302302

303-
if !helpers.IsExperimentMode(t) {
304-
t.Skip("Skipping filter documentation tests - TG_EXPERIMENT_MODE not enabled")
305-
}
306-
307303
tmpDirRaw := helpers.TmpDirWOSymlinks(t)
308304
tmpDir, err := filepath.EvalSymlinks(tmpDirRaw)
309305
require.NoError(t, err)
@@ -586,11 +582,6 @@ func TestFilterDocumentationExamples(t *testing.T) {
586582
func TestFilterDocumentationExamplesWithUnion(t *testing.T) {
587583
t.Parallel()
588584

589-
// Skip if experiment mode is not enabled
590-
if !helpers.IsExperimentMode(t) {
591-
t.Skip("Skipping filter documentation tests - TG_EXPERIMENT_MODE not enabled")
592-
}
593-
594585
// Create temporary directory for dynamic fixtures
595586
tmpDirRaw := helpers.TmpDirWOSymlinks(t)
596587
tmpDir, err := filepath.EvalSymlinks(tmpDirRaw)

test/integration_download_test.go

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -349,10 +349,6 @@ func TestExcludeDirs(t *testing.T) {
349349
func TestExcludeDirsWithFilter(t *testing.T) {
350350
t.Parallel()
351351

352-
if !helpers.IsExperimentMode(t) {
353-
t.Skip("Skipping filter flag tests - TG_EXPERIMENT_MODE not enabled")
354-
}
355-
356352
// Populate module paths.
357353
moduleNames := []string{
358354
"integration-env/aws/module-aws-a",
@@ -550,11 +546,6 @@ I'm not sure we're getting good value from the time taken on tests like this.
550546
func TestIncludeDirsWithFilter(t *testing.T) {
551547
t.Parallel()
552548

553-
// Skip if filter-flag experiment is not enabled
554-
if !helpers.IsExperimentMode(t) {
555-
t.Skip("Skipping filter flag tests - TG_EXPERIMENT_MODE not enabled")
556-
}
557-
558549
// Copy the entire download fixture directory to ensure all referenced sources are available
559550
tmpDir := helpers.CopyEnvironment(t, "fixtures/download")
560551
workingDir := filepath.Join(tmpDir, testFixtureLocalWithIncludeDir)
@@ -779,10 +770,6 @@ func TestTerragruntExternalDependencies(t *testing.T) {
779770
func TestTerragruntExternalDependenciesWithFilter(t *testing.T) {
780771
t.Parallel()
781772

782-
if !helpers.IsExperimentMode(t) {
783-
t.Skip("Skipping filter flag tests - TG_EXPERIMENT_MODE not enabled")
784-
}
785-
786773
modules := []string{
787774
"module-a",
788775
"module-b",

test/integration_filter_graph_test.go

Lines changed: 0 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -24,9 +24,6 @@ func TestFilterFlagWithFindGraphExpressions(t *testing.T) {
2424
t.Parallel()
2525

2626
// Skip if filter-flag experiment is not enabled
27-
if !helpers.IsExperimentMode(t) {
28-
t.Skip("Skipping filter flag tests - TG_EXPERIMENT_MODE not enabled")
29-
}
3027

3128
testCases := []struct {
3229
name string
@@ -110,11 +107,6 @@ func TestFilterFlagWithFindGraphExpressions(t *testing.T) {
110107
func TestFilterFlagWithFindGraphExpressionsJSON(t *testing.T) {
111108
t.Parallel()
112109

113-
// Skip if filter-flag experiment is not enabled
114-
if !helpers.IsExperimentMode(t) {
115-
t.Skip("Skipping filter flag tests - TG_EXPERIMENT_MODE not enabled")
116-
}
117-
118110
testCases := []struct {
119111
name string
120112
filterQuery string
@@ -186,9 +178,6 @@ func TestFilterFlagWithRunGraphExpressions(t *testing.T) {
186178
t.Parallel()
187179

188180
// Skip if filter-flag experiment is not enabled
189-
if !helpers.IsExperimentMode(t) {
190-
t.Skip("Skipping filter flag tests - TG_EXPERIMENT_MODE not enabled")
191-
}
192181

193182
testCases := []struct {
194183
name string
@@ -279,9 +268,6 @@ func TestFilterFlagWithRunAllGraphExpressions(t *testing.T) {
279268
t.Parallel()
280269

281270
// Skip if filter-flag experiment is not enabled
282-
if !helpers.IsExperimentMode(t) {
283-
t.Skip("Skipping filter flag tests - TG_EXPERIMENT_MODE not enabled")
284-
}
285271

286272
testCases := []struct {
287273
name string
@@ -430,11 +416,6 @@ func TestFilterFlagWithRunAllGraphExpressions(t *testing.T) {
430416
func TestFilterFlagWithRunAllGraphExpressionsVerifyExecutionOrder(t *testing.T) {
431417
t.Parallel()
432418

433-
// Skip if filter-flag experiment is not enabled
434-
if !helpers.IsExperimentMode(t) {
435-
t.Skip("Skipping filter flag tests - TG_EXPERIMENT_MODE not enabled")
436-
}
437-
438419
// This test verifies that when using graph expressions, dependencies are executed before dependents
439420
// We'll use a simple dependency chain to verify execution order
440421
helpers.CleanupTerraformFolder(t, testFixtureRunFilter)

0 commit comments

Comments
 (0)