You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@shardingsphere.apache.org by yx...@apache.org on 2023/03/25 16:50:04 UTC
[shardingsphere] branch master updated: Use package default modifier with IT and E2E (#24827)
This is an automated email from the ASF dual-hosted git repository.
yx9o pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/shardingsphere.git
The following commit(s) were added to refs/heads/master by this push:
new af0fad3a559 Use package default modifier with IT and E2E (#24827)
af0fad3a559 is described below
commit af0fad3a559f70f03fef9d4778b5cee7b49dbb11
Author: Liang Zhang <zh...@apache.org>
AuthorDate: Sun Mar 26 00:49:55 2023 +0800
Use package default modifier with IT and E2E (#24827)
---
.../yaml/it/YamlShardingRuleConfigurationIT.java | 6 +++---
.../sqlfederation/optimizer/it/SQLOptimizeEngineIT.java | 6 +++---
.../mode/repository/cluster/etcd/EtcdRepositoryTest.java | 2 +-
.../DatabaseOperateBackendHandlerFactoryTest.java | 2 +-
.../parser/it/mysql/external/ExternalMySQLParserIT.java | 2 +-
.../parser/it/mysql/internal/InternalMySQLParserIT.java | 2 +-
.../mysql/internal/InternalUnsupportedMySQLParserIT.java | 2 +-
.../it/opengauss/internal/InternalOpenGaussParserIT.java | 2 +-
.../internal/InternalUnsupportedOpenGaussParserIT.java | 2 +-
.../it/oracle/internal/InternalOracleParserIT.java | 2 +-
.../internal/InternalUnsupportedOracleParserIT.java | 2 +-
.../postgresql/external/ExternalPostgreSQLParserIT.java | 2 +-
.../postgresql/internal/InternalPostgreSQLParserIT.java | 2 +-
.../internal/InternalUnsupportedPostgreSQLParserIT.java | 2 +-
.../parser/it/sql92/internal/InternalSQL92ParserIT.java | 2 +-
.../sql92/internal/InternalUnsupportedSQL92ParserIT.java | 2 +-
.../it/sqlserver/internal/InternalSQLServerParserIT.java | 2 +-
.../internal/InternalUnsupportedSQLServerParserIT.java | 2 +-
.../test/e2e/agent/file/FilePluginE2EIT.java | 4 ++--
.../test/e2e/agent/metrics/MetricsPluginE2EIT.java | 4 ++--
.../test/e2e/agent/jaeger/JaegerPluginE2EIT.java | 4 ++--
.../test/e2e/agent/zipkin/ZipkinPluginE2EIT.java | 4 ++--
.../test/e2e/driver/AbstractYamlDataSourceE2EIT.java | 2 +-
.../mix/YamlShardingWithReadwriteSplittingE2EIT.java | 4 ++--
.../driver/readwrite/YamlReadwriteSplittingE2EIT.java | 6 +++---
.../test/e2e/driver/sharding/YamlShardingE2EIT.java | 4 ++--
.../discovery/cases/mysql/MySQLMGRDiscoveryE2EIT.java | 4 ++--
.../test/e2e/data/pipeline/cases/cdc/CDCE2EIT.java | 4 ++--
.../cases/createtable/CreateTableSQLGeneratorIT.java | 6 +++---
.../migration/general/MySQLMigrationGeneralE2EIT.java | 4 ++--
.../general/PostgreSQLMigrationGeneralE2EIT.java | 4 ++--
.../cases/migration/general/RulesMigrationE2EIT.java | 6 +++---
.../migration/primarykey/IndexesMigrationE2EIT.java | 10 +++++-----
.../migration/primarykey/MariaDBMigrationE2EIT.java | 4 ++--
.../primarykey/TextPrimaryKeyMigrationE2EIT.java | 4 ++--
.../e2e/showprocesslist/engine/ShowProcessListE2EIT.java | 4 ++--
.../transaction/engine/base/TransactionBaseE2EIT.java | 2 +-
.../shardingsphere/test/e2e/engine/type/DALE2EIT.java | 4 ++--
.../shardingsphere/test/e2e/engine/type/DCLE2EIT.java | 6 +++---
.../shardingsphere/test/e2e/engine/type/DDLE2EIT.java | 6 +++---
.../shardingsphere/test/e2e/engine/type/RALE2EIT.java | 4 ++--
.../shardingsphere/test/e2e/engine/type/RDLE2EIT.java | 4 ++--
.../shardingsphere/test/e2e/engine/type/RQLE2EIT.java | 4 ++--
.../test/e2e/engine/type/dml/AdditionalDMLE2EIT.java | 16 ++++++++--------
.../test/e2e/engine/type/dml/BatchDMLE2EIT.java | 6 +++---
.../test/e2e/engine/type/dml/GeneralDMLE2EIT.java | 6 +++---
.../test/e2e/engine/type/dql/AdditionalDQLE2EIT.java | 10 +++++-----
.../test/e2e/engine/type/dql/GeneralDQLE2EIT.java | 6 +++---
.../test/it/optimize/SQLNodeConverterEngineIT.java | 4 ++--
.../it/sql/parser/internal/InternalDistSQLParserIT.java | 2 +-
.../it/rewrite/engine/scenario/EncryptSQLRewriterIT.java | 2 +-
.../it/rewrite/engine/scenario/MixSQLRewriterIT.java | 2 +-
.../rewrite/engine/scenario/ShardingSQLRewriterIT.java | 2 +-
53 files changed, 106 insertions(+), 106 deletions(-)
diff --git a/features/sharding/core/src/test/java/org/apache/shardingsphere/sharding/yaml/it/YamlShardingRuleConfigurationIT.java b/features/sharding/core/src/test/java/org/apache/shardingsphere/sharding/yaml/it/YamlShardingRuleConfigurationIT.java
index 96a35bf36c7..09b8dbf6ef4 100644
--- a/features/sharding/core/src/test/java/org/apache/shardingsphere/sharding/yaml/it/YamlShardingRuleConfigurationIT.java
+++ b/features/sharding/core/src/test/java/org/apache/shardingsphere/sharding/yaml/it/YamlShardingRuleConfigurationIT.java
@@ -35,17 +35,17 @@ import static org.junit.jupiter.api.Assertions.assertNotNull;
import static org.hamcrest.MatcherAssert.assertThat;
import static org.junit.jupiter.api.Assertions.assertTrue;
-public final class YamlShardingRuleConfigurationIT {
+class YamlShardingRuleConfigurationIT {
@Test
- public void assertUnmarshalWithYamlFile() throws IOException {
+ void assertUnmarshalWithYamlFile() throws IOException {
URL url = getClass().getClassLoader().getResource("yaml/sharding-rule.yaml");
assertNotNull(url);
assertYamlShardingConfiguration(YamlEngine.unmarshal(new File(url.getFile()), YamlRootConfiguration.class));
}
@Test
- public void assertUnmarshalWithYamlBytes() throws IOException {
+ void assertUnmarshalWithYamlBytes() throws IOException {
URL url = getClass().getClassLoader().getResource("yaml/sharding-rule.yaml");
assertNotNull(url);
StringBuilder yamlContent = new StringBuilder();
diff --git a/kernel/sql-federation/optimizer/src/test/java/org/apache/shardingsphere/sqlfederation/optimizer/it/SQLOptimizeEngineIT.java b/kernel/sql-federation/optimizer/src/test/java/org/apache/shardingsphere/sqlfederation/optimizer/it/SQLOptimizeEngineIT.java
index d815364841e..ea3c1af4f01 100644
--- a/kernel/sql-federation/optimizer/src/test/java/org/apache/shardingsphere/sqlfederation/optimizer/it/SQLOptimizeEngineIT.java
+++ b/kernel/sql-federation/optimizer/src/test/java/org/apache/shardingsphere/sqlfederation/optimizer/it/SQLOptimizeEngineIT.java
@@ -60,7 +60,7 @@ import static org.hamcrest.CoreMatchers.is;
import static org.hamcrest.MatcherAssert.assertThat;
import static org.mockito.Mockito.mock;
-public final class SQLOptimizeEngineIT {
+class SQLOptimizeEngineIT {
private static final String SCHEMA_NAME = "federate_jdbc";
@@ -69,7 +69,7 @@ public final class SQLOptimizeEngineIT {
private SQLOptimizeEngine optimizeEngine;
@BeforeEach
- public void init() {
+ void init() {
Map<String, ShardingSphereTable> tables = new HashMap<>();
tables.put("t_order_federate", createOrderFederationTableMetaData());
tables.put("t_user_info", createUserInfoTableMetaData());
@@ -245,7 +245,7 @@ public final class SQLOptimizeEngineIT {
@ParameterizedTest(name = "{0}")
@ArgumentsSource(TestCaseArgumentsProvider.class)
- public void assertOptimize(final TestCase testcase) {
+ void assertOptimize(final TestCase testcase) {
SQLStatement sqlStatement = sqlParserRule.getSQLParserEngine(DatabaseTypeEngine.getTrunkDatabaseTypeName(new H2DatabaseType())).parse(testcase.getSql(), false);
String actual = optimizeEngine.optimize(sqlStatement).getBestPlan().explain().replaceAll("[\r\n]", "");
assertThat(actual, is(testcase.getAssertion().getExpectedResult()));
diff --git a/mode/type/cluster/repository/provider/etcd/src/test/java/org/apache/shardingsphere/mode/repository/cluster/etcd/EtcdRepositoryTest.java b/mode/type/cluster/repository/provider/etcd/src/test/java/org/apache/shardingsphere/mode/repository/cluster/etcd/EtcdRepositoryTest.java
index 964cd2b68e8..2c9e8b1245a 100644
--- a/mode/type/cluster/repository/provider/etcd/src/test/java/org/apache/shardingsphere/mode/repository/cluster/etcd/EtcdRepositoryTest.java
+++ b/mode/type/cluster/repository/provider/etcd/src/test/java/org/apache/shardingsphere/mode/repository/cluster/etcd/EtcdRepositoryTest.java
@@ -161,7 +161,7 @@ class EtcdRepositoryTest {
@Test
@SuppressWarnings("unchecked")
- public void assertPersistEphemeral() {
+ void assertPersistEphemeral() {
repository.persistEphemeral("key1", "value1");
verify(lease).grant(anyLong());
verify(lease).keepAlive(anyLong(), any(StreamObserver.class));
diff --git a/proxy/backend/core/src/test/java/org/apache/shardingsphere/proxy/backend/handler/database/DatabaseOperateBackendHandlerFactoryTest.java b/proxy/backend/core/src/test/java/org/apache/shardingsphere/proxy/backend/handler/database/DatabaseOperateBackendHandlerFactoryTest.java
index 5d207313a71..223a5797941 100644
--- a/proxy/backend/core/src/test/java/org/apache/shardingsphere/proxy/backend/handler/database/DatabaseOperateBackendHandlerFactoryTest.java
+++ b/proxy/backend/core/src/test/java/org/apache/shardingsphere/proxy/backend/handler/database/DatabaseOperateBackendHandlerFactoryTest.java
@@ -132,7 +132,7 @@ class DatabaseOperateBackendHandlerFactoryTest {
assertExecuteCreateDatabaseContextWithException(new PostgreSQLCreateDatabaseStatement());
}
- public void assertExecuteCreateDatabaseContextWithException(final CreateDatabaseStatement sqlStatement) {
+ private void assertExecuteCreateDatabaseContextWithException(final CreateDatabaseStatement sqlStatement) {
sqlStatement.setDatabaseName("foo_db");
setGovernanceMetaDataContexts(true);
try {
diff --git a/sql-parser/dialect/mysql/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/mysql/external/ExternalMySQLParserIT.java b/sql-parser/dialect/mysql/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/mysql/external/ExternalMySQLParserIT.java
index e81ffd77b57..bde03a3a4a5 100644
--- a/sql-parser/dialect/mysql/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/mysql/external/ExternalMySQLParserIT.java
+++ b/sql-parser/dialect/mysql/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/mysql/external/ExternalMySQLParserIT.java
@@ -21,7 +21,7 @@ import org.apache.shardingsphere.test.it.sql.parser.external.ExternalSQLParserIT
import org.apache.shardingsphere.test.it.sql.parser.external.ExternalSQLParserITSettings;
@ExternalSQLParserITSettings(value = "MySQL", caseURL = ExternalMySQLParserIT.CASE_URL, resultURL = ExternalMySQLParserIT.RESULT_URL)
-public final class ExternalMySQLParserIT extends ExternalSQLParserIT {
+class ExternalMySQLParserIT extends ExternalSQLParserIT {
static final String CASE_URL = "https://github.com/mysql/mysql-server/tree/8.0/mysql-test/t";
diff --git a/sql-parser/dialect/mysql/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/mysql/internal/InternalMySQLParserIT.java b/sql-parser/dialect/mysql/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/mysql/internal/InternalMySQLParserIT.java
index e06e1449497..61154507847 100644
--- a/sql-parser/dialect/mysql/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/mysql/internal/InternalMySQLParserIT.java
+++ b/sql-parser/dialect/mysql/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/mysql/internal/InternalMySQLParserIT.java
@@ -21,5 +21,5 @@ import org.apache.shardingsphere.test.it.sql.parser.internal.InternalSQLParserIT
import org.apache.shardingsphere.test.it.sql.parser.internal.InternalSQLParserITSettings;
@InternalSQLParserITSettings({"MySQL", "H2"})
-public final class InternalMySQLParserIT extends InternalSQLParserIT {
+class InternalMySQLParserIT extends InternalSQLParserIT {
}
diff --git a/sql-parser/dialect/mysql/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/mysql/internal/InternalUnsupportedMySQLParserIT.java b/sql-parser/dialect/mysql/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/mysql/internal/InternalUnsupportedMySQLParserIT.java
index 6754b9e400d..50abf72c886 100644
--- a/sql-parser/dialect/mysql/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/mysql/internal/InternalUnsupportedMySQLParserIT.java
+++ b/sql-parser/dialect/mysql/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/mysql/internal/InternalUnsupportedMySQLParserIT.java
@@ -21,5 +21,5 @@ import org.apache.shardingsphere.test.it.sql.parser.internal.InternalSQLParserIT
import org.apache.shardingsphere.test.it.sql.parser.internal.InternalUnsupportedSQLParserIT;
@InternalSQLParserITSettings("MySQL")
-public final class InternalUnsupportedMySQLParserIT extends InternalUnsupportedSQLParserIT {
+class InternalUnsupportedMySQLParserIT extends InternalUnsupportedSQLParserIT {
}
diff --git a/sql-parser/dialect/opengauss/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/opengauss/internal/InternalOpenGaussParserIT.java b/sql-parser/dialect/opengauss/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/opengauss/internal/InternalOpenGaussParserIT.java
index d1ae3510091..54b445cb2fe 100644
--- a/sql-parser/dialect/opengauss/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/opengauss/internal/InternalOpenGaussParserIT.java
+++ b/sql-parser/dialect/opengauss/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/opengauss/internal/InternalOpenGaussParserIT.java
@@ -21,5 +21,5 @@ import org.apache.shardingsphere.test.it.sql.parser.internal.InternalSQLParserIT
import org.apache.shardingsphere.test.it.sql.parser.internal.InternalSQLParserITSettings;
@InternalSQLParserITSettings("openGauss")
-public final class InternalOpenGaussParserIT extends InternalSQLParserIT {
+class InternalOpenGaussParserIT extends InternalSQLParserIT {
}
diff --git a/sql-parser/dialect/opengauss/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/opengauss/internal/InternalUnsupportedOpenGaussParserIT.java b/sql-parser/dialect/opengauss/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/opengauss/internal/InternalUnsupportedOpenGaussParserIT.java
index fa6fd40a05a..1769c5c6b83 100644
--- a/sql-parser/dialect/opengauss/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/opengauss/internal/InternalUnsupportedOpenGaussParserIT.java
+++ b/sql-parser/dialect/opengauss/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/opengauss/internal/InternalUnsupportedOpenGaussParserIT.java
@@ -21,5 +21,5 @@ import org.apache.shardingsphere.test.it.sql.parser.internal.InternalSQLParserIT
import org.apache.shardingsphere.test.it.sql.parser.internal.InternalUnsupportedSQLParserIT;
@InternalSQLParserITSettings("openGauss")
-public final class InternalUnsupportedOpenGaussParserIT extends InternalUnsupportedSQLParserIT {
+class InternalUnsupportedOpenGaussParserIT extends InternalUnsupportedSQLParserIT {
}
diff --git a/sql-parser/dialect/oracle/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/oracle/internal/InternalOracleParserIT.java b/sql-parser/dialect/oracle/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/oracle/internal/InternalOracleParserIT.java
index 8103f6f350d..3bf137e838e 100644
--- a/sql-parser/dialect/oracle/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/oracle/internal/InternalOracleParserIT.java
+++ b/sql-parser/dialect/oracle/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/oracle/internal/InternalOracleParserIT.java
@@ -21,5 +21,5 @@ import org.apache.shardingsphere.test.it.sql.parser.internal.InternalSQLParserIT
import org.apache.shardingsphere.test.it.sql.parser.internal.InternalSQLParserITSettings;
@InternalSQLParserITSettings("Oracle")
-public final class InternalOracleParserIT extends InternalSQLParserIT {
+class InternalOracleParserIT extends InternalSQLParserIT {
}
diff --git a/sql-parser/dialect/oracle/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/oracle/internal/InternalUnsupportedOracleParserIT.java b/sql-parser/dialect/oracle/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/oracle/internal/InternalUnsupportedOracleParserIT.java
index 286f9910fbd..f92e3a1d43d 100644
--- a/sql-parser/dialect/oracle/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/oracle/internal/InternalUnsupportedOracleParserIT.java
+++ b/sql-parser/dialect/oracle/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/oracle/internal/InternalUnsupportedOracleParserIT.java
@@ -21,5 +21,5 @@ import org.apache.shardingsphere.test.it.sql.parser.internal.InternalSQLParserIT
import org.apache.shardingsphere.test.it.sql.parser.internal.InternalUnsupportedSQLParserIT;
@InternalSQLParserITSettings("Oracle")
-public final class InternalUnsupportedOracleParserIT extends InternalUnsupportedSQLParserIT {
+class InternalUnsupportedOracleParserIT extends InternalUnsupportedSQLParserIT {
}
diff --git a/sql-parser/dialect/postgresql/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/postgresql/external/ExternalPostgreSQLParserIT.java b/sql-parser/dialect/postgresql/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/postgresql/external/ExternalPostgreSQLParserIT.java
index 0e3dea729f7..6916dade58f 100644
--- a/sql-parser/dialect/postgresql/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/postgresql/external/ExternalPostgreSQLParserIT.java
+++ b/sql-parser/dialect/postgresql/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/postgresql/external/ExternalPostgreSQLParserIT.java
@@ -21,7 +21,7 @@ import org.apache.shardingsphere.test.it.sql.parser.external.ExternalSQLParserIT
import org.apache.shardingsphere.test.it.sql.parser.external.ExternalSQLParserITSettings;
@ExternalSQLParserITSettings(value = "PostgreSQL", caseURL = ExternalPostgreSQLParserIT.CASE_URL, resultURL = ExternalPostgreSQLParserIT.RESULT_URL)
-public final class ExternalPostgreSQLParserIT extends ExternalSQLParserIT {
+class ExternalPostgreSQLParserIT extends ExternalSQLParserIT {
static final String CASE_URL = "https://github.com/postgres/postgres/tree/master/src/test/regress/sql";
diff --git a/sql-parser/dialect/postgresql/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/postgresql/internal/InternalPostgreSQLParserIT.java b/sql-parser/dialect/postgresql/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/postgresql/internal/InternalPostgreSQLParserIT.java
index 96b69f234c7..a14fc10e496 100644
--- a/sql-parser/dialect/postgresql/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/postgresql/internal/InternalPostgreSQLParserIT.java
+++ b/sql-parser/dialect/postgresql/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/postgresql/internal/InternalPostgreSQLParserIT.java
@@ -21,5 +21,5 @@ import org.apache.shardingsphere.test.it.sql.parser.internal.InternalSQLParserIT
import org.apache.shardingsphere.test.it.sql.parser.internal.InternalSQLParserITSettings;
@InternalSQLParserITSettings("PostgreSQL")
-public final class InternalPostgreSQLParserIT extends InternalSQLParserIT {
+class InternalPostgreSQLParserIT extends InternalSQLParserIT {
}
diff --git a/sql-parser/dialect/postgresql/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/postgresql/internal/InternalUnsupportedPostgreSQLParserIT.java b/sql-parser/dialect/postgresql/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/postgresql/internal/InternalUnsupportedPostgreSQLParserIT.java
index 70d8799f4f6..2ebb1dbf77c 100644
--- a/sql-parser/dialect/postgresql/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/postgresql/internal/InternalUnsupportedPostgreSQLParserIT.java
+++ b/sql-parser/dialect/postgresql/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/postgresql/internal/InternalUnsupportedPostgreSQLParserIT.java
@@ -21,5 +21,5 @@ import org.apache.shardingsphere.test.it.sql.parser.internal.InternalSQLParserIT
import org.apache.shardingsphere.test.it.sql.parser.internal.InternalUnsupportedSQLParserIT;
@InternalSQLParserITSettings("PostgreSQL")
-public final class InternalUnsupportedPostgreSQLParserIT extends InternalUnsupportedSQLParserIT {
+class InternalUnsupportedPostgreSQLParserIT extends InternalUnsupportedSQLParserIT {
}
diff --git a/sql-parser/dialect/sql92/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/sql92/internal/InternalSQL92ParserIT.java b/sql-parser/dialect/sql92/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/sql92/internal/InternalSQL92ParserIT.java
index 60002e8bbea..95bd4c49ff9 100644
--- a/sql-parser/dialect/sql92/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/sql92/internal/InternalSQL92ParserIT.java
+++ b/sql-parser/dialect/sql92/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/sql92/internal/InternalSQL92ParserIT.java
@@ -21,5 +21,5 @@ import org.apache.shardingsphere.test.it.sql.parser.internal.InternalSQLParserIT
import org.apache.shardingsphere.test.it.sql.parser.internal.InternalSQLParserITSettings;
@InternalSQLParserITSettings("SQL92")
-public final class InternalSQL92ParserIT extends InternalSQLParserIT {
+class InternalSQL92ParserIT extends InternalSQLParserIT {
}
diff --git a/sql-parser/dialect/sql92/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/sql92/internal/InternalUnsupportedSQL92ParserIT.java b/sql-parser/dialect/sql92/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/sql92/internal/InternalUnsupportedSQL92ParserIT.java
index 752a09f286a..78a61ff3274 100644
--- a/sql-parser/dialect/sql92/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/sql92/internal/InternalUnsupportedSQL92ParserIT.java
+++ b/sql-parser/dialect/sql92/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/sql92/internal/InternalUnsupportedSQL92ParserIT.java
@@ -21,5 +21,5 @@ import org.apache.shardingsphere.test.it.sql.parser.internal.InternalSQLParserIT
import org.apache.shardingsphere.test.it.sql.parser.internal.InternalUnsupportedSQLParserIT;
@InternalSQLParserITSettings("SQL92")
-public final class InternalUnsupportedSQL92ParserIT extends InternalUnsupportedSQLParserIT {
+class InternalUnsupportedSQL92ParserIT extends InternalUnsupportedSQLParserIT {
}
diff --git a/sql-parser/dialect/sqlserver/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/sqlserver/internal/InternalSQLServerParserIT.java b/sql-parser/dialect/sqlserver/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/sqlserver/internal/InternalSQLServerParserIT.java
index 0230e445b78..9ad207aa5ae 100644
--- a/sql-parser/dialect/sqlserver/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/sqlserver/internal/InternalSQLServerParserIT.java
+++ b/sql-parser/dialect/sqlserver/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/sqlserver/internal/InternalSQLServerParserIT.java
@@ -21,5 +21,5 @@ import org.apache.shardingsphere.test.it.sql.parser.internal.InternalSQLParserIT
import org.apache.shardingsphere.test.it.sql.parser.internal.InternalSQLParserITSettings;
@InternalSQLParserITSettings("SQLServer")
-public final class InternalSQLServerParserIT extends InternalSQLParserIT {
+class InternalSQLServerParserIT extends InternalSQLParserIT {
}
diff --git a/sql-parser/dialect/sqlserver/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/sqlserver/internal/InternalUnsupportedSQLServerParserIT.java b/sql-parser/dialect/sqlserver/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/sqlserver/internal/InternalUnsupportedSQLServerParserIT.java
index f1d121af2e0..b3436989999 100644
--- a/sql-parser/dialect/sqlserver/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/sqlserver/internal/InternalUnsupportedSQLServerParserIT.java
+++ b/sql-parser/dialect/sqlserver/src/test/java/org/apache/shardingsphere/test/it/sql/parser/it/sqlserver/internal/InternalUnsupportedSQLServerParserIT.java
@@ -21,5 +21,5 @@ import org.apache.shardingsphere.test.it.sql.parser.internal.InternalSQLParserIT
import org.apache.shardingsphere.test.it.sql.parser.internal.InternalUnsupportedSQLParserIT;
@InternalSQLParserITSettings("SQLServer")
-public final class InternalUnsupportedSQLServerParserIT extends InternalUnsupportedSQLParserIT {
+class InternalUnsupportedSQLServerParserIT extends InternalUnsupportedSQLParserIT {
}
diff --git a/test/e2e/agent/plugins/logging/file/src/test/java/org/apache/shardingsphere/test/e2e/agent/file/FilePluginE2EIT.java b/test/e2e/agent/plugins/logging/file/src/test/java/org/apache/shardingsphere/test/e2e/agent/file/FilePluginE2EIT.java
index f3c7b061511..5f30c437060 100644
--- a/test/e2e/agent/plugins/logging/file/src/test/java/org/apache/shardingsphere/test/e2e/agent/file/FilePluginE2EIT.java
+++ b/test/e2e/agent/plugins/logging/file/src/test/java/org/apache/shardingsphere/test/e2e/agent/file/FilePluginE2EIT.java
@@ -31,10 +31,10 @@ import static org.junit.jupiter.api.Assertions.assertFalse;
import static org.junit.jupiter.api.Assertions.assertTrue;
@ExtendWith(AgentTestActionExtension.class)
-public final class FilePluginE2EIT {
+class FilePluginE2EIT {
@Test
- public void assertWithAgent() {
+ void assertWithAgent() {
assertTrue(new File(LogLoader.getLogFilePath(E2ETestEnvironment.getInstance().isAdaptedProxy())).exists(),
String.format("The file `%s` does not exist", LogLoader.getLogFilePath(E2ETestEnvironment.getInstance().isAdaptedProxy())));
Collection<String> actualLogLines = LogLoader.getLogLines(E2ETestEnvironment.getInstance().isAdaptedProxy());
diff --git a/test/e2e/agent/plugins/metrics/prometheus/src/test/java/org/apache/shardingsphere/test/e2e/agent/metrics/MetricsPluginE2EIT.java b/test/e2e/agent/plugins/metrics/prometheus/src/test/java/org/apache/shardingsphere/test/e2e/agent/metrics/MetricsPluginE2EIT.java
index 2ebe6f9c082..607df847fba 100644
--- a/test/e2e/agent/plugins/metrics/prometheus/src/test/java/org/apache/shardingsphere/test/e2e/agent/metrics/MetricsPluginE2EIT.java
+++ b/test/e2e/agent/plugins/metrics/prometheus/src/test/java/org/apache/shardingsphere/test/e2e/agent/metrics/MetricsPluginE2EIT.java
@@ -42,11 +42,11 @@ import java.util.stream.Stream;
@ExtendWith(AgentTestActionExtension.class)
@Slf4j
-public final class MetricsPluginE2EIT {
+class MetricsPluginE2EIT {
@ParameterizedTest
@ArgumentsSource(TestCaseArgumentsProvider.class)
- public void assertWithAgent(final MetricTestCase metricTestCase) {
+ void assertWithAgent(final MetricTestCase metricTestCase) {
Properties props = E2ETestEnvironment.getInstance().getProps();
String metaDataURL = props.getProperty("prometheus.metadata.url");
String queryURL = props.getProperty("prometheus.query.url");
diff --git a/test/e2e/agent/plugins/tracing/jaeger/src/test/java/org/apache/shardingsphere/test/e2e/agent/jaeger/JaegerPluginE2EIT.java b/test/e2e/agent/plugins/tracing/jaeger/src/test/java/org/apache/shardingsphere/test/e2e/agent/jaeger/JaegerPluginE2EIT.java
index 210e660a12e..579aaf75fe8 100644
--- a/test/e2e/agent/plugins/tracing/jaeger/src/test/java/org/apache/shardingsphere/test/e2e/agent/jaeger/JaegerPluginE2EIT.java
+++ b/test/e2e/agent/plugins/tracing/jaeger/src/test/java/org/apache/shardingsphere/test/e2e/agent/jaeger/JaegerPluginE2EIT.java
@@ -32,11 +32,11 @@ import org.junit.jupiter.params.provider.ArgumentsSource;
import java.util.stream.Stream;
@ExtendWith(AgentTestActionExtension.class)
-public final class JaegerPluginE2EIT {
+class JaegerPluginE2EIT {
@ParameterizedTest
@ArgumentsSource(TestCaseArgumentsProvider.class)
- public void assertWithAgent(final SpanTestCase spanTestCase) {
+ void assertWithAgent(final SpanTestCase spanTestCase) {
SpanAssert.assertIs(E2ETestEnvironment.getInstance().getProps().getProperty("jaeger.url"), spanTestCase);
}
diff --git a/test/e2e/agent/plugins/tracing/zipkin/src/test/java/org/apache/shardingsphere/test/e2e/agent/zipkin/ZipkinPluginE2EIT.java b/test/e2e/agent/plugins/tracing/zipkin/src/test/java/org/apache/shardingsphere/test/e2e/agent/zipkin/ZipkinPluginE2EIT.java
index d3aec28e643..d29788574ba 100644
--- a/test/e2e/agent/plugins/tracing/zipkin/src/test/java/org/apache/shardingsphere/test/e2e/agent/zipkin/ZipkinPluginE2EIT.java
+++ b/test/e2e/agent/plugins/tracing/zipkin/src/test/java/org/apache/shardingsphere/test/e2e/agent/zipkin/ZipkinPluginE2EIT.java
@@ -32,11 +32,11 @@ import org.junit.jupiter.params.provider.ArgumentsSource;
import java.util.stream.Stream;
@ExtendWith(AgentTestActionExtension.class)
-public final class ZipkinPluginE2EIT {
+class ZipkinPluginE2EIT {
@ParameterizedTest
@ArgumentsSource(TestCaseArgumentsProvider.class)
- public void assertWithAgent(final SpanTestCase spanTestCase) {
+ void assertWithAgent(final SpanTestCase spanTestCase) {
SpanAssert.assertIs(E2ETestEnvironment.getInstance().getProps().getProperty("zipkin.url"), spanTestCase);
}
diff --git a/test/e2e/driver/src/test/java/org/apache/shardingsphere/test/e2e/driver/AbstractYamlDataSourceE2EIT.java b/test/e2e/driver/src/test/java/org/apache/shardingsphere/test/e2e/driver/AbstractYamlDataSourceE2EIT.java
index 36da9eaef32..df78d39b418 100644
--- a/test/e2e/driver/src/test/java/org/apache/shardingsphere/test/e2e/driver/AbstractYamlDataSourceE2EIT.java
+++ b/test/e2e/driver/src/test/java/org/apache/shardingsphere/test/e2e/driver/AbstractYamlDataSourceE2EIT.java
@@ -33,7 +33,7 @@ import java.util.Objects;
public abstract class AbstractYamlDataSourceE2EIT {
@BeforeAll
- public static void createSchema() throws SQLException {
+ static void createSchema() throws SQLException {
for (String each : getSchemaFiles()) {
RunScript.execute(
createDataSource(getFileName(each)).getConnection(), new InputStreamReader(Objects.requireNonNull(AbstractYamlDataSourceE2EIT.class.getClassLoader().getResourceAsStream(each))));
diff --git a/test/e2e/driver/src/test/java/org/apache/shardingsphere/test/e2e/driver/mix/YamlShardingWithReadwriteSplittingE2EIT.java b/test/e2e/driver/src/test/java/org/apache/shardingsphere/test/e2e/driver/mix/YamlShardingWithReadwriteSplittingE2EIT.java
index 089bb604f0a..665cb8bf689 100644
--- a/test/e2e/driver/src/test/java/org/apache/shardingsphere/test/e2e/driver/mix/YamlShardingWithReadwriteSplittingE2EIT.java
+++ b/test/e2e/driver/src/test/java/org/apache/shardingsphere/test/e2e/driver/mix/YamlShardingWithReadwriteSplittingE2EIT.java
@@ -35,11 +35,11 @@ import java.util.Map;
import java.util.Objects;
import java.util.stream.Stream;
-public final class YamlShardingWithReadwriteSplittingE2EIT extends AbstractYamlDataSourceE2EIT {
+class YamlShardingWithReadwriteSplittingE2EIT extends AbstractYamlDataSourceE2EIT {
@ParameterizedTest(name = "{index}:{0}-{1}")
@ArgumentsSource(TestCaseArgumentsProvider.class)
- public void assertWithDataSource(final String filePath, final boolean hasDataSource) throws Exception {
+ void assertWithDataSource(final String filePath, final boolean hasDataSource) throws Exception {
File yamlFile = new File(Objects.requireNonNull(YamlShardingWithReadwriteSplittingE2EIT.class.getResource(filePath)).toURI());
DataSource dataSource;
if (hasDataSource) {
diff --git a/test/e2e/driver/src/test/java/org/apache/shardingsphere/test/e2e/driver/readwrite/YamlReadwriteSplittingE2EIT.java b/test/e2e/driver/src/test/java/org/apache/shardingsphere/test/e2e/driver/readwrite/YamlReadwriteSplittingE2EIT.java
index 59d3cd07a31..13d53d264a9 100644
--- a/test/e2e/driver/src/test/java/org/apache/shardingsphere/test/e2e/driver/readwrite/YamlReadwriteSplittingE2EIT.java
+++ b/test/e2e/driver/src/test/java/org/apache/shardingsphere/test/e2e/driver/readwrite/YamlReadwriteSplittingE2EIT.java
@@ -35,11 +35,11 @@ import java.util.Map;
import java.util.Objects;
import java.util.stream.Stream;
-public final class YamlReadwriteSplittingE2EIT extends AbstractYamlDataSourceE2EIT {
+class YamlReadwriteSplittingE2EIT extends AbstractYamlDataSourceE2EIT {
@ParameterizedTest(name = "{index}:{0}-{1}")
@ArgumentsSource(TestCaseArgumentsProvider.class)
- public void assertWithDataSource(final String filePath, final boolean hasDataSource) throws Exception {
+ void assertWithDataSource(final String filePath, final boolean hasDataSource) throws Exception {
File yamlFile = new File(Objects.requireNonNull(YamlReadwriteSplittingE2EIT.class.getResource(filePath)).toURI());
DataSource dataSource;
if (hasDataSource) {
@@ -63,7 +63,7 @@ public final class YamlReadwriteSplittingE2EIT extends AbstractYamlDataSourceE2E
@ParameterizedTest(name = "{index}:{0}-{1}")
@ArgumentsSource(TestCaseArgumentsProvider.class)
- public void assertWithDataSourceByYamlBytes(final String filePath, final boolean hasDataSource) throws Exception {
+ void assertWithDataSourceByYamlBytes(final String filePath, final boolean hasDataSource) throws Exception {
File yamlFile = new File(Objects.requireNonNull(YamlReadwriteSplittingE2EIT.class.getResource(filePath)).toURI());
DataSource dataSource;
if (hasDataSource) {
diff --git a/test/e2e/driver/src/test/java/org/apache/shardingsphere/test/e2e/driver/sharding/YamlShardingE2EIT.java b/test/e2e/driver/src/test/java/org/apache/shardingsphere/test/e2e/driver/sharding/YamlShardingE2EIT.java
index 8a4ccf2bf76..359eeae7e67 100644
--- a/test/e2e/driver/src/test/java/org/apache/shardingsphere/test/e2e/driver/sharding/YamlShardingE2EIT.java
+++ b/test/e2e/driver/src/test/java/org/apache/shardingsphere/test/e2e/driver/sharding/YamlShardingE2EIT.java
@@ -35,11 +35,11 @@ import java.util.Map;
import java.util.Objects;
import java.util.stream.Stream;
-public final class YamlShardingE2EIT extends AbstractYamlDataSourceE2EIT {
+class YamlShardingE2EIT extends AbstractYamlDataSourceE2EIT {
@ParameterizedTest(name = "{index}:{0}-{1}")
@ArgumentsSource(TestCaseArgumentsProvider.class)
- public void assertWithDataSource(final String filePath, final boolean hasDataSource) throws Exception {
+ void assertWithDataSource(final String filePath, final boolean hasDataSource) throws Exception {
File yamlFile = new File(Objects.requireNonNull(YamlShardingE2EIT.class.getResource(filePath)).toURI());
DataSource dataSource;
if (hasDataSource) {
diff --git a/test/e2e/operation/discovery/src/test/java/org/apache/shardingsphere/test/e2e/discovery/cases/mysql/MySQLMGRDiscoveryE2EIT.java b/test/e2e/operation/discovery/src/test/java/org/apache/shardingsphere/test/e2e/discovery/cases/mysql/MySQLMGRDiscoveryE2EIT.java
index edd5d3f3e5e..fe92f9d952e 100644
--- a/test/e2e/operation/discovery/src/test/java/org/apache/shardingsphere/test/e2e/discovery/cases/mysql/MySQLMGRDiscoveryE2EIT.java
+++ b/test/e2e/operation/discovery/src/test/java/org/apache/shardingsphere/test/e2e/discovery/cases/mysql/MySQLMGRDiscoveryE2EIT.java
@@ -34,14 +34,14 @@ import org.junit.jupiter.params.provider.ArgumentsSource;
import java.sql.SQLException;
import java.util.stream.Stream;
-public final class MySQLMGRDiscoveryE2EIT {
+class MySQLMGRDiscoveryE2EIT {
private static final MySQLDatabaseType DATABASE_TYPE = new MySQLDatabaseType();
@ParameterizedTest(name = "{0}")
@EnabledIf("isEnabled")
@ArgumentsSource(TestCaseArgumentsProvider.class)
- public void assertDiscoveryMySQLMGR(final DiscoveryTestParameter testParam) throws SQLException {
+ void assertDiscoveryMySQLMGR(final DiscoveryTestParameter testParam) throws SQLException {
try (DiscoveryContainerComposer composer = new DiscoveryContainerComposer(testParam)) {
DatabaseClusterEnvironment databaseClusterEnv = DatabaseClusterEnvironmentFactory.newInstance("MySQL.MGR", composer.getMappedDataSources());
new DiscoveryTestAction(composer, databaseClusterEnv).execute();
diff --git a/test/e2e/operation/pipeline/src/test/java/org/apache/shardingsphere/test/e2e/data/pipeline/cases/cdc/CDCE2EIT.java b/test/e2e/operation/pipeline/src/test/java/org/apache/shardingsphere/test/e2e/data/pipeline/cases/cdc/CDCE2EIT.java
index 22e314d0433..98da9afa77e 100644
--- a/test/e2e/operation/pipeline/src/test/java/org/apache/shardingsphere/test/e2e/data/pipeline/cases/cdc/CDCE2EIT.java
+++ b/test/e2e/operation/pipeline/src/test/java/org/apache/shardingsphere/test/e2e/data/pipeline/cases/cdc/CDCE2EIT.java
@@ -81,7 +81,7 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
@PipelineE2EDatabaseSettings(type = "MySQL", scenarioFiles = "env/scenario/general/mysql.xml"),
@PipelineE2EDatabaseSettings(type = "openGauss", scenarioFiles = "env/scenario/general/postgresql.xml")})
@Slf4j
-public final class CDCE2EIT {
+class CDCE2EIT {
private static final String CREATE_SHARDING_RULE_SQL = String.format("CREATE SHARDING TABLE RULE t_order("
+ "STORAGE_UNITS(%s,%s),"
@@ -97,7 +97,7 @@ public final class CDCE2EIT {
@ParameterizedTest(name = "{0}")
@EnabledIf("isEnabled")
@ArgumentsSource(PipelineE2ETestCaseArgumentsProvider.class)
- public void assertCDCDataImportSuccess(final PipelineTestParameter testParam) throws SQLException, InterruptedException {
+ void assertCDCDataImportSuccess(final PipelineTestParameter testParam) throws SQLException, InterruptedException {
if (TimeZone.getDefault() != TimeZone.getTimeZone("UTC") && PipelineEnvTypeEnum.DOCKER == PipelineE2EEnvironment.getInstance().getItEnvType()) {
// make sure the time zone of locally running program same with the database server at CI.
TimeZone.setDefault(TimeZone.getTimeZone("UTC"));
diff --git a/test/e2e/operation/pipeline/src/test/java/org/apache/shardingsphere/test/e2e/data/pipeline/cases/createtable/CreateTableSQLGeneratorIT.java b/test/e2e/operation/pipeline/src/test/java/org/apache/shardingsphere/test/e2e/data/pipeline/cases/createtable/CreateTableSQLGeneratorIT.java
index 5a2a904a1f8..8a8082fbc70 100644
--- a/test/e2e/operation/pipeline/src/test/java/org/apache/shardingsphere/test/e2e/data/pipeline/cases/createtable/CreateTableSQLGeneratorIT.java
+++ b/test/e2e/operation/pipeline/src/test/java/org/apache/shardingsphere/test/e2e/data/pipeline/cases/createtable/CreateTableSQLGeneratorIT.java
@@ -64,7 +64,7 @@ import static org.hamcrest.MatcherAssert.assertThat;
@PipelineE2EDatabaseSettings(type = "MySQL", scenarioFiles = "env/scenario/create-table-generator/mysql/create-table-sql-generator.xml"),
@PipelineE2EDatabaseSettings(type = "PostgreSQL", scenarioFiles = "env/scenario/create-table-generator/postgresql/create-table-sql-generator.xml"),
@PipelineE2EDatabaseSettings(type = "openGauss", scenarioFiles = "env/scenario/create-table-generator/opengauss/create-table-sql-generator.xml")})
-public final class CreateTableSQLGeneratorIT {
+class CreateTableSQLGeneratorIT {
private static final String DEFAULT_SCHEMA = "public";
@@ -75,14 +75,14 @@ public final class CreateTableSQLGeneratorIT {
private DockerStorageContainer storageContainer;
@AfterEach
- public void stopContainer() {
+ void stopContainer() {
storageContainer.stop();
}
@ParameterizedTest(name = "{0}")
@EnabledIf("isEnabled")
@ArgumentsSource(PipelineE2ETestCaseArgumentsProvider.class)
- public void assertGenerateCreateTableSQL(final PipelineTestParameter testParam) throws SQLException {
+ void assertGenerateCreateTableSQL(final PipelineTestParameter testParam) throws SQLException {
startStorageContainer(testParam.getDatabaseType(), testParam.getStorageContainerImage());
CreateTableSQLGeneratorAssertionsRootEntity rootEntity = JAXB.unmarshal(
Objects.requireNonNull(CreateTableSQLGeneratorIT.class.getClassLoader().getResource(testParam.getScenario())), CreateTableSQLGeneratorAssertionsRootEntity.class);
diff --git a/test/e2e/operation/pipeline/src/test/java/org/apache/shardingsphere/test/e2e/data/pipeline/cases/migration/general/MySQLMigrationGeneralE2EIT.java b/test/e2e/operation/pipeline/src/test/java/org/apache/shardingsphere/test/e2e/data/pipeline/cases/migration/general/MySQLMigrationGeneralE2EIT.java
index c26a1793ffe..e9fecfd1954 100644
--- a/test/e2e/operation/pipeline/src/test/java/org/apache/shardingsphere/test/e2e/data/pipeline/cases/migration/general/MySQLMigrationGeneralE2EIT.java
+++ b/test/e2e/operation/pipeline/src/test/java/org/apache/shardingsphere/test/e2e/data/pipeline/cases/migration/general/MySQLMigrationGeneralE2EIT.java
@@ -47,7 +47,7 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
@PipelineE2ESettings(database = @PipelineE2EDatabaseSettings(type = "MySQL", scenarioFiles = "env/scenario/general/mysql.xml"))
@Slf4j
-public final class MySQLMigrationGeneralE2EIT extends AbstractMigrationE2EIT {
+class MySQLMigrationGeneralE2EIT extends AbstractMigrationE2EIT {
private static final String SOURCE_TABLE_NAME = "t_order_copy";
@@ -56,7 +56,7 @@ public final class MySQLMigrationGeneralE2EIT extends AbstractMigrationE2EIT {
@ParameterizedTest(name = "{0}")
@EnabledIf("isEnabled")
@ArgumentsSource(PipelineE2ETestCaseArgumentsProvider.class)
- public void assertMigrationSuccess(final PipelineTestParameter testParam) throws SQLException, InterruptedException {
+ void assertMigrationSuccess(final PipelineTestParameter testParam) throws SQLException, InterruptedException {
try (PipelineContainerComposer containerComposer = new PipelineContainerComposer(testParam, new MigrationJobType())) {
addMigrationProcessConfig(containerComposer);
containerComposer.createSourceOrderTable(SOURCE_TABLE_NAME);
diff --git a/test/e2e/operation/pipeline/src/test/java/org/apache/shardingsphere/test/e2e/data/pipeline/cases/migration/general/PostgreSQLMigrationGeneralE2EIT.java b/test/e2e/operation/pipeline/src/test/java/org/apache/shardingsphere/test/e2e/data/pipeline/cases/migration/general/PostgreSQLMigrationGeneralE2EIT.java
index 266ef8ac0df..1c5f18bb194 100644
--- a/test/e2e/operation/pipeline/src/test/java/org/apache/shardingsphere/test/e2e/data/pipeline/cases/migration/general/PostgreSQLMigrationGeneralE2EIT.java
+++ b/test/e2e/operation/pipeline/src/test/java/org/apache/shardingsphere/test/e2e/data/pipeline/cases/migration/general/PostgreSQLMigrationGeneralE2EIT.java
@@ -49,7 +49,7 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
@PipelineE2EDatabaseSettings(type = "PostgreSQL", scenarioFiles = "env/scenario/general/postgresql.xml"),
@PipelineE2EDatabaseSettings(type = "openGauss", scenarioFiles = "env/scenario/general/postgresql.xml")})
@Slf4j
-public final class PostgreSQLMigrationGeneralE2EIT extends AbstractMigrationE2EIT {
+class PostgreSQLMigrationGeneralE2EIT extends AbstractMigrationE2EIT {
private static final String SOURCE_TABLE_NAME = "t_order_copy";
@@ -58,7 +58,7 @@ public final class PostgreSQLMigrationGeneralE2EIT extends AbstractMigrationE2EI
@ParameterizedTest(name = "{0}")
@EnabledIf("isEnabled")
@ArgumentsSource(PipelineE2ETestCaseArgumentsProvider.class)
- public void assertMigrationSuccess(final PipelineTestParameter testParam) throws SQLException, InterruptedException {
+ void assertMigrationSuccess(final PipelineTestParameter testParam) throws SQLException, InterruptedException {
try (PipelineContainerComposer containerComposer = new PipelineContainerComposer(testParam, new MigrationJobType())) {
addMigrationProcessConfig(containerComposer);
createSourceSchema(containerComposer, PipelineContainerComposer.SCHEMA_NAME);
diff --git a/test/e2e/operation/pipeline/src/test/java/org/apache/shardingsphere/test/e2e/data/pipeline/cases/migration/general/RulesMigrationE2EIT.java b/test/e2e/operation/pipeline/src/test/java/org/apache/shardingsphere/test/e2e/data/pipeline/cases/migration/general/RulesMigrationE2EIT.java
index 5be6538fd7f..57a27c7e217 100644
--- a/test/e2e/operation/pipeline/src/test/java/org/apache/shardingsphere/test/e2e/data/pipeline/cases/migration/general/RulesMigrationE2EIT.java
+++ b/test/e2e/operation/pipeline/src/test/java/org/apache/shardingsphere/test/e2e/data/pipeline/cases/migration/general/RulesMigrationE2EIT.java
@@ -44,7 +44,7 @@ import static org.hamcrest.Matchers.is;
* 2) only encrypt rule.
*/
@PipelineE2ESettings(fetchSingle = true, database = @PipelineE2EDatabaseSettings(type = "MySQL", scenarioFiles = "env/scenario/primary_key/text_primary_key/mysql.xml"))
-public final class RulesMigrationE2EIT extends AbstractMigrationE2EIT {
+class RulesMigrationE2EIT extends AbstractMigrationE2EIT {
private static final String SOURCE_TABLE_NAME = "t_order";
@@ -53,7 +53,7 @@ public final class RulesMigrationE2EIT extends AbstractMigrationE2EIT {
@ParameterizedTest(name = "{0}")
@EnabledIf("isEnabled")
@ArgumentsSource(PipelineE2ETestCaseArgumentsProvider.class)
- public void assertNoRuleMigrationSuccess(final PipelineTestParameter testParam) throws Exception {
+ void assertNoRuleMigrationSuccess(final PipelineTestParameter testParam) throws Exception {
try (PipelineContainerComposer containerComposer = new PipelineContainerComposer(testParam, new MigrationJobType())) {
assertMigrationSuccess(containerComposer, null);
}
@@ -62,7 +62,7 @@ public final class RulesMigrationE2EIT extends AbstractMigrationE2EIT {
@ParameterizedTest(name = "{0}")
@EnabledIf("isEnabled")
@ArgumentsSource(PipelineE2ETestCaseArgumentsProvider.class)
- public void assertOnlyEncryptRuleMigrationSuccess(final PipelineTestParameter testParam) throws Exception {
+ void assertOnlyEncryptRuleMigrationSuccess(final PipelineTestParameter testParam) throws Exception {
try (PipelineContainerComposer containerComposer = new PipelineContainerComposer(testParam, new MigrationJobType())) {
assertMigrationSuccess(containerComposer, () -> {
createTargetOrderTableEncryptRule(containerComposer);
diff --git a/test/e2e/operation/pipeline/src/test/java/org/apache/shardingsphere/test/e2e/data/pipeline/cases/migration/primarykey/IndexesMigrationE2EIT.java b/test/e2e/operation/pipeline/src/test/java/org/apache/shardingsphere/test/e2e/data/pipeline/cases/migration/primarykey/IndexesMigrationE2EIT.java
index f33deeef6c1..daa8812b8d9 100644
--- a/test/e2e/operation/pipeline/src/test/java/org/apache/shardingsphere/test/e2e/data/pipeline/cases/migration/primarykey/IndexesMigrationE2EIT.java
+++ b/test/e2e/operation/pipeline/src/test/java/org/apache/shardingsphere/test/e2e/data/pipeline/cases/migration/primarykey/IndexesMigrationE2EIT.java
@@ -56,7 +56,7 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
@PipelineE2ESettings(fetchSingle = true, database = {
@PipelineE2EDatabaseSettings(type = "MySQL", scenarioFiles = "env/common/none.xml"),
@PipelineE2EDatabaseSettings(type = "PostgreSQL", scenarioFiles = "env/common/none.xml")})
-public final class IndexesMigrationE2EIT extends AbstractMigrationE2EIT {
+class IndexesMigrationE2EIT extends AbstractMigrationE2EIT {
private static final String ORDER_TABLE_SHARDING_RULE_FORMAT = "CREATE SHARDING TABLE RULE t_order(\n"
+ "STORAGE_UNITS(ds_2,ds_3,ds_4),\n"
@@ -71,7 +71,7 @@ public final class IndexesMigrationE2EIT extends AbstractMigrationE2EIT {
@ParameterizedTest(name = "{0}")
@EnabledIf("isEnabled")
@ArgumentsSource(PipelineE2ETestCaseArgumentsProvider.class)
- public void assertNoUniqueKeyMigrationSuccess(final PipelineTestParameter testParam) throws Exception {
+ void assertNoUniqueKeyMigrationSuccess(final PipelineTestParameter testParam) throws Exception {
try (PipelineContainerComposer containerComposer = new PipelineContainerComposer(testParam, new MigrationJobType())) {
String sql;
String consistencyCheckAlgorithmType;
@@ -108,7 +108,7 @@ public final class IndexesMigrationE2EIT extends AbstractMigrationE2EIT {
@ParameterizedTest(name = "{0}")
@EnabledIf("isEnabled")
@ArgumentsSource(PipelineE2ETestCaseArgumentsProvider.class)
- public void assertMultiPrimaryKeyMigrationSuccess(final PipelineTestParameter testParam) throws Exception {
+ void assertMultiPrimaryKeyMigrationSuccess(final PipelineTestParameter testParam) throws Exception {
try (PipelineContainerComposer containerComposer = new PipelineContainerComposer(testParam, new MigrationJobType())) {
String sql;
String consistencyCheckAlgorithmType;
@@ -131,7 +131,7 @@ public final class IndexesMigrationE2EIT extends AbstractMigrationE2EIT {
@ParameterizedTest(name = "{0}")
@EnabledIf("isEnabled")
@ArgumentsSource(PipelineE2ETestCaseArgumentsProvider.class)
- public void assertMultiUniqueKeyMigrationSuccess(final PipelineTestParameter testParam) throws Exception {
+ void assertMultiUniqueKeyMigrationSuccess(final PipelineTestParameter testParam) throws Exception {
try (PipelineContainerComposer containerComposer = new PipelineContainerComposer(testParam, new MigrationJobType())) {
String sql;
String consistencyCheckAlgorithmType;
@@ -154,7 +154,7 @@ public final class IndexesMigrationE2EIT extends AbstractMigrationE2EIT {
@ParameterizedTest(name = "{0}")
@EnabledIf("isEnabled")
@ArgumentsSource(PipelineE2ETestCaseArgumentsProvider.class)
- public void assertSpecialTypeSingleColumnUniqueKeyMigrationSuccess(final PipelineTestParameter testParam) throws Exception {
+ void assertSpecialTypeSingleColumnUniqueKeyMigrationSuccess(final PipelineTestParameter testParam) throws Exception {
try (PipelineContainerComposer containerComposer = new PipelineContainerComposer(testParam, new MigrationJobType())) {
String sql;
String consistencyCheckAlgorithmType;
diff --git a/test/e2e/operation/pipeline/src/test/java/org/apache/shardingsphere/test/e2e/data/pipeline/cases/migration/primarykey/MariaDBMigrationE2EIT.java b/test/e2e/operation/pipeline/src/test/java/org/apache/shardingsphere/test/e2e/data/pipeline/cases/migration/primarykey/MariaDBMigrationE2EIT.java
index 40bd432a662..e726f02e360 100644
--- a/test/e2e/operation/pipeline/src/test/java/org/apache/shardingsphere/test/e2e/data/pipeline/cases/migration/primarykey/MariaDBMigrationE2EIT.java
+++ b/test/e2e/operation/pipeline/src/test/java/org/apache/shardingsphere/test/e2e/data/pipeline/cases/migration/primarykey/MariaDBMigrationE2EIT.java
@@ -44,7 +44,7 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
@PipelineE2ESettings(fetchSingle = true, database = @PipelineE2EDatabaseSettings(type = "MySQL", scenarioFiles = "env/common/none.xml"))
@Slf4j
-public final class MariaDBMigrationE2EIT extends AbstractMigrationE2EIT {
+class MariaDBMigrationE2EIT extends AbstractMigrationE2EIT {
private static final String SOURCE_TABLE_NAME = "t_order";
@@ -53,7 +53,7 @@ public final class MariaDBMigrationE2EIT extends AbstractMigrationE2EIT {
@ParameterizedTest(name = "{0}")
@EnabledIf("isEnabled")
@ArgumentsSource(PipelineE2ETestCaseArgumentsProvider.class)
- public void assertMigrationSuccess(final PipelineTestParameter testParam) throws SQLException, InterruptedException {
+ void assertMigrationSuccess(final PipelineTestParameter testParam) throws SQLException, InterruptedException {
try (PipelineContainerComposer containerComposer = new PipelineContainerComposer(testParam, new MigrationJobType())) {
String sqlPattern = "CREATE TABLE `%s` (`order_id` VARCHAR(64) NOT NULL, `user_id` INT NOT NULL, `status` varchar(255), PRIMARY KEY (`order_id`)) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4";
containerComposer.sourceExecuteWithLog(String.format(sqlPattern, SOURCE_TABLE_NAME));
diff --git a/test/e2e/operation/pipeline/src/test/java/org/apache/shardingsphere/test/e2e/data/pipeline/cases/migration/primarykey/TextPrimaryKeyMigrationE2EIT.java b/test/e2e/operation/pipeline/src/test/java/org/apache/shardingsphere/test/e2e/data/pipeline/cases/migration/primarykey/TextPrimaryKeyMigrationE2EIT.java
index df860d51323..d946a0016cd 100644
--- a/test/e2e/operation/pipeline/src/test/java/org/apache/shardingsphere/test/e2e/data/pipeline/cases/migration/primarykey/TextPrimaryKeyMigrationE2EIT.java
+++ b/test/e2e/operation/pipeline/src/test/java/org/apache/shardingsphere/test/e2e/data/pipeline/cases/migration/primarykey/TextPrimaryKeyMigrationE2EIT.java
@@ -45,14 +45,14 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
@PipelineE2EDatabaseSettings(type = "MySQL", scenarioFiles = "env/scenario/primary_key/text_primary_key/mysql.xml"),
@PipelineE2EDatabaseSettings(type = "PostgreSQL", scenarioFiles = "env/scenario/primary_key/text_primary_key/postgresql.xml"),
@PipelineE2EDatabaseSettings(type = "openGauss", scenarioFiles = "env/scenario/primary_key/text_primary_key/postgresql.xml")})
-public final class TextPrimaryKeyMigrationE2EIT extends AbstractMigrationE2EIT {
+class TextPrimaryKeyMigrationE2EIT extends AbstractMigrationE2EIT {
private static final String TARGET_TABLE_NAME = "t_order";
@ParameterizedTest(name = "{0}")
@EnabledIf("isEnabled")
@ArgumentsSource(PipelineE2ETestCaseArgumentsProvider.class)
- public void assertTextPrimaryMigrationSuccess(final PipelineTestParameter testParam) throws SQLException, InterruptedException {
+ void assertTextPrimaryMigrationSuccess(final PipelineTestParameter testParam) throws SQLException, InterruptedException {
try (PipelineContainerComposer containerComposer = new PipelineContainerComposer(testParam, new MigrationJobType())) {
containerComposer.createSourceOrderTable(getSourceTableName(containerComposer));
try (Connection connection = containerComposer.getSourceDataSource().getConnection()) {
diff --git a/test/e2e/operation/showprocesslist/src/test/java/org/apache/shardingsphere/test/e2e/showprocesslist/engine/ShowProcessListE2EIT.java b/test/e2e/operation/showprocesslist/src/test/java/org/apache/shardingsphere/test/e2e/showprocesslist/engine/ShowProcessListE2EIT.java
index 5fb8f2a79f1..6a3cf827b07 100644
--- a/test/e2e/operation/showprocesslist/src/test/java/org/apache/shardingsphere/test/e2e/showprocesslist/engine/ShowProcessListE2EIT.java
+++ b/test/e2e/operation/showprocesslist/src/test/java/org/apache/shardingsphere/test/e2e/showprocesslist/engine/ShowProcessListE2EIT.java
@@ -43,7 +43,7 @@ import static org.hamcrest.CoreMatchers.is;
import static org.hamcrest.MatcherAssert.assertThat;
// TODO add jdbc
-public final class ShowProcessListE2EIT {
+class ShowProcessListE2EIT {
private static final ShowProcessListEnvironment ENV = ShowProcessListEnvironment.getInstance();
@@ -52,7 +52,7 @@ public final class ShowProcessListE2EIT {
@ParameterizedTest(name = "{0}")
@EnabledIf("isEnabled")
@ArgumentsSource(TestCaseArgumentsProvider.class)
- public void assertShowProcessList(final ShowProcessListTestParameter testParam) throws SQLException, InterruptedException {
+ void assertShowProcessList(final ShowProcessListTestParameter testParam) throws SQLException, InterruptedException {
try (ClusterShowProcessListContainerComposer containerComposer = new ClusterShowProcessListContainerComposer(testParam)) {
containerComposer.start();
CompletableFuture<Void> executeSelectSleep = CompletableFuture.runAsync(getExecuteSleepThread("proxy", containerComposer));
diff --git a/test/e2e/operation/transaction/src/test/java/org/apache/shardingsphere/test/e2e/transaction/engine/base/TransactionBaseE2EIT.java b/test/e2e/operation/transaction/src/test/java/org/apache/shardingsphere/test/e2e/transaction/engine/base/TransactionBaseE2EIT.java
index 4b7cb3bb6e4..facdc0299f8 100644
--- a/test/e2e/operation/transaction/src/test/java/org/apache/shardingsphere/test/e2e/transaction/engine/base/TransactionBaseE2EIT.java
+++ b/test/e2e/operation/transaction/src/test/java/org/apache/shardingsphere/test/e2e/transaction/engine/base/TransactionBaseE2EIT.java
@@ -92,7 +92,7 @@ public abstract class TransactionBaseE2EIT {
@ParameterizedTest(name = "{0}")
@EnabledIf("isEnabled")
@ArgumentsSource(TestCaseArgumentsProvider.class)
- public void assertTransaction(final TransactionTestParameter testParam) throws SQLException {
+ void assertTransaction(final TransactionTestParameter testParam) throws SQLException {
if (null == testParam) {
return;
}
diff --git a/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/engine/type/DALE2EIT.java b/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/engine/type/DALE2EIT.java
index daca5b800dc..77d1122f680 100644
--- a/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/engine/type/DALE2EIT.java
+++ b/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/engine/type/DALE2EIT.java
@@ -48,12 +48,12 @@ import static org.hamcrest.MatcherAssert.assertThat;
import static org.junit.jupiter.api.Assertions.assertTrue;
@E2ETestCaseSettings(SQLCommandType.DAL)
-public final class DALE2EIT {
+class DALE2EIT {
@ParameterizedTest(name = "{0}")
@EnabledIf("isEnabled")
@ArgumentsSource(E2ETestCaseArgumentsProvider.class)
- public void assertExecute(final AssertionTestParameter testParam) throws SQLException, ParseException {
+ void assertExecute(final AssertionTestParameter testParam) throws SQLException, ParseException {
// TODO make sure test case can not be null
if (null == testParam.getTestCaseContext()) {
return;
diff --git a/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/engine/type/DCLE2EIT.java b/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/engine/type/DCLE2EIT.java
index 68ff55fe014..6568e05aca9 100644
--- a/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/engine/type/DCLE2EIT.java
+++ b/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/engine/type/DCLE2EIT.java
@@ -39,12 +39,12 @@ import java.sql.Statement;
import java.text.ParseException;
@E2ETestCaseSettings(SQLCommandType.DCL)
-public final class DCLE2EIT {
+class DCLE2EIT {
@ParameterizedTest(name = "{0}")
@EnabledIf("isEnabled")
@ArgumentsSource(E2ETestCaseArgumentsProvider.class)
- public void assertExecuteUpdate(final AssertionTestParameter testParam) throws SQLException, ParseException, JAXBException, IOException {
+ void assertExecuteUpdate(final AssertionTestParameter testParam) throws SQLException, ParseException, JAXBException, IOException {
// TODO make sure test case can not be null
if (null == testParam.getTestCaseContext()) {
return;
@@ -75,7 +75,7 @@ public final class DCLE2EIT {
@ParameterizedTest(name = "{0}")
@EnabledIf("isEnabled")
@ArgumentsSource(E2ETestCaseArgumentsProvider.class)
- public void assertExecute(final AssertionTestParameter testParam) throws SQLException, ParseException, JAXBException, IOException {
+ void assertExecute(final AssertionTestParameter testParam) throws SQLException, ParseException, JAXBException, IOException {
// TODO make sure test case can not be null
if (null == testParam.getTestCaseContext()) {
return;
diff --git a/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/engine/type/DDLE2EIT.java b/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/engine/type/DDLE2EIT.java
index 719320d1dee..7eddcd645c0 100644
--- a/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/engine/type/DDLE2EIT.java
+++ b/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/engine/type/DDLE2EIT.java
@@ -55,12 +55,12 @@ import static org.junit.jupiter.api.Assertions.assertFalse;
import static org.junit.jupiter.api.Assertions.assertNotNull;
@E2ETestCaseSettings(SQLCommandType.DDL)
-public final class DDLE2EIT {
+class DDLE2EIT {
@ParameterizedTest(name = "{0}")
@EnabledIf("isEnabled")
@ArgumentsSource(E2ETestCaseArgumentsProvider.class)
- public void assertExecuteUpdate(final AssertionTestParameter testParam) throws SQLException, ParseException {
+ void assertExecuteUpdate(final AssertionTestParameter testParam) throws SQLException, ParseException {
// TODO make sure test case can not be null
if (null == testParam.getTestCaseContext()) {
return;
@@ -96,7 +96,7 @@ public final class DDLE2EIT {
@ParameterizedTest(name = "{0}")
@EnabledIf("isEnabled")
@ArgumentsSource(E2ETestCaseArgumentsProvider.class)
- public void assertExecute(final AssertionTestParameter testParam) throws Exception {
+ void assertExecute(final AssertionTestParameter testParam) throws Exception {
// TODO make sure test case can not be null
if (null == testParam.getTestCaseContext()) {
return;
diff --git a/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/engine/type/RALE2EIT.java b/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/engine/type/RALE2EIT.java
index 8d1d69f3750..d631d5f2d90 100644
--- a/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/engine/type/RALE2EIT.java
+++ b/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/engine/type/RALE2EIT.java
@@ -49,12 +49,12 @@ import static org.hamcrest.MatcherAssert.assertThat;
import static org.junit.jupiter.api.Assertions.assertTrue;
@E2ETestCaseSettings(SQLCommandType.RAL)
-public final class RALE2EIT {
+class RALE2EIT {
@ParameterizedTest(name = "{0}")
@EnabledIf("isEnabled")
@ArgumentsSource(E2ETestCaseArgumentsProvider.class)
- public void assertExecute(final AssertionTestParameter testParam) throws SQLException, ParseException {
+ void assertExecute(final AssertionTestParameter testParam) throws SQLException, ParseException {
// TODO make sure test case can not be null
if (null == testParam.getTestCaseContext()) {
return;
diff --git a/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/engine/type/RDLE2EIT.java b/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/engine/type/RDLE2EIT.java
index 68f1b3c310c..865d043f11f 100644
--- a/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/engine/type/RDLE2EIT.java
+++ b/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/engine/type/RDLE2EIT.java
@@ -49,12 +49,12 @@ import static org.junit.jupiter.api.Assertions.assertNotNull;
import static org.junit.jupiter.api.Assertions.assertTrue;
@E2ETestCaseSettings(SQLCommandType.RDL)
-public final class RDLE2EIT {
+class RDLE2EIT {
@ParameterizedTest(name = "{0}")
@EnabledIf("isEnabled")
@ArgumentsSource(E2ETestCaseArgumentsProvider.class)
- public void assertExecute(final AssertionTestParameter testParam) throws SQLException, ParseException {
+ void assertExecute(final AssertionTestParameter testParam) throws SQLException, ParseException {
// TODO make sure test case can not be null
if (null == testParam.getTestCaseContext()) {
return;
diff --git a/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/engine/type/RQLE2EIT.java b/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/engine/type/RQLE2EIT.java
index 53e132d77c5..08c2efe9790 100644
--- a/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/engine/type/RQLE2EIT.java
+++ b/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/engine/type/RQLE2EIT.java
@@ -45,12 +45,12 @@ import static org.hamcrest.MatcherAssert.assertThat;
import static org.junit.jupiter.api.Assertions.assertTrue;
@E2ETestCaseSettings(SQLCommandType.RQL)
-public final class RQLE2EIT {
+class RQLE2EIT {
@ParameterizedTest(name = "{0}")
@EnabledIf("isEnabled")
@ArgumentsSource(E2ETestCaseArgumentsProvider.class)
- public void assertExecute(final AssertionTestParameter testParam) throws SQLException, ParseException {
+ void assertExecute(final AssertionTestParameter testParam) throws SQLException, ParseException {
// TODO make sure test case can not be null
if (null == testParam.getTestCaseContext()) {
return;
diff --git a/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/engine/type/dml/AdditionalDMLE2EIT.java b/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/engine/type/dml/AdditionalDMLE2EIT.java
index 4b555b3eb87..019ba119bc8 100644
--- a/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/engine/type/dml/AdditionalDMLE2EIT.java
+++ b/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/engine/type/dml/AdditionalDMLE2EIT.java
@@ -42,12 +42,12 @@ import java.text.ParseException;
import static org.junit.jupiter.api.Assertions.assertFalse;
@E2ETestCaseSettings(SQLCommandType.DML)
-public final class AdditionalDMLE2EIT extends BaseDMLE2EIT {
+class AdditionalDMLE2EIT extends BaseDMLE2EIT {
@ParameterizedTest(name = "{0}")
@EnabledIf("isEnabled")
@ArgumentsSource(E2ETestCaseArgumentsProvider.class)
- public void executeUpdateWithAutoGeneratedKeys(final AssertionTestParameter testParam) throws SQLException, ParseException, JAXBException, IOException {
+ void assertExecuteUpdateWithAutoGeneratedKeys(final AssertionTestParameter testParam) throws SQLException, ParseException, JAXBException, IOException {
// TODO make sure test case can not be null
if (null == testParam.getTestCaseContext()) {
return;
@@ -97,7 +97,7 @@ public final class AdditionalDMLE2EIT extends BaseDMLE2EIT {
@ParameterizedTest(name = "{0}")
@EnabledIf("isEnabled")
@ArgumentsSource(E2ETestCaseArgumentsProvider.class)
- public void assertExecuteUpdateWithColumnIndexes(final AssertionTestParameter testParam) throws SQLException, ParseException, JAXBException, IOException {
+ void assertExecuteUpdateWithColumnIndexes(final AssertionTestParameter testParam) throws SQLException, ParseException, JAXBException, IOException {
if (isPostgreSQLOrOpenGauss(testParam.getDatabaseType().getType())) {
return;
}
@@ -133,7 +133,7 @@ public final class AdditionalDMLE2EIT extends BaseDMLE2EIT {
@ParameterizedTest(name = "{0}")
@EnabledIf("isEnabled")
@ArgumentsSource(E2ETestCaseArgumentsProvider.class)
- public void assertExecuteUpdateWithColumnNames(final AssertionTestParameter testParam) throws SQLException, ParseException, JAXBException, IOException {
+ void assertExecuteUpdateWithColumnNames(final AssertionTestParameter testParam) throws SQLException, ParseException, JAXBException, IOException {
if (isPostgreSQLOrOpenGauss(testParam.getDatabaseType().getType())) {
return;
}
@@ -167,7 +167,7 @@ public final class AdditionalDMLE2EIT extends BaseDMLE2EIT {
@ParameterizedTest(name = "{0}")
@EnabledIf("isEnabled")
@ArgumentsSource(E2ETestCaseArgumentsProvider.class)
- public void assertExecuteWithoutAutoGeneratedKeys(final AssertionTestParameter testParam) throws SQLException, ParseException, JAXBException, IOException {
+ void assertExecuteWithoutAutoGeneratedKeys(final AssertionTestParameter testParam) throws SQLException, ParseException, JAXBException, IOException {
// TODO make sure test case can not be null
if (null == testParam.getTestCaseContext()) {
return;
@@ -207,7 +207,7 @@ public final class AdditionalDMLE2EIT extends BaseDMLE2EIT {
@ParameterizedTest(name = "{0}")
@EnabledIf("isEnabled")
@ArgumentsSource(E2ETestCaseArgumentsProvider.class)
- public void assertExecuteWithAutoGeneratedKeys(final AssertionTestParameter testParam) throws SQLException, ParseException, JAXBException, IOException {
+ void assertExecuteWithAutoGeneratedKeys(final AssertionTestParameter testParam) throws SQLException, ParseException, JAXBException, IOException {
// TODO make sure test case can not be null
if (null == testParam.getTestCaseContext()) {
return;
@@ -254,7 +254,7 @@ public final class AdditionalDMLE2EIT extends BaseDMLE2EIT {
@ParameterizedTest(name = "{0}")
@EnabledIf("isEnabled")
@ArgumentsSource(E2ETestCaseArgumentsProvider.class)
- public void assertExecuteWithColumnIndexes(final AssertionTestParameter testParam) throws SQLException, ParseException, JAXBException, IOException {
+ void assertExecuteWithColumnIndexes(final AssertionTestParameter testParam) throws SQLException, ParseException, JAXBException, IOException {
if (isPostgreSQLOrOpenGauss(testParam.getDatabaseType().getType())) {
return;
}
@@ -292,7 +292,7 @@ public final class AdditionalDMLE2EIT extends BaseDMLE2EIT {
@ParameterizedTest(name = "{0}")
@EnabledIf("isEnabled")
@ArgumentsSource(E2ETestCaseArgumentsProvider.class)
- public void assertExecuteWithColumnNames(final AssertionTestParameter testParam) throws SQLException, ParseException, JAXBException, IOException {
+ void assertExecuteWithColumnNames(final AssertionTestParameter testParam) throws SQLException, ParseException, JAXBException, IOException {
if (isPostgreSQLOrOpenGauss(testParam.getDatabaseType().getType())) {
return;
}
diff --git a/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/engine/type/dml/BatchDMLE2EIT.java b/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/engine/type/dml/BatchDMLE2EIT.java
index 1fc22443dee..d1645447f10 100644
--- a/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/engine/type/dml/BatchDMLE2EIT.java
+++ b/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/engine/type/dml/BatchDMLE2EIT.java
@@ -40,12 +40,12 @@ import static org.hamcrest.CoreMatchers.is;
import static org.hamcrest.MatcherAssert.assertThat;
@E2ETestCaseSettings(value = SQLCommandType.DML, batch = true)
-public final class BatchDMLE2EIT {
+class BatchDMLE2EIT {
@ParameterizedTest(name = "{0}")
@EnabledIf("isEnabled")
@ArgumentsSource(E2ETestCaseArgumentsProvider.class)
- public void assertExecuteBatch(final CaseTestParameter testParam) throws SQLException, ParseException, JAXBException, IOException {
+ void assertExecuteBatch(final CaseTestParameter testParam) throws SQLException, ParseException, JAXBException, IOException {
// TODO make sure test case can not be null
if (null == testParam.getTestCaseContext()) {
return;
@@ -78,7 +78,7 @@ public final class BatchDMLE2EIT {
@ParameterizedTest(name = "{0}")
@EnabledIf("isEnabled")
@ArgumentsSource(E2ETestCaseArgumentsProvider.class)
- public void assertClearBatch(final CaseTestParameter testParam) throws SQLException, ParseException, JAXBException, IOException {
+ void assertClearBatch(final CaseTestParameter testParam) throws SQLException, ParseException, JAXBException, IOException {
// TODO make sure test case can not be null
if (null == testParam.getTestCaseContext()) {
return;
diff --git a/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/engine/type/dml/GeneralDMLE2EIT.java b/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/engine/type/dml/GeneralDMLE2EIT.java
index a5bab81f952..3dbf5fef291 100644
--- a/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/engine/type/dml/GeneralDMLE2EIT.java
+++ b/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/engine/type/dml/GeneralDMLE2EIT.java
@@ -40,12 +40,12 @@ import java.text.ParseException;
import static org.junit.jupiter.api.Assertions.assertFalse;
@E2ETestCaseSettings(SQLCommandType.DML)
-public final class GeneralDMLE2EIT extends BaseDMLE2EIT {
+class GeneralDMLE2EIT extends BaseDMLE2EIT {
@ParameterizedTest(name = "{0}")
@EnabledIf("isEnabled")
@ArgumentsSource(E2ETestCaseArgumentsProvider.class)
- public void assertExecuteUpdate(final AssertionTestParameter testParam) throws SQLException, ParseException, JAXBException, IOException {
+ void assertExecuteUpdate(final AssertionTestParameter testParam) throws SQLException, ParseException, JAXBException, IOException {
// TODO make sure test case can not be null
if (null == testParam.getTestCaseContext()) {
return;
@@ -80,7 +80,7 @@ public final class GeneralDMLE2EIT extends BaseDMLE2EIT {
@ParameterizedTest(name = "{0}")
@EnabledIf("isEnabled")
@ArgumentsSource(E2ETestCaseArgumentsProvider.class)
- public void assertExecute(final AssertionTestParameter testParam) throws SQLException, ParseException, JAXBException, IOException {
+ void assertExecute(final AssertionTestParameter testParam) throws SQLException, ParseException, JAXBException, IOException {
// TODO make sure test case can not be null
if (null == testParam.getTestCaseContext()) {
return;
diff --git a/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/engine/type/dql/AdditionalDQLE2EIT.java b/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/engine/type/dql/AdditionalDQLE2EIT.java
index 0ae7c252dc2..074b6493221 100644
--- a/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/engine/type/dql/AdditionalDQLE2EIT.java
+++ b/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/engine/type/dql/AdditionalDQLE2EIT.java
@@ -42,12 +42,12 @@ import java.text.ParseException;
import static org.junit.jupiter.api.Assertions.assertTrue;
@E2ETestCaseSettings(SQLCommandType.DQL)
-public final class AdditionalDQLE2EIT extends BaseDQLE2EIT {
+class AdditionalDQLE2EIT extends BaseDQLE2EIT {
@ParameterizedTest(name = "{0}")
@EnabledIf("isEnabled")
@ArgumentsSource(E2ETestCaseArgumentsProvider.class)
- public void assertExecuteQueryWithResultSetTypeAndConcurrency(final AssertionTestParameter testParam) throws SQLException, ParseException, JAXBException, IOException {
+ void assertExecuteQueryWithResultSetTypeAndConcurrency(final AssertionTestParameter testParam) throws SQLException, ParseException, JAXBException, IOException {
// TODO make sure test case can not be null
if (null == testParam.getTestCaseContext()) {
return;
@@ -69,7 +69,7 @@ public final class AdditionalDQLE2EIT extends BaseDQLE2EIT {
@ParameterizedTest(name = "{0}")
@EnabledIf("isEnabled")
@ArgumentsSource(E2ETestCaseArgumentsProvider.class)
- public void assertExecuteQueryWithResultSetTypeAndConcurrencyAndHoldability(final AssertionTestParameter testParam) throws SQLException, ParseException, JAXBException, IOException {
+ void assertExecuteQueryWithResultSetTypeAndConcurrencyAndHoldability(final AssertionTestParameter testParam) throws SQLException, ParseException, JAXBException, IOException {
// TODO make sure test case can not be null
if (null == testParam.getTestCaseContext()) {
return;
@@ -91,7 +91,7 @@ public final class AdditionalDQLE2EIT extends BaseDQLE2EIT {
@ParameterizedTest(name = "{0}")
@EnabledIf("isEnabled")
@ArgumentsSource(E2ETestCaseArgumentsProvider.class)
- public void assertExecuteWithResultSetTypeAndConcurrency(final AssertionTestParameter testParam) throws SQLException, ParseException, JAXBException, IOException {
+ void assertExecuteWithResultSetTypeAndConcurrency(final AssertionTestParameter testParam) throws SQLException, ParseException, JAXBException, IOException {
// TODO make sure test case can not be null
if (null == testParam.getTestCaseContext()) {
return;
@@ -113,7 +113,7 @@ public final class AdditionalDQLE2EIT extends BaseDQLE2EIT {
@ParameterizedTest(name = "{0}")
@EnabledIf("isEnabled")
@ArgumentsSource(E2ETestCaseArgumentsProvider.class)
- public void assertExecuteWithResultSetTypeAndConcurrencyAndHoldability(final AssertionTestParameter testParam) throws SQLException, ParseException, JAXBException, IOException {
+ void assertExecuteWithResultSetTypeAndConcurrencyAndHoldability(final AssertionTestParameter testParam) throws SQLException, ParseException, JAXBException, IOException {
// TODO make sure test case can not be null
if (null == testParam.getTestCaseContext()) {
return;
diff --git a/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/engine/type/dql/GeneralDQLE2EIT.java b/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/engine/type/dql/GeneralDQLE2EIT.java
index 5907ac04290..e9a327d0eab 100644
--- a/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/engine/type/dql/GeneralDQLE2EIT.java
+++ b/test/e2e/sql/src/test/java/org/apache/shardingsphere/test/e2e/engine/type/dql/GeneralDQLE2EIT.java
@@ -41,12 +41,12 @@ import java.text.ParseException;
import static org.junit.jupiter.api.Assertions.assertTrue;
@E2ETestCaseSettings(SQLCommandType.DQL)
-public final class GeneralDQLE2EIT extends BaseDQLE2EIT {
+class GeneralDQLE2EIT extends BaseDQLE2EIT {
@ParameterizedTest(name = "{0}")
@EnabledIf("isEnabled")
@ArgumentsSource(E2ETestCaseArgumentsProvider.class)
- public void assertExecuteQuery(final AssertionTestParameter testParam) throws SQLException, ParseException, IOException, JAXBException {
+ void assertExecuteQuery(final AssertionTestParameter testParam) throws SQLException, ParseException, IOException, JAXBException {
// TODO make sure test case can not be null
if (null == testParam.getTestCaseContext()) {
return;
@@ -118,7 +118,7 @@ public final class GeneralDQLE2EIT extends BaseDQLE2EIT {
@ParameterizedTest(name = "{0}")
@EnabledIf("isEnabled")
@ArgumentsSource(E2ETestCaseArgumentsProvider.class)
- public void assertExecute(final AssertionTestParameter testParam) throws SQLException, ParseException, JAXBException, IOException {
+ void assertExecute(final AssertionTestParameter testParam) throws SQLException, ParseException, JAXBException, IOException {
// TODO make sure test case can not be null
if (null == testParam.getTestCaseContext()) {
return;
diff --git a/test/it/optimizer/src/test/java/org/apache/shardingsphere/test/it/optimize/SQLNodeConverterEngineIT.java b/test/it/optimizer/src/test/java/org/apache/shardingsphere/test/it/optimize/SQLNodeConverterEngineIT.java
index aa7d21fefa0..954d930de75 100644
--- a/test/it/optimizer/src/test/java/org/apache/shardingsphere/test/it/optimize/SQLNodeConverterEngineIT.java
+++ b/test/it/optimizer/src/test/java/org/apache/shardingsphere/test/it/optimize/SQLNodeConverterEngineIT.java
@@ -58,7 +58,7 @@ import java.util.stream.Stream;
import static org.junit.jupiter.api.Assertions.assertTrue;
-public final class SQLNodeConverterEngineIT {
+class SQLNodeConverterEngineIT {
private static final SQLCases SQL_CASES = SQLCasesRegistry.getInstance().getCases();
@@ -68,7 +68,7 @@ public final class SQLNodeConverterEngineIT {
@ParameterizedTest(name = "{0}")
@ArgumentsSource(TestCaseArgumentsProvider.class)
- public void assertConvert(final String sqlCaseId, final SQLCaseType sqlCaseType, final String databaseType) {
+ void assertConvert(final String sqlCaseId, final SQLCaseType sqlCaseType, final String databaseType) {
String sql = SQL_CASES.getSQL(sqlCaseId, sqlCaseType, SQL_PARSER_TEST_CASES.get(sqlCaseId).getParameters());
SqlNode actual = SQLNodeConverterEngine.convert(parseSQLStatement(databaseType, sql));
SqlNode expected = parseSQLNode(databaseType, sql);
diff --git a/test/it/parser/src/test/java/org/apache/shardingsphere/test/it/sql/parser/internal/InternalDistSQLParserIT.java b/test/it/parser/src/test/java/org/apache/shardingsphere/test/it/sql/parser/internal/InternalDistSQLParserIT.java
index 6abd0153fe1..c49f516009b 100644
--- a/test/it/parser/src/test/java/org/apache/shardingsphere/test/it/sql/parser/internal/InternalDistSQLParserIT.java
+++ b/test/it/parser/src/test/java/org/apache/shardingsphere/test/it/sql/parser/internal/InternalDistSQLParserIT.java
@@ -18,5 +18,5 @@
package org.apache.shardingsphere.test.it.sql.parser.internal;
@InternalSQLParserITSettings("ShardingSphere")
-public final class InternalDistSQLParserIT extends InternalSQLParserIT {
+class InternalDistSQLParserIT extends InternalSQLParserIT {
}
diff --git a/test/it/rewriter/src/test/java/org/apache/shardingsphere/test/it/rewrite/engine/scenario/EncryptSQLRewriterIT.java b/test/it/rewriter/src/test/java/org/apache/shardingsphere/test/it/rewrite/engine/scenario/EncryptSQLRewriterIT.java
index 9b2415c74cf..96a84a89b9e 100644
--- a/test/it/rewriter/src/test/java/org/apache/shardingsphere/test/it/rewrite/engine/scenario/EncryptSQLRewriterIT.java
+++ b/test/it/rewriter/src/test/java/org/apache/shardingsphere/test/it/rewrite/engine/scenario/EncryptSQLRewriterIT.java
@@ -52,7 +52,7 @@ import static org.mockito.Mockito.mock;
import static org.mockito.Mockito.when;
@SQLRewriterITSettings("scenario/encrypt/case")
-public final class EncryptSQLRewriterIT extends SQLRewriterIT {
+class EncryptSQLRewriterIT extends SQLRewriterIT {
@Override
protected YamlRootConfiguration createRootConfiguration(final SQLRewriteEngineTestParameters testParams) throws IOException {
diff --git a/test/it/rewriter/src/test/java/org/apache/shardingsphere/test/it/rewrite/engine/scenario/MixSQLRewriterIT.java b/test/it/rewriter/src/test/java/org/apache/shardingsphere/test/it/rewrite/engine/scenario/MixSQLRewriterIT.java
index 52ea31d8512..e4bf022cd58 100644
--- a/test/it/rewriter/src/test/java/org/apache/shardingsphere/test/it/rewrite/engine/scenario/MixSQLRewriterIT.java
+++ b/test/it/rewriter/src/test/java/org/apache/shardingsphere/test/it/rewrite/engine/scenario/MixSQLRewriterIT.java
@@ -45,7 +45,7 @@ import static org.mockito.Mockito.mock;
import static org.mockito.Mockito.when;
@SQLRewriterITSettings("scenario/mix/case")
-public final class MixSQLRewriterIT extends SQLRewriterIT {
+class MixSQLRewriterIT extends SQLRewriterIT {
@Override
protected YamlRootConfiguration createRootConfiguration(final SQLRewriteEngineTestParameters testParams) throws IOException {
diff --git a/test/it/rewriter/src/test/java/org/apache/shardingsphere/test/it/rewrite/engine/scenario/ShardingSQLRewriterIT.java b/test/it/rewriter/src/test/java/org/apache/shardingsphere/test/it/rewrite/engine/scenario/ShardingSQLRewriterIT.java
index d597c62c8c8..4d2085deb8b 100644
--- a/test/it/rewriter/src/test/java/org/apache/shardingsphere/test/it/rewrite/engine/scenario/ShardingSQLRewriterIT.java
+++ b/test/it/rewriter/src/test/java/org/apache/shardingsphere/test/it/rewrite/engine/scenario/ShardingSQLRewriterIT.java
@@ -49,7 +49,7 @@ import static org.mockito.Mockito.mock;
import static org.mockito.Mockito.when;
@SQLRewriterITSettings("scenario/sharding/case")
-public final class ShardingSQLRewriterIT extends SQLRewriterIT {
+class ShardingSQLRewriterIT extends SQLRewriterIT {
@Override
protected YamlRootConfiguration createRootConfiguration(final SQLRewriteEngineTestParameters testParams) throws IOException {