rfscholte commented on a change in pull request #82:
URL:
https://github.com/apache/maven-integration-testing/pull/82#discussion_r546804763
##########
File path:
core-it-suite/src/test/java/org/apache/maven/it/MavenITmng5576CdFriendlyVersions.java
##########
@@ -66,4 +67,33 @@ public void
testContinuousDeliveryFriendlyVersionsAreWarningFree()
}
}
+ /**
+ * Verifies that property references with dotted notation work within
+ * POM interpolation.
+ */
+ public void
testContinuousDeliveryFriendlyVersionsAreWarningFreeWithBuildConsumer()
+ throws Exception
+ {
+ File testDir = ResourceExtractor.simpleExtractResources( getClass(),
"/mng-5576-cd-friendly-versions" );
+
+ Verifier verifier = newVerifier( testDir.getAbsolutePath() );
+ verifier.setLogFileName( "log-bc.txt" );
+ verifier.setAutoclean( false );
+ verifier.deleteDirectory( "target" );
+ verifier.addCliOption( "-Dchangelist=changelist" );
+ verifier.addCliOption( "-Dmaven.experimental.buildconsumer=true" );
+ verifier.executeGoal( "validate" );
+ verifier.verifyErrorFreeLog();
+ verifier.resetStreams();
+
+ Properties props = verifier.loadProperties( "target/pom.properties" );
+ assertEquals( "1.0.0.changelist", props.getProperty( "project.version"
) );
+
+ List<String> lines = verifier.loadFile( new File( testDir,
"log-bc.txt" ), false );
+ for( String line : lines )
Review comment:
It has never been implemented and it is a shared component, so requires
an extra step to make it available. See
https://maven.apache.org/shared/maven-verifier/apidocs/org/apache/maven/it/Verifier.html
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
