[MSOURCES-92] Generate Maven Descriptor by default
Fixed unit tests and added appropriate integration test
to be sure you can disable the behaviour.


git-svn-id: https://svn.apache.org/repos/asf/maven/plugins/trunk@1717030 13f79535-47bb-0310-9956-ffa450edef68
diff --git a/src/it/jar-without-descriptor-config/invoker.properties b/src/it/jar-without-descriptor-config/invoker.properties
new file mode 100644
index 0000000..47840a1
--- /dev/null
+++ b/src/it/jar-without-descriptor-config/invoker.properties
@@ -0,0 +1,18 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements.  See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership.  The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License.  You may obtain a copy of the License at
+# 
+#   http://www.apache.org/licenses/LICENSE-2.0
+# 
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied.  See the License for the
+# specific language governing permissions and limitations
+# under the License.
+
+invoker.goals=${project.groupId}:${project.artifactId}:${project.version}:jar
diff --git a/src/it/jar-without-descriptor-config/pom.xml b/src/it/jar-without-descriptor-config/pom.xml
new file mode 100644
index 0000000..b2801d9
--- /dev/null
+++ b/src/it/jar-without-descriptor-config/pom.xml
@@ -0,0 +1,51 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<!--
+  ~ Licensed to the Apache Software Foundation (ASF) under one
+  ~ or more contributor license agreements.  See the NOTICE file
+  ~ distributed with this work for additional information
+  ~ regarding copyright ownership.  The ASF licenses this file
+  ~ to you under the Apache License, Version 2.0 (the
+  ~ "License"); you may not use this file except in compliance
+  ~ with the License.  You may obtain a copy of the License at
+  ~
+  ~   http://www.apache.org/licenses/LICENSE-2.0
+  ~
+  ~ Unless required by applicable law or agreed to in writing,
+  ~ software distributed under the License is distributed on an
+  ~ "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+  ~ KIND, either express or implied.  See the License for the
+  ~ specific language governing permissions and limitations
+  ~ under the License.
+  -->
+
+<project xmlns="http://maven.apache.org/POM/4.0.0"
+         xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+         xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd">
+  <modelVersion>4.0.0</modelVersion>
+
+  <groupId>org.apache.maven.its.ejb</groupId>
+  <artifactId>jar-without-descriptor-config</artifactId>
+  <version>1.0-SNAPSHOT</version>
+
+  <name>Test of archive configuration without Maven Descriptor as pre 3.0.0 era</name>
+
+  <properties>
+    <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
+  </properties>
+
+  <build>
+    <plugins>
+      <plugin>
+        <groupId>org.apache.maven.plugins</groupId>
+        <artifactId>maven-source-plugin</artifactId>
+        <version>@project.version@</version>
+        <configuration>
+          <archive>
+            <addMavenDescriptor>false</addMavenDescriptor>
+          </archive>
+        </configuration>
+      </plugin>
+    </plugins>
+  </build>
+
+</project>
diff --git a/src/it/jar-without-descriptor-config/src/main/java/MyClass.java b/src/it/jar-without-descriptor-config/src/main/java/MyClass.java
new file mode 100644
index 0000000..4945381
--- /dev/null
+++ b/src/it/jar-without-descriptor-config/src/main/java/MyClass.java
@@ -0,0 +1,23 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ * 
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ * 
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+
+public class MyClass
+{
+
+}
diff --git a/src/it/jar-without-descriptor-config/src/main/resources/main.properties b/src/it/jar-without-descriptor-config/src/main/resources/main.properties
new file mode 100644
index 0000000..00ae6c0
--- /dev/null
+++ b/src/it/jar-without-descriptor-config/src/main/resources/main.properties
@@ -0,0 +1,17 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements.  See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership.  The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License.  You may obtain a copy of the License at
+# 
+#   http://www.apache.org/licenses/LICENSE-2.0
+# 
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied.  See the License for the
+# specific language governing permissions and limitations
+# under the License.
+
diff --git a/src/it/jar-without-descriptor-config/src/test/java/MyTest.java b/src/it/jar-without-descriptor-config/src/test/java/MyTest.java
new file mode 100644
index 0000000..9d124be
--- /dev/null
+++ b/src/it/jar-without-descriptor-config/src/test/java/MyTest.java
@@ -0,0 +1,23 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ * 
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ * 
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+
+public class MyTest
+{
+
+}
diff --git a/src/it/jar-without-descriptor-config/src/test/resources/test.properties b/src/it/jar-without-descriptor-config/src/test/resources/test.properties
new file mode 100644
index 0000000..00ae6c0
--- /dev/null
+++ b/src/it/jar-without-descriptor-config/src/test/resources/test.properties
@@ -0,0 +1,17 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements.  See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership.  The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License.  You may obtain a copy of the License at
+# 
+#   http://www.apache.org/licenses/LICENSE-2.0
+# 
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied.  See the License for the
+# specific language governing permissions and limitations
+# under the License.
+
diff --git a/src/it/jar-without-descriptor-config/verify.bsh b/src/it/jar-without-descriptor-config/verify.bsh
new file mode 100644
index 0000000..2c2d028
--- /dev/null
+++ b/src/it/jar-without-descriptor-config/verify.bsh
@@ -0,0 +1,73 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ * 
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ * 
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+
+import java.io.*;
+import java.util.*;
+import java.util.jar.*;
+import java.util.regex.*;
+
+try
+{
+    File jarFile = new File( basedir, "target/jar-without-descriptor-config-1.0-SNAPSHOT-sources.jar" );
+    System.out.println( "Checking for existence of " + jarFile );
+    if ( !jarFile.isFile() )
+    {
+        System.out.println( "FAILURE!" );
+        return false;
+    }
+
+    JarFile jar = new JarFile( jarFile );
+
+    String[] includedEntries = {
+        "META-INF/MANIFEST.MF",
+        "MyClass.java",
+        "main.properties",
+    };
+    for ( String included : includedEntries )
+    {
+        System.out.println( "Checking for existence of " + included );
+        if ( jar.getEntry( included ) == null )
+        {
+            System.out.println( "FAILURE!" );
+            return false;
+        }
+    }
+
+    String[] excludedEntries = {
+        "MyTest.java",
+        "test.properties",
+    };
+    for ( String excluded : excludedEntries )
+    {
+        System.out.println( "Checking for absence of " + excluded );
+        if ( jar.getEntry( excluded ) != null )
+        {
+            System.out.println( "FAILURE!" );
+            return false;
+        }
+    }
+
+}
+catch( Throwable t )
+{
+    t.printStackTrace();
+    return false;
+}
+
+return true;
diff --git a/src/main/java/org/apache/maven/plugins/source/AbstractSourceJarMojo.java b/src/main/java/org/apache/maven/plugins/source/AbstractSourceJarMojo.java
index 772fcf1..d8af697 100644
--- a/src/main/java/org/apache/maven/plugins/source/AbstractSourceJarMojo.java
+++ b/src/main/java/org/apache/maven/plugins/source/AbstractSourceJarMojo.java
@@ -96,11 +96,11 @@
 
     /**
      * The archive configuration to use. See <a href="http://maven.apache.org/shared/maven-archiver/index.html">Maven
-     * Archiver Reference</a>.
-     * <br/>
-     * <b>Note: Since 3.0.0 the resulting archives contain a maven descriptor. If you need to suppress the
-     * generation of the maven descriptor you can simply achieve this by using the 
+     * Archiver Reference</a>. <br/>
+     * <b>Note: Since 3.0.0 the resulting archives contain a maven descriptor. If you need to suppress the generation of
+     * the maven descriptor you can simply achieve this by using the
      * <a href="http://maven.apache.org/shared/maven-archiver/index.html#archive">archiver configuration</a>.</b>.
+     * 
      * @since 2.1
      */
     @Parameter
diff --git a/src/main/java/org/apache/maven/plugins/source/SourceJarNoForkMojo.java b/src/main/java/org/apache/maven/plugins/source/SourceJarNoForkMojo.java
index b70aca0..895c3d3 100644
--- a/src/main/java/org/apache/maven/plugins/source/SourceJarNoForkMojo.java
+++ b/src/main/java/org/apache/maven/plugins/source/SourceJarNoForkMojo.java
@@ -19,19 +19,37 @@
  * under the License.
  */
 
+import java.util.Collections;
+import java.util.List;
+
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+
 import org.apache.maven.model.Resource;
 import org.apache.maven.plugins.annotations.LifecyclePhase;
 import org.apache.maven.plugins.annotations.Mojo;
 import org.apache.maven.plugins.annotations.Parameter;
 import org.apache.maven.project.MavenProject;
 
-import java.util.Collections;
-import java.util.List;
-
 /**
- * This goal bundles all the sources into a jar archive.  This goal functions the same
- * as the jar goal but does not fork the build and is suitable for attaching
- * to the build lifecycle.
+ * This goal bundles all the sources into a jar archive. This goal functions the same as the jar goal but does not fork
+ * the build and is suitable for attaching to the build lifecycle.
  *
  * @author pgier
  * @version $Id$
diff --git a/src/main/java/org/apache/maven/plugins/source/TestSourceJarNoForkMojo.java b/src/main/java/org/apache/maven/plugins/source/TestSourceJarNoForkMojo.java
index 5ca90a6..11eb699 100644
--- a/src/main/java/org/apache/maven/plugins/source/TestSourceJarNoForkMojo.java
+++ b/src/main/java/org/apache/maven/plugins/source/TestSourceJarNoForkMojo.java
@@ -19,15 +19,34 @@
  * under the License.
  */
 
+import java.util.Collections;
+import java.util.List;
+
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+
 import org.apache.maven.model.Resource;
 import org.apache.maven.plugins.annotations.LifecyclePhase;
 import org.apache.maven.plugins.annotations.Mojo;
 import org.apache.maven.plugins.annotations.Parameter;
 import org.apache.maven.project.MavenProject;
 
-import java.util.Collections;
-import java.util.List;
-
 /**
  * This goal bundles all the test sources into a jar archive.  This goal functions the same
  * as the test-jar goal but does not fork the build, and is suitable for attaching
diff --git a/src/test/java/org/apache/maven/plugins/source/AbstractSourcePluginTestCase.java b/src/test/java/org/apache/maven/plugins/source/AbstractSourcePluginTestCase.java
index b004d8c..e683b22 100644
--- a/src/test/java/org/apache/maven/plugins/source/AbstractSourcePluginTestCase.java
+++ b/src/test/java/org/apache/maven/plugins/source/AbstractSourcePluginTestCase.java
@@ -55,6 +55,9 @@
         File testPom = new File( getBasedir(), getTestDir( projectName ) + "/pom.xml" );
         AbstractSourceJarMojo mojo = (AbstractSourceJarMojo) lookupMojo( getGoal(), testPom );
 
+        //Without the following line the tests will fail, cause the project.getFile() will result with null.
+        mojo.project.setFile( testPom );
+
         setVariableValueToObject( mojo, "classifier", classifier );
 
         mojo.execute();
diff --git a/src/test/java/org/apache/maven/plugins/source/SourceJarMojoTest.java b/src/test/java/org/apache/maven/plugins/source/SourceJarMojoTest.java
index a11ab21..0fd57e1 100644
--- a/src/test/java/org/apache/maven/plugins/source/SourceJarMojoTest.java
+++ b/src/test/java/org/apache/maven/plugins/source/SourceJarMojoTest.java
@@ -35,19 +35,21 @@
     public void testDefaultConfiguration()
         throws Exception
     {
-        doTestProjectWithSourceArchive( "project-001",
-                                        new String[]{ "default-configuration.properties", "foo/project001/App.java",
-                                            "foo/project001/", "foo/", "META-INF/MANIFEST.MF", "META-INF/" },
-                                        "sources" );
+        doTestProjectWithSourceArchive( "project-001", new String[] { "default-configuration.properties",
+            "foo/project001/App.java", "foo/project001/", "foo/", "META-INF/MANIFEST.MF", "META-INF/",
+            "META-INF/maven/", "META-INF/maven/source/", "META-INF/maven/source/maven-source-plugin-test-project-001/",
+            "META-INF/maven/source/maven-source-plugin-test-project-001/pom.properties",
+            "META-INF/maven/source/maven-source-plugin-test-project-001/pom.xml" }, "sources" );
     }
 
     public void testExcludes()
         throws Exception
     {
-        doTestProjectWithSourceArchive( "project-003",
-                                        new String[]{ "default-configuration.properties", "foo/project003/App.java",
-                                            "foo/project003/", "foo/", "META-INF/MANIFEST.MF", "META-INF/" },
-                                        "sources" );
+        doTestProjectWithSourceArchive( "project-003", new String[] { "default-configuration.properties",
+            "foo/project003/App.java", "foo/project003/", "foo/", "META-INF/MANIFEST.MF", "META-INF/",
+            "META-INF/maven/", "META-INF/maven/source/", "META-INF/maven/source/maven-source-plugin-test-project-003/",
+            "META-INF/maven/source/maven-source-plugin-test-project-003/pom.properties",
+            "META-INF/maven/source/maven-source-plugin-test-project-003/pom.xml" }, "sources" );
     }
 
     public void testNoSources()
@@ -56,38 +58,39 @@
         executeMojo( "project-005", "sources" );
         // Now make sure that no archive got created
         final File expectedFile = getTestTargetDir( "project-005" );
-        assertFalse("Source archive should not have been created[" + expectedFile.getAbsolutePath() + "]",
-                expectedFile.exists());
+        assertFalse( "Source archive should not have been created[" + expectedFile.getAbsolutePath() + "]",
+                     expectedFile.exists() );
     }
 
     public void testIncludes()
         throws Exception
     {
-        doTestProjectWithSourceArchive("project-007", new String[]{"templates/configuration-template.properties",
-                "foo/project007/App.java", "templates/", "foo/project007/", "foo/", "META-INF/MANIFEST.MF", "META-INF/"
+        doTestProjectWithSourceArchive( "project-007", new String[] { "templates/configuration-template.properties",
+            "foo/project007/App.java", "templates/", "foo/project007/", "foo/", "META-INF/MANIFEST.MF", "META-INF/",
+            "META-INF/maven/", "META-INF/maven/source/", "META-INF/maven/source/maven-source-plugin-test-project-007/",
+            "META-INF/maven/source/maven-source-plugin-test-project-007/pom.properties",
+            "META-INF/maven/source/maven-source-plugin-test-project-007/pom.xml"
 
-        }, "sources");
+        }, "sources" );
     }
 
     public void testIncludePom()
         throws Exception
     {
-        doTestProjectWithSourceArchive( "project-009", new String[]{ "default-configuration.properties", "pom.xml",
-            "foo/project009/App.java", "foo/project009/", "foo/", "META-INF/MANIFEST.MF", "META-INF/" }, "sources" );
+        doTestProjectWithSourceArchive( "project-009", new String[] { "default-configuration.properties", "pom.xml",
+            "foo/project009/App.java", "foo/project009/", "foo/", "META-INF/MANIFEST.MF", "META-INF/",
+            "META-INF/maven/", "META-INF/maven/source/", "META-INF/maven/source/maven-source-plugin-test-project-009/",
+            "META-INF/maven/source/maven-source-plugin-test-project-009/pom.properties",
+            "META-INF/maven/source/maven-source-plugin-test-project-009/pom.xml" }, "sources" );
     }
 
     public void testIncludeMavenDescriptorWhenExplicitlyConfigured()
-            throws Exception {
-        doTestProjectWithSourceArchive("project-010",
-                new String[]{
-                        "default-configuration.properties", "foo/project010/App.java",
-                        "foo/project010/", "foo/", "META-INF/MANIFEST.MF", "META-INF/",
-                        "META-INF/maven/", "META-INF/maven/source/",
-                        "META-INF/maven/source/maven-source-plugin-test-project-010/",
-                        "META-INF/maven/source/maven-source-plugin-test-project-010/pom.xml",
-                        "META-INF/maven/source/maven-source-plugin-test-project-010/pom" +
-                                ".properties"
-                },
-                "sources");
+        throws Exception
+    {
+        doTestProjectWithSourceArchive( "project-010", new String[] { "default-configuration.properties",
+            "foo/project010/App.java", "foo/project010/", "foo/", "META-INF/MANIFEST.MF", "META-INF/",
+            "META-INF/maven/", "META-INF/maven/source/", "META-INF/maven/source/maven-source-plugin-test-project-010/",
+            "META-INF/maven/source/maven-source-plugin-test-project-010/pom.xml",
+            "META-INF/maven/source/maven-source-plugin-test-project-010/pom" + ".properties" }, "sources" );
     }
 }
diff --git a/src/test/java/org/apache/maven/plugins/source/TestSourceJarMojoTest.java b/src/test/java/org/apache/maven/plugins/source/TestSourceJarMojoTest.java
index 0520c62..6014952 100644
--- a/src/test/java/org/apache/maven/plugins/source/TestSourceJarMojoTest.java
+++ b/src/test/java/org/apache/maven/plugins/source/TestSourceJarMojoTest.java
@@ -33,25 +33,26 @@
         return "test-jar";
     }
 
-
     public void testDefaultConfiguration()
         throws Exception
     {
-        doTestProjectWithTestSourceArchive( "project-001", new String[]{ "test-default-configuration.properties",
-            "foo/project001/AppTest.java", "foo/project001/", "foo/", "META-INF/MANIFEST.MF", "META-INF/" },
-                                            "test-sources" );
+        doTestProjectWithTestSourceArchive( "project-001", new String[] { "test-default-configuration.properties",
+            "foo/project001/AppTest.java", "foo/project001/", "foo/", "META-INF/MANIFEST.MF", "META-INF/",
+            "META-INF/maven/", "META-INF/maven/source/", "META-INF/maven/source/maven-source-plugin-test-project-001/",
+            "META-INF/maven/source/maven-source-plugin-test-project-001/pom.properties",
+            "META-INF/maven/source/maven-source-plugin-test-project-001/pom.xml" }, "test-sources" );
     }
 
-
     public void testExcludes()
         throws Exception
     {
-        doTestProjectWithTestSourceArchive( "project-003", new String[]{ "test-default-configuration.properties",
-            "foo/project003/AppTest.java", "foo/project003/", "foo/", "META-INF/MANIFEST.MF", "META-INF/" },
-                                            "test-sources" );
+        doTestProjectWithTestSourceArchive( "project-003", new String[] { "test-default-configuration.properties",
+            "foo/project003/AppTest.java", "foo/project003/", "foo/", "META-INF/MANIFEST.MF", "META-INF/",
+            "META-INF/maven/", "META-INF/maven/source/", "META-INF/maven/source/maven-source-plugin-test-project-003/",
+            "META-INF/maven/source/maven-source-plugin-test-project-003/pom.properties",
+            "META-INF/maven/source/maven-source-plugin-test-project-003/pom.xml" }, "test-sources" );
     }
 
-
     public void testNoSources()
         throws Exception
     {
@@ -59,23 +60,18 @@
 
         // Now make sure that no archive got created
         final File expectedFile = getTestTargetDir( "project-005" );
-        assertFalse("Test source archive should not have been created[" + expectedFile.getAbsolutePath() + "]",
-                expectedFile.exists());
+        assertFalse( "Test source archive should not have been created[" + expectedFile.getAbsolutePath() + "]",
+                     expectedFile.exists() );
     }
 
     public void testIncludeMavenDescriptorWhenExplicitlyConfigured()
-            throws Exception {
-        doTestProjectWithSourceArchive("project-010",
-                new String[]{
-                        "default-configuration.properties", "foo/project010/App.java",
-                        "foo/project010/", "foo/", "META-INF/MANIFEST.MF", "META-INF/",
-                        "META-INF/maven/", "META-INF/maven/source/",
-                        "META-INF/maven/source/maven-source-plugin-test-project-010/",
-                        "META-INF/maven/source/maven-source-plugin-test-project-010/pom.xml",
-                        "META-INF/maven/source/maven-source-plugin-test-project-010/pom" +
-                                ".properties"
-                },
-                "test-sources");
+        throws Exception
+    {
+        doTestProjectWithSourceArchive( "project-010", new String[] { "default-configuration.properties",
+            "foo/project010/App.java", "foo/project010/", "foo/", "META-INF/MANIFEST.MF", "META-INF/",
+            "META-INF/maven/", "META-INF/maven/source/", "META-INF/maven/source/maven-source-plugin-test-project-010/",
+            "META-INF/maven/source/maven-source-plugin-test-project-010/pom.xml",
+            "META-INF/maven/source/maven-source-plugin-test-project-010/pom" + ".properties" }, "test-sources" );
     }
 
 }