Consistently rename tests to unit tests/integration test

Author: mp911de

pom.xml

@@ -339,7 +339,6 @@
                     </excludes>
                     <includes>
                         <include>**/*Tests.java</include>
-                        <include>**/*Test.java</include>
                     </includes>
                     <systemPropertyVariables>
                         <io.netty.leakDetectionLevel>paranoid</io.netty.leakDetectionLevel>

src/test/java/io/r2dbc/postgresql/AbstractCodecIntegrationTests.java

@@ -65,6 +65,12 @@
 import static io.r2dbc.postgresql.util.TestByteBufAllocator.TEST;
 import static org.assertj.core.api.Assertions.assertThat;
 
+/**
+ * Integrations tests for our built-in codecs. Executed typicall in text and binary mode.
+ *
+ * @see CodecBinaryFormatIntegrationTests
+ * @see CodecTextFormatIntegrationTests
+ */
 abstract class AbstractCodecIntegrationTests extends AbstractIntegrationTests {
 
     @Override

src/test/java/io/r2dbc/postgresql/BoundedStatementCacheTest.java src/test/java/io/r2dbc/postgresql/BoundedStatementCacheUnitTests.java

@@ -41,7 +41,10 @@
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.assertj.core.api.Assertions.assertThatIllegalArgumentException;
 
-final class BoundedStatementCacheTest {
+/**
+ * Unit tests for {@link BoundedStatementCache}.
+ */
+final class BoundedStatementCacheUnitTests {
 
     @Test
     void constructorInvalidLimit() {

src/test/java/io/r2dbc/postgresql/CodecBinaryFormatIntegrationTests.java

@@ -16,6 +16,11 @@
 
 package io.r2dbc.postgresql;
 
+import io.r2dbc.postgresql.codec.DefaultCodecs;
+
+/**
+ * Integration tests for {@link DefaultCodecs} executed using binary encoding.
+ */
 final class CodecBinaryFormatIntegrationTests extends AbstractCodecIntegrationTests {
 
     @Override

src/test/java/io/r2dbc/postgresql/CodecExtensionIntegrationTests.java

@@ -35,7 +35,10 @@
 
 import static org.assertj.core.api.Assertions.assertThat;
 
-public class CodecExtensionIntegrationTests extends AbstractIntegrationTests {
+/**
+ * Integration tests for {@link CodecRegistrar}.
+ */
+final class CodecExtensionIntegrationTests extends AbstractIntegrationTests {
 
     @BeforeEach
     void setUp() {

src/test/java/io/r2dbc/postgresql/CodecTextFormatIntegrationTests.java

@@ -16,6 +16,9 @@
 
 package io.r2dbc.postgresql;
 
+/**
+ * Integration tests for {@link DefaultCodecs} executed using text encoding.
+ */
 final class CodecTextFormatIntegrationTests extends AbstractCodecIntegrationTests {
 
 }

src/test/java/io/r2dbc/postgresql/DefaultPortalNameSupplierTest.java src/test/java/io/r2dbc/postgresql/DefaultPortalNameSupplierUnitTests.java

@@ -20,7 +20,10 @@
 
 import static org.assertj.core.api.Assertions.assertThat;
 
-final class DefaultPortalNameSupplierTest {
+/**
+ * Unit tests for {@link DefaultPortalNameSupplier}.
+ */
+final class DefaultPortalNameSupplierUnitTests {
 
     @Test
     void test() {

src/test/java/io/r2dbc/postgresql/DisabledStatementCacheTest.java src/test/java/io/r2dbc/postgresql/DisabledStatementCacheUnitTests.java

@@ -36,7 +36,10 @@
 import static io.r2dbc.postgresql.util.TestByteBufAllocator.TEST;
 import static org.assertj.core.api.Assertions.assertThatIllegalArgumentException;
 
-class DisabledStatementCacheTest {
+/**
+ * Unit tests for {@link DisabledStatementCache}.
+ */
+final class DisabledStatementCacheUnitTests {
 
     @Test
     void constructorNoClient() {

src/test/java/io/r2dbc/postgresql/ExceptionFactoryTest.java src/test/java/io/r2dbc/postgresql/ExceptionFactoryUnitTests.java

@@ -35,7 +35,10 @@
 import static org.mockito.Mockito.times;
 import static org.mockito.Mockito.verify;
 
-final class ExceptionFactoryTest {
+/**
+ * Unit tests for {@link ExceptionFactory}.
+ */
+final class ExceptionFactoryUnitTests {
 
     @Test
     void isCreatingAuthenticationException() {

src/test/java/io/r2dbc/postgresql/ExtendedQueryPostgresqlStatementTest.java src/test/java/io/r2dbc/postgresql/ExtendedQueryPostgresqlStatementUnitTests.java

@@ -56,7 +56,10 @@
 import static org.mockito.ArgumentMatchers.any;
 import static org.mockito.Mockito.when;
 
-final class ExtendedQueryPostgresqlStatementTest {
+/**
+ * Unit tests for {@link ExtendedQueryPostgresqlStatement}.
+ */
+final class ExtendedQueryPostgresqlStatementUnitTests {
 
     private final Parameter parameter = new Parameter(FORMAT_BINARY, INT4.getObjectId(), Flux.just(TEST.buffer(4).writeInt(100)));
 

src/test/java/io/r2dbc/postgresql/FetchSizeIntegrationTests.java src/test/java/io/r2dbc/postgresql/FetchSizeIntegrationIntegrationTests.java

@@ -23,7 +23,7 @@
 /**
  * Integration tests for {@link Statement#fetchSize(int)}.
  */
-final class FetchSizeIntegrationTests extends AbstractIntegrationTests {
+final class FetchSizeIntegrationIntegrationTests extends AbstractIntegrationTests {
 
     @Override
     protected void customize(PostgresqlConnectionConfiguration.Builder builder) {

src/test/java/io/r2dbc/postgresql/IndefiniteStatementCacheTest.java src/test/java/io/r2dbc/postgresql/IndefiniteStatementCacheUnitTests.java

@@ -36,7 +36,10 @@
 import static io.r2dbc.postgresql.util.TestByteBufAllocator.TEST;
 import static org.assertj.core.api.Assertions.assertThatIllegalArgumentException;
 
-final class IndefiniteStatementCacheTest {
+/**
+ * Unit tests for {@link IndefiniteStatementCache}.
+ */
+final class IndefiniteStatementCacheUnitTests {
 
     @Test
     void constructorNoClient() {

src/test/java/io/r2dbc/postgresql/JsonIntegrationTests.java

@@ -25,7 +25,10 @@
 import org.springframework.jdbc.core.JdbcOperations;
 import reactor.test.StepVerifier;
 
-class JsonIntegrationTests {
+/**
+ * Integration tests for {@link Json}.
+ */
+final class JsonIntegrationTests {
 
     @RegisterExtension
     static final PostgresqlServerExtension SERVER = new PostgresqlServerExtension();

src/test/java/io/r2dbc/postgresql/client/PgBouncerIntegrationTests.java src/test/java/io/r2dbc/postgresql/PgBouncerIntegrationTests.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2019-2020 the original author or authors.
+ * Copyright 2020 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -14,7 +14,7 @@
  * limitations under the License.
  */
 
-package io.r2dbc.postgresql.client;
+package io.r2dbc.postgresql;
 
 import io.r2dbc.postgresql.PostgresqlConnectionConfiguration;
 import io.r2dbc.postgresql.PostgresqlConnectionFactory;
@@ -29,6 +29,9 @@
 import reactor.core.publisher.Flux;
 import reactor.test.StepVerifier;
 
+/**
+ * Integration tests for {@link PostgresqlConnectionFactory} through PG Bouncer.
+ */
 final class PgBouncerIntegrationTests {
 
     @RegisterExtension

src/test/java/io/r2dbc/postgresql/PostgresCancelIntegrationTests.java

@@ -34,7 +34,7 @@
  * Integration tests for cancellation.
  */
 @TestInstance(TestInstance.Lifecycle.PER_CLASS)
-class PostgresCancelIntegrationTests extends AbstractIntegrationTests {
+final class PostgresCancelIntegrationTests extends AbstractIntegrationTests {
 
     static final int NUMBER_REPETITIONS = 300;
 

src/test/java/io/r2dbc/postgresql/PostgresNotificationIntegrationTests.java

@@ -32,6 +32,9 @@
 
 import static org.assertj.core.api.Assertions.assertThat;
 
+/**
+ * Integration tests for {@link Notification} through {@link PostgresqlConnection#getNotifications()}.
+ */
 final class PostgresNotificationIntegrationTests extends AbstractIntegrationTests {
 
     @Test

src/test/java/io/r2dbc/postgresql/PostgresqlAuthenticationFailureIntegrationTests.java

@@ -34,6 +34,9 @@
 import static io.r2dbc.spi.ConnectionFactoryOptions.USER;
 import static org.assertj.core.api.Assertions.assertThatExceptionOfType;
 
+/**
+ * Integration tests for authentication failures.
+ */
 final class PostgresqlAuthenticationFailureIntegrationTests {
 
     @RegisterExtension

src/test/java/io/r2dbc/postgresql/PostgresqlBatchTest.java src/test/java/io/r2dbc/postgresql/PostgresqlBatchUnitTests.java

@@ -22,6 +22,7 @@
 import io.r2dbc.postgresql.message.backend.EmptyQueryResponse;
 import io.r2dbc.postgresql.message.backend.ErrorResponse;
 import io.r2dbc.postgresql.message.frontend.Query;
+import io.r2dbc.spi.Batch;
 import io.r2dbc.spi.R2dbcNonTransientResourceException;
 import org.junit.jupiter.api.Test;
 import reactor.test.StepVerifier;
@@ -30,7 +31,10 @@
 
 import static org.assertj.core.api.Assertions.assertThatIllegalArgumentException;
 
-final class PostgresqlBatchTest {
+/**
+ * Unit tests for {@link Batch}.
+ */
+final class PostgresqlBatchUnitTests {
 
     @Test
     void add() {

src/test/java/io/r2dbc/postgresql/PostgresqlColumnMetadataTest.java src/test/java/io/r2dbc/postgresql/PostgresqlColumnMetadataUnitTests.java

@@ -23,7 +23,10 @@
 import static io.r2dbc.postgresql.message.Format.FORMAT_TEXT;
 import static org.assertj.core.api.Assertions.assertThat;
 
-final class PostgresqlColumnMetadataTest {
+/**
+ * Unit tests for {@link PostgresqlColumnMetadata}.
+ */
+final class PostgresqlColumnMetadataUnitTests {
 
     @Test
     void toColumnMetadata() {

src/test/java/io/r2dbc/postgresql/PostgresqlConnectionConfigurationTest.java src/test/java/io/r2dbc/postgresql/PostgresqlConnectionConfigurationUnitTests.java

@@ -26,7 +26,10 @@
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.assertj.core.api.Assertions.assertThatIllegalArgumentException;
 
-final class PostgresqlConnectionConfigurationTest {
+/**
+ * Unit tests for {@link PostgresqlConnectionConfiguration}.
+ */
+final class PostgresqlConnectionConfigurationUnitTests {
 
     @Test
     void builderNoApplicationName() {

src/test/java/io/r2dbc/postgresql/PostgresqlConnectionFactoryMetadataTest.java src/test/java/io/r2dbc/postgresql/PostgresqlConnectionFactoryMetadataUnitTests.java

@@ -20,7 +20,10 @@
 
 import static org.assertj.core.api.Assertions.assertThat;
 
-final class PostgresqlConnectionFactoryMetadataTest {
+/**
+ * Unit tests for {@link PostgresqlConnectionFactoryMetadata}.
+ */
+final class PostgresqlConnectionFactoryMetadataUnitTests {
 
     @Test
     void name() {

src/test/java/io/r2dbc/postgresql/PostgresqlConnectionFactoryProviderTest.java src/test/java/io/r2dbc/postgresql/PostgresqlConnectionFactoryProviderUnitTests.java

@@ -47,7 +47,10 @@
 import static org.assertj.core.api.Assertions.assertThatIllegalStateException;
 import static org.assertj.core.api.Assertions.assertThatThrownBy;
 
-final class PostgresqlConnectionFactoryProviderTest {
+/**
+ * Unit tests for {@link PostgresqlConnectionFactoryProvider}.
+ */
+final class PostgresqlConnectionFactoryProviderUnitTests {
 
     private final PostgresqlConnectionFactoryProvider provider = new PostgresqlConnectionFactoryProvider();
 

src/test/java/io/r2dbc/postgresql/PostgresqlConnectionFactoryTest.java src/test/java/io/r2dbc/postgresql/PostgresqlConnectionFactoryUnitTests.java

@@ -40,7 +40,10 @@
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.assertj.core.api.Assertions.assertThatIllegalArgumentException;
 
-final class PostgresqlConnectionFactoryTest {
+/**
+ * Unit tests for {@link PostgresqlConnectionFactory}.
+ */
+final class PostgresqlConnectionFactoryUnitTests {
 
     @Test
     void constructorNoClientFactory() {

src/test/java/io/r2dbc/postgresql/PostgresqlConnectionRuntimeOptionsIntegrationTests.java

@@ -8,6 +8,9 @@
 import java.util.HashMap;
 import java.util.Map;
 
+/**
+ * Integration tests for {@link PostgresqlConnection} connection options.
+ */
 final class PostgresqlConnectionRuntimeOptionsIntegrationTests {
 
     @RegisterExtension

src/test/java/io/r2dbc/postgresql/PostgresqlConnectionTest.java src/test/java/io/r2dbc/postgresql/PostgresqlConnectionUnitTests.java

@@ -41,7 +41,10 @@
 import static org.mockito.Mockito.RETURNS_SMART_NULLS;
 import static org.mockito.Mockito.mock;
 
-final class PostgresqlConnectionTest {
+/**
+ * Unit tests for {@link PostgresqlConnection}.
+ */
+final class PostgresqlConnectionUnitTests {
 
     private final StatementCache statementCache = mock(StatementCache.class, RETURNS_SMART_NULLS);
 

src/test/java/io/r2dbc/postgresql/PostgresqlResultTest.java src/test/java/io/r2dbc/postgresql/PostgresqlResultUnitTests.java

@@ -26,7 +26,10 @@
 
 import java.util.Collections;
 
-final class PostgresqlResultTest {
+/**
+ * Unit tests for {@link PostgresqlResult}.
+ */
+final class PostgresqlResultUnitTests {
 
     @Test
     void toResultCommandComplete() {

src/test/java/io/r2dbc/postgresql/PostgresqlRowMetadataTest.java src/test/java/io/r2dbc/postgresql/PostgresqlRowMetadataUnitTests.java

@@ -31,7 +31,10 @@
 import static org.assertj.core.api.Assertions.assertThatExceptionOfType;
 import static org.assertj.core.api.Assertions.assertThatIllegalArgumentException;
 
-final class PostgresqlRowMetadataTest {
+/**
+ * Unit tests for {@link PostgresqlRowMetadata}.
+ */
+final class PostgresqlRowMetadataUnitTests {
 
     private final List<PostgresqlColumnMetadata> columnMetadatas = Arrays.asList(
         new PostgresqlColumnMetadata(MockCodecs.empty(), "test-name-2", 400, (short) 300),

src/test/java/io/r2dbc/postgresql/PostgresqlRowTest.java src/test/java/io/r2dbc/postgresql/PostgresqlRowUnitTests.java

@@ -33,7 +33,10 @@
 import static org.assertj.core.api.Assertions.assertThatIllegalArgumentException;
 import static org.assertj.core.api.Assertions.assertThatIllegalStateException;
 
-final class PostgresqlRowTest {
+/**
+ * Unit tests for {@link PostgresqlRow}.
+ */
+final class PostgresqlRowUnitTests {
 
     private final List<RowDescription.Field> columns = Arrays.asList(
         new RowDescription.Field((short) 100, 200, 300, (short) 400, FORMAT_BINARY, "test-name-1", 500),

src/test/java/io/r2dbc/postgresql/PostgresqlStatementErrorsIntegrationTests.java

@@ -16,6 +16,7 @@
 
 package io.r2dbc.postgresql;
 
+import io.r2dbc.postgresql.api.PostgresqlStatement;
 import io.r2dbc.spi.R2dbcDataIntegrityViolationException;
 import io.r2dbc.spi.Result;
 import org.junit.jupiter.api.AfterEach;
@@ -24,6 +25,9 @@
 import reactor.core.publisher.Flux;
 import reactor.test.StepVerifier;
 
+/**
+ * Integration tests for various error cases using {@link PostgresqlStatement}.
+ */
 final class PostgresqlStatementErrorsIntegrationTests extends AbstractIntegrationTests {
 
     @BeforeEach

src/test/java/io/r2dbc/postgresql/PostgresqlTestKit.java

@@ -32,6 +32,9 @@
 import static io.r2dbc.spi.ConnectionFactoryOptions.PORT;
 import static io.r2dbc.spi.ConnectionFactoryOptions.USER;
 
+/**
+ * R2DBC TCK implementation.
+ */
 final class PostgresqlTestKit implements TestKit<String> {
 
     @RegisterExtension

src/test/java/io/r2dbc/postgresql/SimpleQueryPostgresqlStatementTest.java src/test/java/io/r2dbc/postgresql/SimpleQueryPostgresqlStatementUnitTests.java

@@ -53,7 +53,10 @@
 import static org.mockito.ArgumentMatchers.any;
 import static org.mockito.Mockito.when;
 
-final class SimpleQueryPostgresqlStatementTest {
+/**
+ * Unit tests for {@link SimpleQueryPostgresqlStatement}.
+ */
+final class SimpleQueryPostgresqlStatementUnitTests {
 
     @Test
     void bind() {

src/test/java/io/r2dbc/postgresql/authentication/PasswordAuthenticationHandlerTest.java src/test/java/io/r2dbc/postgresql/authentication/PasswordAuthenticationHandlerUnitTests.java

@@ -26,7 +26,10 @@
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.assertj.core.api.Assertions.assertThatIllegalArgumentException;
 
-final class PasswordAuthenticationHandlerTest {
+/**
+ * Unit tests for {@link PasswordAuthenticationHandler}.
+ */
+final class PasswordAuthenticationHandlerUnitTests {
 
     @Test
     void constructorNoPassword() {