Merge pull request #44 from yrodiere/i26

yrodiere · web-flow · commit 452e6701a204 · 2025-03-21T14:17:31.000+01:00
Fix incorrect condition to ignore environment variables for build cache
diff --git a/src/main/java/org/hibernate/infra/develocity/plugins/SurefireConfiguredPlugin.java b/src/main/java/org/hibernate/infra/develocity/plugins/SurefireConfiguredPlugin.java
@@ -64,7 +64,7 @@ private void configureEnvironmentVariables(GoalMetadataProvider.Context context)
 					value = "";
 				}
 				if ( value.startsWith( context.metadata().getSession().getExecutionRootDirectory() ) ) {
-					if ( TEST_INDEXES_PATTERN.matcher( "test-indexes" ).find() ) {
+					if ( TEST_INDEXES_PATTERN.matcher( value ).find() ) {
 						// Lucene indexes used in tests -- we don't care about these.
 						inputs.ignore( keyForDevelocity );
 					}

Original file line number	Diff line number	Diff line change
`@@ -64,7 +64,7 @@ private void configureEnvironmentVariables(GoalMetadataProvider.Context context)`
`64`	`64`	`value = "";`
`65`	`65`	`}`
`66`	`66`	`if ( value.startsWith( context.metadata().getSession().getExecutionRootDirectory() ) ) {`
`67`		`- if ( TEST_INDEXES_PATTERN.matcher( "test-indexes" ).find() ) {`
	`67`	`+ if ( TEST_INDEXES_PATTERN.matcher( value ).find() ) {`
`68`	`68`	`// Lucene indexes used in tests -- we don't care about these.`
`69`	`69`	`inputs.ignore( keyForDevelocity );`
`70`	`70`	`}`