Skip to content

Commit dfa5f10

Browse files
[refactor] Standardize tests (darrachequesne#66)
1 parent ab6464f commit dfa5f10

39 files changed

+326
-1925
lines changed

pom.xml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -158,6 +158,13 @@
158158
<scope>test</scope>
159159
</dependency>
160160

161+
<dependency>
162+
<groupId>org.assertj</groupId>
163+
<artifactId>assertj-core</artifactId>
164+
<version>2.8.0</version>
165+
<scope>test</scope>
166+
</dependency>
167+
161168
<dependency>
162169
<groupId>org.springframework</groupId>
163170
<artifactId>spring-test</artifactId>

src/main/java/org/springframework/data/jpa/datatables/ColumnFilter.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,7 @@ public javax.persistence.criteria.Predicate createPredicate(From<?, ?> from, Cri
8181

8282
javax.persistence.criteria.Predicate predicate;
8383
if (isBooleanComparison) {
84-
predicate = expression.as(Boolean.class).in(booleanValues);
84+
predicate = expression.in(booleanValues);
8585
} else {
8686
predicate = expression.as(String.class).in(values);
8787
}

src/test/java/org/springframework/data/jpa/datatables/Config.java

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -76,13 +76,6 @@ public AbstractEntityManagerFactoryBean entityManagerFactory(DataSource dataSour
7676
bean.setPackagesToScan(Config.class.getPackage().getName());
7777
bean.setDataSource(dataSource);
7878

79-
Properties jpaProperties = new Properties();
80-
jpaProperties.setProperty(Environment.HBM2DDL_AUTO, "create-drop");
81-
jpaProperties.setProperty(Environment.HBM2DDL_IMPORT_FILES, "init.sql");
82-
jpaProperties.setProperty(Environment.HBM2DDL_IMPORT_FILES_SQL_EXTRACTOR,
83-
MultipleLinesSqlCommandExtractor.class.getName());
84-
bean.setJpaProperties(jpaProperties);
85-
8679
return bean;
8780
}
8881

src/test/java/org/springframework/data/jpa/datatables/model/Bill.java

Lines changed: 0 additions & 54 deletions
This file was deleted.

src/test/java/org/springframework/data/jpa/datatables/model/BillRepository.java

Lines changed: 0 additions & 7 deletions
This file was deleted.

src/test/java/org/springframework/data/jpa/datatables/model/Course.java

Lines changed: 0 additions & 47 deletions
This file was deleted.

src/test/java/org/springframework/data/jpa/datatables/model/CourseType.java

Lines changed: 0 additions & 32 deletions
This file was deleted.

src/test/java/org/springframework/data/jpa/datatables/model/Employee.java

Lines changed: 30 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,9 @@
77
import lombok.experimental.Tolerate;
88

99
import javax.persistence.*;
10+
import java.util.List;
11+
12+
import static java.util.Arrays.asList;
1013

1114
@Data
1215
@Setter(AccessLevel.NONE)
@@ -19,6 +22,8 @@ public class Employee {
1922
private String lastName;
2023
private String position;
2124
private int age;
25+
private boolean isWorkingRemotely;
26+
private String comment;
2227

2328
@ManyToOne(cascade = CascadeType.ALL)
2429
@JoinColumn(name = "id_office")
@@ -34,6 +39,7 @@ private Employee() {}
3439
.position("Accountant")
3540
.age(33)
3641
.office(Office.TOKYO)
42+
.comment(null)
3743
.build();
3844

3945
public static Employee ANGELICA_RAMOS = Employee.builder()
@@ -43,6 +49,7 @@ private Employee() {}
4349
.position("Chief Executive Officer (CEO)")
4450
.age(47)
4551
.office(Office.LONDON)
52+
.comment("\\NULL")
4653
.build();
4754

4855
public static Employee ASHTON_COX = Employee.builder()
@@ -52,6 +59,8 @@ private Employee() {}
5259
.position("Junior Technical Author")
5360
.age(66)
5461
.office(Office.SAN_FRANCISCO)
62+
.isWorkingRemotely(true)
63+
.comment("~foo~~")
5564
.build();
5665

5766
public static Employee BRADLEY_GREER = Employee.builder()
@@ -61,6 +70,7 @@ private Employee() {}
6170
.position("Software Engineer")
6271
.age(41)
6372
.office(Office.LONDON)
73+
.comment("%foo%%")
6474
.build();
6575

6676
public static Employee BRENDEN_WAGNER = Employee.builder()
@@ -70,6 +80,7 @@ private Employee() {}
7080
.position("Software Engineer")
7181
.age(28)
7282
.office(Office.SAN_FRANCISCO)
83+
.comment("_foo__")
7384
.build();
7485

7586
public static Employee BRIELLE_WILLIAMSON = Employee.builder()
@@ -79,6 +90,25 @@ private Employee() {}
7990
.position("Integration Specialist")
8091
.age(61)
8192
.office(Office.NEW_YORK)
93+
.comment("@foo@@")
8294
.build();
8395

96+
public static List<Employee> ALL = asList(
97+
AIRI_SATOU,
98+
ANGELICA_RAMOS,
99+
ASHTON_COX,
100+
BRADLEY_GREER,
101+
BRENDEN_WAGNER,
102+
BRIELLE_WILLIAMSON
103+
);
104+
105+
public static List<Employee> ALL_SORTED_BY_AGE = asList(
106+
BRENDEN_WAGNER,
107+
AIRI_SATOU,
108+
BRADLEY_GREER,
109+
ANGELICA_RAMOS,
110+
BRIELLE_WILLIAMSON,
111+
ASHTON_COX
112+
);
113+
84114
}
Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
package org.springframework.data.jpa.datatables.model;
2+
3+
import lombok.Value;
4+
5+
@Value
6+
public class EmployeeDto {
7+
private int id;
8+
private String firstName;
9+
private String lastName;
10+
11+
public static EmployeeDto AIRI_SATOU = new EmployeeDto(5407, "Airi", "Satou");
12+
13+
}

src/test/java/org/springframework/data/jpa/datatables/model/Game.java

Lines changed: 0 additions & 33 deletions
This file was deleted.

0 commit comments

Comments
 (0)