Skip to content

Commit c9f1b85

Browse files
test: improve and fix
1 parent 35cb76d commit c9f1b85

File tree

6 files changed

+20
-7
lines changed

6 files changed

+20
-7
lines changed

acceptance-test/src/test/java/br/com/helpdev/atdd/ApplicationIT.java

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,6 @@ public void whenDeleteMessageThenRemove() {
7070
.statusCode(201)
7171
.extract()
7272
.path("id");
73-
System.out.println("Id: " + id);
7473
given()
7574
.when()
7675
.delete("/v1/message/{id}", id)
@@ -130,7 +129,6 @@ public void whenCreateNewSmsScheduleThenReturn() {
130129
.body(buildMessage(ZonedDateTime.now(), CommunicationChannelDto.SMS))
131130
.when()
132131
.post("/v1/message/")
133-
.prettyPeek()
134132
.then()
135133
.statusCode(201)
136134
.body("id", notNullValue())

adapter/input/jaxrs-controller-v1/pom.xml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17,9 +17,9 @@
1717
<maven.compiler.target>11</maven.compiler.target>
1818
<swagger-core.version>2.1.6</swagger-core.version>
1919
<swagger-maven-plugin.version>2.1.5</swagger-maven-plugin.version>
20-
<coverage.line>0</coverage.line>
21-
<coverage.branch>0</coverage.branch>
22-
<mutation.threshold>0</mutation.threshold>
20+
<coverage.line>1</coverage.line>
21+
<coverage.branch>1</coverage.branch>
22+
<mutation.threshold>100</mutation.threshold>
2323
</properties>
2424

2525
<dependencies>

adapter/input/jaxrs-controller-v1/src/main/java/br/com/helpdev/controller/mapper/ControllerMessageMapper.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -61,15 +61,15 @@ private CommunicationChannelDto parse(final CommunicationChannel value) {
6161
}
6262

6363
private Recipient parse(final RecipientDto recipient) {
64-
return recipient == null ? null : Recipient.builder()
64+
return Recipient.builder()
6565
.email(recipient.getEmail())
6666
.name(recipient.getName())
6767
.phone(Phone.from(recipient.getPhoneId(), recipient.getPhoneNumber()))
6868
.build();
6969
}
7070

7171
private RecipientDto parse(final Recipient recipient) {
72-
return recipient == null ? null : RecipientDto.builder()
72+
return RecipientDto.builder()
7373
.email(recipient.getEmail())
7474
.name(recipient.getName())
7575
.phoneId(recipient.getPhone().getPhoneId().orElse(null))
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
package br.com.helpdev.controller.handler;
2+
3+
import static org.junit.jupiter.api.Assertions.*;
4+
5+
class MethodArgumentNotValidExceptionMapperTest {
6+
7+
}
Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
package br.com.helpdev.controller.handler;
2+
3+
import static org.junit.jupiter.api.Assertions.*;
4+
5+
class NotFoundExceptionMapperTest {
6+
7+
}

adapter/input/jaxrs-controller-v1/src/test/java/br/com/helpdev/controller/mapper/ControllerMessageMapperTest.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -322,4 +322,5 @@ public void whenHasRecipientPhoneNumberInCreateDtoThenConvertToEntity() {
322322
assertThat(entity.getRecipient().getPhone().getPhoneNumber()).isEqualTo("999988887777");
323323
}
324324

325+
325326
}

0 commit comments

Comments
 (0)