Skip to content

Commit 66e3ad9

Browse files
committed
DATAJDBC-349 - Polishing.
Original pull requests: #248, #242. Fixes superfluous commata and unsupported data types. Cleaning up database objects before (re)creating them.
1 parent e26f664 commit 66e3ad9

8 files changed

+42
-15
lines changed

spring-data-jdbc/src/test/java/org/springframework/data/jdbc/repository/JdbcRepositoryPropertyConversionIntegrationTests.java

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -197,15 +197,16 @@ static class EntityWithColumnsRequiringConversions {
197197
Date date;
198198
LocalDateTime localDateTime;
199199
// ensures conversion on id querying
200-
@Id private LocalDateTime idTimestamp;
200+
@Id
201+
private LocalDateTime idTimestamp;
201202

202-
@MappedCollection(idColumn = "ID_TIMESTAMP") Set<EntityWithColumnsRequiringConversionsRelation> relation;
203+
@MappedCollection(idColumn = "ID_TIMESTAMP")
204+
Set<EntityWithColumnsRequiringConversionsRelation> relation;
203205
}
204206

205207
// DATAJDBC-349
206208
@Data
207209
static class EntityWithColumnsRequiringConversionsRelation {
208-
@Id private LocalDateTime idTimestamp;
209210
String data;
210211
}
211212
}

spring-data-jdbc/src/test/resources/org.springframework.data.jdbc.core/JdbcAggregateTemplateIntegrationTests-mssql.sql

Lines changed: 24 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -49,14 +49,18 @@ CREATE TABLE BYTE_ARRAY_OWNER
4949
BINARY_DATA VARBINARY(20) NOT NULL
5050
);
5151

52+
DROP TABLE IF EXISTS CHAIN0;
53+
DROP TABLE IF EXISTS CHAIN1;
54+
DROP TABLE IF EXISTS CHAIN2;
55+
DROP TABLE IF EXISTS CHAIN3;
5256
DROP TABLE IF EXISTS CHAIN4;
57+
5358
CREATE TABLE CHAIN4
5459
(
5560
FOUR BIGINT IDENTITY PRIMARY KEY,
5661
FOUR_VALUE VARCHAR(20)
5762
);
5863

59-
DROP TABLE IF EXISTS CHAIN3;
6064
CREATE TABLE CHAIN3
6165
(
6266
THREE BIGINT IDENTITY PRIMARY KEY,
@@ -65,7 +69,6 @@ CREATE TABLE CHAIN3
6569
FOREIGN KEY (CHAIN4) REFERENCES CHAIN4 (FOUR)
6670
);
6771

68-
DROP TABLE IF EXISTS CHAIN2;
6972
CREATE TABLE CHAIN2
7073
(
7174
TWO BIGINT IDENTITY PRIMARY KEY,
@@ -74,7 +77,6 @@ CREATE TABLE CHAIN2
7477
FOREIGN KEY (CHAIN3) REFERENCES CHAIN3 (THREE)
7578
);
7679

77-
DROP TABLE IF EXISTS CHAIN1;
7880
CREATE TABLE CHAIN1
7981
(
8082
ONE BIGINT IDENTITY PRIMARY KEY,
@@ -83,7 +85,6 @@ CREATE TABLE CHAIN1
8385
FOREIGN KEY (CHAIN2) REFERENCES CHAIN2 (TWO)
8486
);
8587

86-
DROP TABLE IF EXISTS CHAIN0;
8788
CREATE TABLE CHAIN0
8889
(
8990
ZERO BIGINT IDENTITY PRIMARY KEY,
@@ -92,6 +93,12 @@ CREATE TABLE CHAIN0
9293
FOREIGN KEY (CHAIN1) REFERENCES CHAIN1 (ONE)
9394
);
9495

96+
DROP TABLE IF EXISTS NO_ID_CHAIN0;
97+
DROP TABLE IF EXISTS NO_ID_CHAIN1;
98+
DROP TABLE IF EXISTS NO_ID_CHAIN2;
99+
DROP TABLE IF EXISTS NO_ID_CHAIN3;
100+
DROP TABLE IF EXISTS NO_ID_CHAIN4;
101+
95102
CREATE TABLE NO_ID_CHAIN4
96103
(
97104
FOUR BIGINT IDENTITY PRIMARY KEY,
@@ -126,6 +133,12 @@ CREATE TABLE NO_ID_CHAIN0
126133
FOREIGN KEY (NO_ID_CHAIN4) REFERENCES NO_ID_CHAIN4 (FOUR)
127134
);
128135

136+
DROP TABLE IF EXISTS NO_ID_LIST_CHAIN0;
137+
DROP TABLE IF EXISTS NO_ID_LIST_CHAIN1;
138+
DROP TABLE IF EXISTS NO_ID_LIST_CHAIN2;
139+
DROP TABLE IF EXISTS NO_ID_LIST_CHAIN3;
140+
DROP TABLE IF EXISTS NO_ID_LIST_CHAIN4;
141+
129142
CREATE TABLE NO_ID_LIST_CHAIN4
130143
(
131144
FOUR BIGINT IDENTITY PRIMARY KEY,
@@ -208,8 +221,11 @@ CREATE TABLE NO_ID_LIST_CHAIN0
208221
)
209222
);
210223

211-
212-
224+
DROP TABLE IF EXISTS NO_ID_MAP_CHAIN0;
225+
DROP TABLE IF EXISTS NO_ID_MAP_CHAIN1;
226+
DROP TABLE IF EXISTS NO_ID_MAP_CHAIN2;
227+
DROP TABLE IF EXISTS NO_ID_MAP_CHAIN3;
228+
DROP TABLE IF EXISTS NO_ID_MAP_CHAIN4;
213229

214230
CREATE TABLE NO_ID_MAP_CHAIN4
215231
(
@@ -293,6 +309,8 @@ CREATE TABLE NO_ID_MAP_CHAIN0
293309
)
294310
);
295311

312+
DROP TABLE IF EXISTS VERSIONED_AGGREGATE;
313+
296314
CREATE TABLE VERSIONED_AGGREGATE
297315
(
298316
ID BIGINT IDENTITY PRIMARY KEY,

spring-data-jdbc/src/test/resources/org.springframework.data.jdbc.core/JdbcAggregateTemplateSchemaIntegrationTests-mssql.sql

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,6 @@
1+
DROP TABLE IF EXISTS OTHER;
2+
DROP SCHEMA IF EXISTS OTHER;
3+
14
CREATE SCHEMA OTHER;
25

36
CREATE TABLE OTHER.DUMMY_ENTITY
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,5 @@
1-
CREATE TABLE ENTITY_WITH_STRINGY_BIG_DECIMAL ( id BIGINT IDENTITY PRIMARY KEY, Stringy_number DECIMAL(20,10))
1+
DROP TABLE OTHER_ENTITY;
2+
DROP TABLE ENTITY_WITH_STRINGY_BIG_DECIMAL;
3+
4+
CREATE TABLE ENTITY_WITH_STRINGY_BIG_DECIMAL ( id BIGINT IDENTITY PRIMARY KEY, Stringy_number DECIMAL(20,10));
25
CREATE TABLE OTHER_ENTITY ( ID BIGINT IDENTITY PRIMARY KEY, CREATED DATE, ENTITY_WITH_STRINGY_BIG_DECIMAL INTEGER);

spring-data-jdbc/src/test/resources/org.springframework.data.jdbc.repository/JdbcRepositoryPropertyConversionIntegrationTests-db2.sql

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,5 +14,5 @@ CREATE TABLE ENTITY_WITH_COLUMNS_REQUIRING_CONVERSIONS (
1414

1515
CREATE TABLE ENTITY_WITH_COLUMNS_REQUIRING_CONVERSIONS_RELATION (
1616
id_Timestamp DATETIME NOT NULL PRIMARY KEY,
17-
data VARCHAR(100),
17+
data VARCHAR(100)
1818
);

spring-data-jdbc/src/test/resources/org.springframework.data.jdbc.repository/JdbcRepositoryPropertyConversionIntegrationTests-oracle.sql

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,5 +14,5 @@ CREATE TABLE ENTITY_WITH_COLUMNS_REQUIRING_CONVERSIONS (
1414

1515
CREATE TABLE ENTITY_WITH_COLUMNS_REQUIRING_CONVERSIONS_RELATION (
1616
ID_TIMESTAMP TIMESTAMP PRIMARY KEY,
17-
DATA VARCHAR2(100),
17+
DATA VARCHAR2(100)
1818
);
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
DROP TABLE ENTITY_WITH_COLUMNS_REQUIRING_CONVERSIONS_RELATION;
22
DROP TABLE ENTITY_WITH_COLUMNS_REQUIRING_CONVERSIONS;
3-
CREATE TABLE ENTITY_WITH_COLUMNS_REQUIRING_CONVERSIONS ( id_Timestamp TIMESTAMP PRIMARY KEY, bool boolean, SOME_ENUM VARCHAR(100), big_Decimal DECIMAL(65), big_Integer BIGINT, date TIMESTAMP, local_Date_Time TIMESTAMP, zoned_Date_Time VARCHAR(30))
4-
CREATE TABLE ENTITY_WITH_COLUMNS_REQUIRING_CONVERSIONS_RELATION ( id_Timestamp DATETIME NOT NULL PRIMARY KEY, data VARCHAR(100));
3+
4+
CREATE TABLE ENTITY_WITH_COLUMNS_REQUIRING_CONVERSIONS ( id_Timestamp TIMESTAMP PRIMARY KEY, bool boolean, SOME_ENUM VARCHAR(100), big_Decimal DECIMAL(65), big_Integer BIGINT, date TIMESTAMP, local_Date_Time TIMESTAMP, zoned_Date_Time VARCHAR(30));
5+
CREATE TABLE ENTITY_WITH_COLUMNS_REQUIRING_CONVERSIONS_RELATION ( "ID_TIMESTAMP" TIMESTAMP PRIMARY KEY, data VARCHAR(100));
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
1-
DROP TABLE IF EXISTS person;
21
DROP TABLE IF EXISTS address;
2+
DROP TABLE IF EXISTS person;
3+
34
CREATE TABLE person ( id int IDENTITY(1,1) PRIMARY KEY, name VARCHAR(100));
45
CREATE TABLE address ( id int IDENTITY(1,1) PRIMARY KEY, street VARCHAR(100), person_id INT);
56
ALTER TABLE address ADD FOREIGN KEY (person_id) REFERENCES person(id);

0 commit comments

Comments
 (0)