fix: revert test category refactoring (#1419)

Reverts moving the category interfaces of tests to their own package.
This is because we export the tests as a jar to maven and it could break
application code.

Author: thiagotnunes

google-cloud-spanner/pom.xml

@@ -50,7 +50,7 @@
             <!-- Executes unit tests -->
             <id>default-test</id>
             <configuration>
-              <excludedGroups>com.google.cloud.spanner.categories.TracerTest,com.google.cloud.spanner.categories.IntegrationTest</excludedGroups>
+              <excludedGroups>com.google.cloud.spanner.TracerTest,com.google.cloud.spanner.IntegrationTest</excludedGroups>
             </configuration>
           </execution>
           <execution>
@@ -59,7 +59,7 @@
               <goal>test</goal>
             </goals>
             <configuration>
-              <groups>com.google.cloud.spanner.categories.TracerTest</groups>
+              <groups>com.google.cloud.spanner.TracerTest</groups>
             </configuration>
           </execution>
         </executions>
@@ -81,7 +81,7 @@
           <execution>
             <id>default</id>
             <configuration>
-              <groups>com.google.cloud.spanner.categories.SerialIntegrationTest</groups>
+              <groups>com.google.cloud.spanner.SerialIntegrationTest</groups>
             </configuration>
           </execution>
           <!-- Executes parallel integration tests -->
@@ -91,7 +91,7 @@
               <goal>integration-test</goal>
             </goals>
             <configuration>
-              <groups>com.google.cloud.spanner.categories.ParallelIntegrationTest</groups>
+              <groups>com.google.cloud.spanner.ParallelIntegrationTest</groups>
               <forkCount>8</forkCount>
               <reuseForks>true</reuseForks>
             </configuration>
@@ -377,7 +377,7 @@
                 <execution>
                   <id>default-test</id>
                   <configuration>
-                    <groups>com.google.cloud.spanner.categories.SlowTest</groups>
+                    <groups>com.google.cloud.spanner.SlowTest</groups>
                   </configuration>
                 </execution>
               </executions>
@@ -389,7 +389,7 @@
                 <execution>
                   <id>default</id>
                   <configuration>
-                    <groups>com.google.cloud.spanner.categories.SlowTest</groups>
+                    <groups>com.google.cloud.spanner.SlowTest</groups>
                   </configuration>
                 </execution>
                 <!-- Overrides default configuration to skip this step -->

google-cloud-spanner/src/test/java/com/google/cloud/spanner/FlakyTest.java

@@ -0,0 +1,25 @@
+/*
+ * Copyright 2017 Google LLC
+ *
+ * Licensed 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.
+ */
+
+package com.google.cloud.spanner;
+
+/**
+ * Annotation for JUnit {@link org.junit.experimental.categories.Category} that indicates a test is
+ * flaky. These will be excluded from integration tests. Use this annotation sparingly: typically it
+ * should only be used for a test where the flakiness is dependent on a fix in a module dependency
+ * (for example, grpc-java) and cannot be addressed locally.
+ */
+public interface FlakyTest {}

google-cloud-spanner/src/test/java/com/google/cloud/spanner/ITSessionPoolIntegrationTest.java

@@ -20,7 +20,6 @@
 
 import com.google.cloud.grpc.GrpcTransportOptions.ExecutorFactory;
 import com.google.cloud.spanner.SessionPool.PooledSessionFuture;
-import com.google.cloud.spanner.categories.SerialIntegrationTest;
 import java.util.ArrayList;
 import java.util.List;
 import java.util.concurrent.CountDownLatch;

…/spanner/categories/IntegrationTest.java …oogle/cloud/spanner/IntegrationTest.javagoogle-cloud-spanner/src/test/java/com/google/cloud/spanner/categories/IntegrationTest.java renamed to google-cloud-spanner/src/test/java/com/google/cloud/spanner/IntegrationTest.java google-cloud-spanner/src/test/java/com/google/cloud/spanner/categories/IntegrationTest.java renamed to google-cloud-spanner/src/test/java/com/google/cloud/spanner/IntegrationTest.java

@@ -14,7 +14,7 @@
  * limitations under the License.
  */
 
-package com.google.cloud.spanner.categories;
+package com.google.cloud.spanner;
 
 /** Integration Test interface. */
 public interface IntegrationTest {}

…/categories/ParallelIntegrationTest.java …oud/spanner/ParallelIntegrationTest.javagoogle-cloud-spanner/src/test/java/com/google/cloud/spanner/categories/ParallelIntegrationTest.java renamed to google-cloud-spanner/src/test/java/com/google/cloud/spanner/ParallelIntegrationTest.java google-cloud-spanner/src/test/java/com/google/cloud/spanner/categories/ParallelIntegrationTest.java renamed to google-cloud-spanner/src/test/java/com/google/cloud/spanner/ParallelIntegrationTest.java

@@ -14,7 +14,7 @@
  * limitations under the License.
  */
 
-package com.google.cloud.spanner.categories;
+package com.google.cloud.spanner;
 
 /** Parallel Integration Test interface. */
 public interface ParallelIntegrationTest extends IntegrationTest {}

…er/categories/SerialIntegrationTest.java …cloud/spanner/SerialIntegrationTest.javagoogle-cloud-spanner/src/test/java/com/google/cloud/spanner/categories/SerialIntegrationTest.java renamed to google-cloud-spanner/src/test/java/com/google/cloud/spanner/SerialIntegrationTest.java google-cloud-spanner/src/test/java/com/google/cloud/spanner/categories/SerialIntegrationTest.java renamed to google-cloud-spanner/src/test/java/com/google/cloud/spanner/SerialIntegrationTest.java

@@ -14,7 +14,7 @@
  * limitations under the License.
  */
 
-package com.google.cloud.spanner.categories;
+package com.google.cloud.spanner;
 
 /** Serial Integration Test interface. */
 public interface SerialIntegrationTest {}

…e/cloud/spanner/categories/SlowTest.java …a/com/google/cloud/spanner/SlowTest.javagoogle-cloud-spanner/src/test/java/com/google/cloud/spanner/categories/SlowTest.java renamed to google-cloud-spanner/src/test/java/com/google/cloud/spanner/SlowTest.java google-cloud-spanner/src/test/java/com/google/cloud/spanner/categories/SlowTest.java renamed to google-cloud-spanner/src/test/java/com/google/cloud/spanner/SlowTest.java

@@ -14,7 +14,7 @@
  * limitations under the License.
  */
 
-package com.google.cloud.spanner.categories;
+package com.google.cloud.spanner;
 
 /** Category of slow tests, to be run on the nightly build * */
 public interface SlowTest extends IntegrationTest {}

google-cloud-spanner/src/test/java/com/google/cloud/spanner/SpanTest.java

@@ -25,7 +25,6 @@
 import com.google.cloud.NoCredentials;
 import com.google.cloud.spanner.MockSpannerServiceImpl.SimulatedExecutionTime;
 import com.google.cloud.spanner.MockSpannerServiceImpl.StatementResult;
-import com.google.cloud.spanner.categories.TracerTest;
 import com.google.protobuf.ListValue;
 import com.google.spanner.v1.ResultSetMetadata;
 import com.google.spanner.v1.StructType;

…cloud/spanner/categories/TracerTest.java …com/google/cloud/spanner/TracerTest.javagoogle-cloud-spanner/src/test/java/com/google/cloud/spanner/categories/TracerTest.java renamed to google-cloud-spanner/src/test/java/com/google/cloud/spanner/TracerTest.java google-cloud-spanner/src/test/java/com/google/cloud/spanner/categories/TracerTest.java renamed to google-cloud-spanner/src/test/java/com/google/cloud/spanner/TracerTest.java

@@ -14,7 +14,7 @@
  * limitations under the License.
  */
 
-package com.google.cloud.spanner.categories;
+package com.google.cloud.spanner;
 
 /**
  * Tests marked with this {@link org.junit.experimental.categories.Category} will be executed in a

google-cloud-spanner/src/test/java/com/google/cloud/spanner/connection/it/ITAsyncTransactionRetryTest.java

@@ -33,11 +33,11 @@
 import com.google.cloud.spanner.KeySet;
 import com.google.cloud.spanner.Mutation;
 import com.google.cloud.spanner.Options;
+import com.google.cloud.spanner.ParallelIntegrationTest;
 import com.google.cloud.spanner.ResultSet;
 import com.google.cloud.spanner.SpannerExceptionFactory;
 import com.google.cloud.spanner.Statement;
 import com.google.cloud.spanner.Struct;
-import com.google.cloud.spanner.categories.ParallelIntegrationTest;
 import com.google.cloud.spanner.connection.Connection;
 import com.google.cloud.spanner.connection.ITAbstractSpannerTest;
 import com.google.cloud.spanner.connection.TransactionRetryListener;