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: us...@infra.apache.org