[tests] Scope the dependencies for spark and storm tests to test scope (

apache#4556)

*Motivation*

In gitsha b7a5677 , it introduced a provided dependency for pulsar-storm-test, which it will fail release process. Since it will attempt to transfer the pom file for  `com.twitter:carbonite:pom:1.5.0` which is a dependency comes from storm-server.

*Changes*

Make all the dependencies scoped at test scope for two test modules. So maven release will not attempt to release them.

tests/pulsar-spark-test/pom.xml

@@ -36,16 +36,18 @@
     <dependencies>
 
         <dependency>
-            <groupId>org.apache.pulsar</groupId>
-            <artifactId>pulsar-spark</artifactId>
-            <version>${project.version}</version>
+          <groupId>org.apache.pulsar</groupId>
+          <artifactId>pulsar-spark</artifactId>
+          <version>${project.version}</version>
+          <scope>test</scope>
         </dependency>
 
         <dependency>
           <groupId>org.apache.pulsar.tests</groupId>
           <artifactId>integration</artifactId>
           <version>${project.version}</version>
           <type>test-jar</type>
+          <scope>test</scope>
         </dependency>
 
         <dependency>
@@ -55,13 +57,15 @@
         </dependency>
 
         <dependency>
-            <groupId>org.mockito</groupId>
-            <artifactId>mockito-core</artifactId>
+          <groupId>org.mockito</groupId>
+          <artifactId>mockito-core</artifactId>
+          <scope>test</scope>
         </dependency>
 
         <dependency>
-            <groupId>org.apache.spark</groupId>
-            <artifactId>spark-streaming_2.10</artifactId>
+          <groupId>org.apache.spark</groupId>
+          <artifactId>spark-streaming_2.10</artifactId>
+          <scope>test</scope>
         </dependency>
 
     </dependencies>

tests/pulsar-storm-test/pom.xml

@@ -39,6 +39,7 @@
       <groupId>org.apache.pulsar</groupId>
       <artifactId>pulsar-storm</artifactId>
       <version>${project.version}</version>
+      <scope>test</scope>
       <exclusions>
         <exclusion>
           <groupId>org.apache.pulsar</groupId>
@@ -51,7 +52,7 @@
       <groupId>org.apache.storm</groupId>
       <artifactId>storm-server</artifactId>
       <version>${storm.version}</version>
-      <scope>provided</scope>
+      <scope>test</scope>
       <exclusions>
         <exclusion>
             <groupId>ch.qos.logback</groupId>
@@ -68,20 +69,23 @@
       <groupId>org.apache.pulsar</groupId>
       <artifactId>pulsar-broker</artifactId>
       <version>${project.version}</version>
+      <scope>test</scope>
     </dependency>
 
     <dependency>
       <groupId>org.apache.pulsar</groupId>
       <artifactId>pulsar-broker</artifactId>
       <version>${project.version}</version>
       <type>test-jar</type>
+      <scope>test</scope>
     </dependency>
 
     <dependency>
       <groupId>org.apache.pulsar</groupId>
       <artifactId>managed-ledger-original</artifactId>
       <version>${project.version}</version>
       <type>test-jar</type>
+      <scope>test</scope>
     </dependency>
 
     <dependency>
@@ -92,12 +96,14 @@
     <dependency>
       <groupId>org.asynchttpclient</groupId>
       <artifactId>async-http-client</artifactId>
+      <scope>test</scope>
     </dependency>
 
     <dependency>
       <groupId>org.apache.storm</groupId>
       <artifactId>storm-core</artifactId>
       <version>${storm.version}</version>
+      <scope>test</scope>
       <exclusions>
         <exclusion>
           <groupId>ch.qos.logback</groupId>