Skip to content

Commit a64c25a

Browse files
committed
Remove duplicated addParameterMarkers while using SQLStatementCopyUtils.copyAttributes(...) and add unit tests for associated situations (fix spotless)
1 parent ef0ee8f commit a64c25a

File tree

1 file changed

+8
-9
lines changed

1 file changed

+8
-9
lines changed

infra/binder/core/src/test/java/org/apache/shardingsphere/infra/binder/engine/statement/SQLStatementCopyUtilsTest.java

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -38,14 +38,14 @@
3838
* Test for SQLStatementCopyUtils.
3939
*/
4040
class SQLStatementCopyUtilsTest {
41-
41+
4242
public static class MockedDatabaseType implements DatabaseType {
43-
43+
4444
@Override
4545
public Collection<String> getJdbcUrlPrefixes() {
4646
return Collections.singleton("jdbc:mock");
4747
}
48-
48+
4949
@Override
5050
public String getType() {
5151
return "MOCKED";
@@ -201,12 +201,11 @@ void assertCopyAttributesWithNullCollections() {
201201
void assertCopyAttributesWithLargeCollections() {
202202
// Test with large collections
203203
Collection<ParameterMarkerSegment> largeParamList = Arrays.asList(
204-
new ParameterMarkerExpressionSegment(0, 0, 1),
205-
new ParameterMarkerExpressionSegment(1, 1, 2),
206-
new ParameterMarkerExpressionSegment(2, 2, 3),
207-
new ParameterMarkerExpressionSegment(3, 3, 4),
208-
new ParameterMarkerExpressionSegment(4, 4, 5)
209-
);
204+
new ParameterMarkerExpressionSegment(0, 0, 1),
205+
new ParameterMarkerExpressionSegment(1, 1, 2),
206+
new ParameterMarkerExpressionSegment(2, 2, 3),
207+
new ParameterMarkerExpressionSegment(3, 3, 4),
208+
new ParameterMarkerExpressionSegment(4, 4, 5));
210209
originalStatement.addParameterMarkers(largeParamList);
211210

212211
for (int i = 0; i < 100; i++) {

0 commit comments

Comments
 (0)