SQOOP-2109: Sqoop2: Shell module is including test dependencies when building binary artifact

(Jarek Jarcec Cecho via Abraham Elmahrek)

common/pom.xml

@@ -35,6 +35,7 @@ limitations under the License.
     <dependency>
       <groupId>org.apache.sqoop</groupId>
       <artifactId>sqoop-common-test</artifactId>
+      <scope>test</scope>
     </dependency>
 
     <dependency>

pom.xml

@@ -123,13 +123,21 @@ limitations under the License.
     <jackson.databind.version>2.2.2</jackson.databind.version>
     <jackson.annotations.version>2.2.2</jackson.annotations.version>
     <avro.version>1.7.7</avro.version>
+    <jcommander.version>1.27</jcommander.version>
   </properties>
 
   <dependencies>
+
     <dependency>
       <groupId>log4j</groupId>
       <artifactId>log4j</artifactId>
     </dependency>
+    <dependency>
+      <groupId>org.testng</groupId>
+      <artifactId>testng</artifactId>
+      <scope>test</scope>
+    </dependency>
+
   </dependencies>
 
   <!-- Profiles for various supported Hadoop distributions -->
@@ -468,6 +476,11 @@ limitations under the License.
         <artifactId>postgresql</artifactId>
         <version>${jdbc.postgresql.version}</version>
       </dependency>
+      <dependency>
+        <groupId>com.beust</groupId>
+        <artifactId>jcommander</artifactId>
+        <version>${jcommander.version}</version>
+      </dependency>
       <dependency>
         <groupId>com.oracle</groupId>
         <artifactId>ojdbc14</artifactId>

security/pom.xml

@@ -39,5 +39,11 @@ limitations under the License.
       <artifactId>hadoop-common</artifactId>
       <scope>provided</scope>
     </dependency>
+
+    <dependency>
+      <groupId>com.beust</groupId>
+      <artifactId>jcommander</artifactId>
+    </dependency>
+
   </dependencies>
 </project>

shell/pom.xml

@@ -59,6 +59,12 @@ limitations under the License.
       <groupId>jline</groupId>
       <artifactId>jline</artifactId>
       <version>0.9.94</version>
+      <exclusions>
+        <exclusion>
+          <groupId>junit</groupId>
+          <artifactId>junit</artifactId>
+        </exclusion>
+      </exclusions>
     </dependency>
     <dependency>
       <groupId>org.fusesource.jansi</groupId>
@@ -90,6 +96,8 @@ limitations under the License.
                   <goal>copy-dependencies</goal>
                 </goals>
                 <configuration>
+                  <includeScope>runtime</includeScope>
+                  <excludeScope>test</excludeScope>
                   <outputDirectory>${project.build.directory}/lib</outputDirectory>
                 </configuration>
               </execution>