maven-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bentm...@apache.org
Subject svn commit: r931922 - in /maven/core-integration-testing/trunk/core-it-suite/src/test/java/org/apache/maven/it: IntegrationTestSuite.java MavenITmng3645POMSyntaxErrorTest.java
Date Thu, 08 Apr 2010 13:11:13 GMT
Author: bentmann
Date: Thu Apr  8 13:11:13 2010
New Revision: 931922

URL: http://svn.apache.org/viewvc?rev=931922&view=rev
Log:
o Enabled overlooked IT

Modified:
    maven/core-integration-testing/trunk/core-it-suite/src/test/java/org/apache/maven/it/IntegrationTestSuite.java
    maven/core-integration-testing/trunk/core-it-suite/src/test/java/org/apache/maven/it/MavenITmng3645POMSyntaxErrorTest.java

Modified: maven/core-integration-testing/trunk/core-it-suite/src/test/java/org/apache/maven/it/IntegrationTestSuite.java
URL: http://svn.apache.org/viewvc/maven/core-integration-testing/trunk/core-it-suite/src/test/java/org/apache/maven/it/IntegrationTestSuite.java?rev=931922&r1=931921&r2=931922&view=diff
==============================================================================
--- maven/core-integration-testing/trunk/core-it-suite/src/test/java/org/apache/maven/it/IntegrationTestSuite.java
(original)
+++ maven/core-integration-testing/trunk/core-it-suite/src/test/java/org/apache/maven/it/IntegrationTestSuite.java
Thu Apr  8 13:11:13 2010
@@ -72,7 +72,6 @@ public class IntegrationTestSuite
         // Tests that currently don't pass for any Maven version, i.e. the corresponding
issue hasn't been resolved yet
         // -------------------------------------------------------------------------------------------------------------
         // suite.addTestSuite( MavenITmng3890TransitiveDependencyScopeUpdateTest.class );
-        // suite.addTestSuite( MavenITmng3645POMSyntaxErrorTest.class );
         // suite.addTestSuite( MavenITmng3038TransitiveDepManVersionTest.class );
         // suite.addTestSuite( MavenITmng2994SnapshotRangeRepositoryTest.class );
         // suite.addTestSuite( MavenITmng2771PomExtensionComponentOverrideTest.class );
@@ -327,6 +326,7 @@ public class IntegrationTestSuite
         suite.addTestSuite( MavenITmng3671PluginLevelDepInterpolationTest.class );
         suite.addTestSuite( MavenITmng3667ResolveDepsWithBadPomVersionTest.class );
         suite.addTestSuite( MavenITmng3652UserAgentHeaderTest.class );
+        suite.addTestSuite( MavenITmng3645POMSyntaxErrorTest.class );
         suite.addTestSuite( MavenITmng3642DynamicResourcesTest.class );
         suite.addTestSuite( MavenITmng3641ProfileActivationWarningTest.class );
         suite.addTestSuite( MavenITmng3621UNCInheritedPathsTest.class );

Modified: maven/core-integration-testing/trunk/core-it-suite/src/test/java/org/apache/maven/it/MavenITmng3645POMSyntaxErrorTest.java
URL: http://svn.apache.org/viewvc/maven/core-integration-testing/trunk/core-it-suite/src/test/java/org/apache/maven/it/MavenITmng3645POMSyntaxErrorTest.java?rev=931922&r1=931921&r2=931922&view=diff
==============================================================================
--- maven/core-integration-testing/trunk/core-it-suite/src/test/java/org/apache/maven/it/MavenITmng3645POMSyntaxErrorTest.java
(original)
+++ maven/core-integration-testing/trunk/core-it-suite/src/test/java/org/apache/maven/it/MavenITmng3645POMSyntaxErrorTest.java
Thu Apr  8 13:11:13 2010
@@ -37,23 +37,27 @@ import java.io.File;
 public class MavenITmng3645POMSyntaxErrorTest
     extends AbstractMavenIntegrationTestCase
 {
+
     public MavenITmng3645POMSyntaxErrorTest()
     {
-        super( "(2.0.9,2.1.0-M1),(2.1.0-M1,)" ); // only test in 2.0.10+, but not 2.1.0-M1
+        super( "[2.0.10,2.1.0-M1),[2.1.0,3.0-alpha-1),[3.0-alpha-3,)" );
     }
 
-    public void testitMNG3645 ()
+    /**
+     * Verify that POMs of reactor projects are parsed in strict mode.
+     */
+    public void testit()
         throws Exception
     {
-        // The testdir is computed from the location of this
-        // file.
         File testDir = ResourceExtractor.simpleExtractResources( getClass(), "/mng-3645"
);
 
         Verifier verifier = new Verifier( testDir.getAbsolutePath() );
+        verifier.setAutoclean( false );
 
         try
         {
             verifier.executeGoal( "validate" );
+            verifier.verifyErrorFreeLog();
 
             fail( "Should fail to validate the POM syntax due to missing dependency element
inside dependencyManagement section." );
         }
@@ -61,5 +65,10 @@ public class MavenITmng3645POMSyntaxErro
         {
             // expect this.
         }
+        finally
+        {
+            verifier.resetStreams();
+        }
     }
+
 }



Mime
View raw message