Skip to content

Commit e3c94bb

Browse files
committed
feat: add validation of null at cassandra
Signed-off-by: Otavio Santana <[email protected]>
1 parent bd58c0f commit e3c94bb

File tree

2 files changed

+16
-14
lines changed

2 files changed

+16
-14
lines changed

jnosql-cassandra/src/main/java/org/eclipse/jnosql/databases/cassandra/mapping/CassandraUDTType.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,9 @@ public boolean isNotEmpty() {
6060

6161
@Override
6262
public List<Column> toColumn(ColumnEntityConverter converter, Converters converters) {
63-
if (Iterable.class.isInstance(value)) {
63+
if (value == null) {
64+
return singletonList(Column.of(field.name(), null));
65+
} else if (Iterable.class.isInstance(value)) {
6466
List<Iterable<Column>> columns = new ArrayList<>();
6567
stream(Iterable.class.cast(value).spliterator(), false)
6668
.forEach(c -> columns.add(converter.toColumn(c).columns()));

jnosql-cassandra/src/test/java/org/eclipse/jnosql/databases/cassandra/mapping/DefaultCassandraTemplateTest.java

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -80,17 +80,17 @@ public class DefaultCassandraTemplateTest {
8080
private CassandraColumnManager manager;
8181

8282
@BeforeEach
83-
public void setUp() {
83+
void setUp() {
8484
this.manager = mock(CassandraColumnManager.class);
8585
Instance instance = mock(Instance.class);
8686
when(instance.get()).thenReturn(manager);
8787
template = new DefaultCassandraTemplate(instance, converter, persistManager, entities, converters);
8888
}
8989

9090
@Test
91-
public void shouldSaveConsistency() {
91+
void shouldSaveConsistency() {
9292
ColumnEntity entity = ColumnEntity.of("Person", asList(Column.of("name", "Name"), Column.of("age", 20)));
93-
93+
entity.addNull("home");
9494
ArgumentCaptor<ColumnEntity> captor = ArgumentCaptor.forClass(ColumnEntity.class);
9595

9696
ConsistencyLevel level = ConsistencyLevel.THREE;
@@ -110,9 +110,9 @@ public void shouldSaveConsistency() {
110110
}
111111

112112
@Test
113-
public void shouldSaveConsistencyIterable() {
113+
void shouldSaveConsistencyIterable() {
114114
ColumnEntity entity = ColumnEntity.of("Person", asList(Column.of("name", "Name"), Column.of("age", 20)));
115-
115+
entity.addNull("home");
116116
ArgumentCaptor<ColumnEntity> captor = ArgumentCaptor.forClass(ColumnEntity.class);
117117

118118
ConsistencyLevel level = ConsistencyLevel.THREE;
@@ -131,10 +131,10 @@ public void shouldSaveConsistencyIterable() {
131131
}
132132

133133
@Test
134-
public void shouldSaveConsntencyDuration() {
134+
void shouldSaveConsntencyDuration() {
135135
Duration duration = Duration.ofHours(2);
136136
ColumnEntity entity = ColumnEntity.of("Person", asList(Column.of("name", "Name"), Column.of("age", 20)));
137-
137+
entity.addNull("home");
138138
ArgumentCaptor<ColumnEntity> captor = ArgumentCaptor.forClass(ColumnEntity.class);
139139

140140
ConsistencyLevel level = ConsistencyLevel.THREE;
@@ -153,10 +153,10 @@ public void shouldSaveConsntencyDuration() {
153153
}
154154

155155
@Test
156-
public void shouldSaveConsntencyDurationIterable() {
156+
void shouldSaveConsistencyDurationIterable() {
157157
Duration duration = Duration.ofHours(2);
158158
ColumnEntity entity = ColumnEntity.of("Person", asList(Column.of("name", "Name"), Column.of("age", 20)));
159-
159+
entity.addNull("home");
160160
ArgumentCaptor<ColumnEntity> captor = ArgumentCaptor.forClass(ColumnEntity.class);
161161

162162
ConsistencyLevel level = ConsistencyLevel.THREE;
@@ -174,7 +174,7 @@ public void shouldSaveConsntencyDurationIterable() {
174174
}
175175

176176
@Test
177-
public void shouldDelete() {
177+
void shouldDelete() {
178178

179179

180180
ColumnDeleteQuery query = ColumnDeleteQuery.delete().from("columnFamily").build();
@@ -185,7 +185,7 @@ public void shouldDelete() {
185185

186186

187187
@Test
188-
public void shouldFind() {
188+
void shouldFind() {
189189
Person person = new Person();
190190
person.setName("Name");
191191
person.setAge(20);
@@ -200,7 +200,7 @@ public void shouldFind() {
200200
}
201201

202202
@Test
203-
public void shouldFindCQL() {
203+
void shouldFindCQL() {
204204
Person person = new Person();
205205
person.setName("Name");
206206
person.setAge(20);
@@ -214,7 +214,7 @@ public void shouldFindCQL() {
214214
}
215215

216216
@Test
217-
public void shouldFindSimpleStatement() {
217+
void shouldFindSimpleStatement() {
218218
SimpleStatement statement = QueryBuilder.selectFrom("Person").all().build();
219219
Person person = new Person();
220220
person.setName("Name");

0 commit comments

Comments
 (0)