Skip to content

Commit cef0bb5

Browse files
updates javadocs, adds SentMessageStatus
1 parent bf7614e commit cef0bb5

5 files changed

Lines changed: 26 additions & 12 deletions

File tree

src/main/java/com/mailjet/client/transactional/SendContact.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ public class SendContact {
44
/**
55
* Represents an object with email and name (optional) that can be used as To, Cc, Bcc, From etc
66
* @param email email address
7-
* @param name the display name of the person (Will be displayed in email like NAME < EMAIL >
7+
* @param name the display name of the person (Will be displayed in email like {@code NAME <EMAIL>}
88
*/
99
public SendContact(String email, String name) {
1010
this(email);

src/main/java/com/mailjet/client/transactional/response/EmailResult.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@ public class EmailResult {
1616

1717
/**
1818
* Unique numeric ID of this message.
19+
* With this ID, you can get the message information through /messages/ API endpoint
1920
*/
2021
private long messageID;
2122

src/main/java/com/mailjet/client/transactional/response/MessageResult.java

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -9,16 +9,14 @@ public class MessageResult {
99

1010
/**
1111
* Indicates the sending status of the message.
12-
* Possible values:
13-
* success
14-
* error
1512
*/
16-
private String status;
13+
private SentMessageStatus status;
1714

1815
/**
1916
* List containing information about any errors with the processing of this message.
20-
* Will be displayed only if processing errors occur.
17+
* Will be filled in only if processing errors occur.
2118
* Each error will generate a separate error object with the below properties.
19+
* null if the message sent successfully
2220
*/
2321
private SendEmailError[] errors;
2422

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
package com.mailjet.client.transactional.response;
2+
3+
import com.google.gson.annotations.SerializedName;
4+
5+
public enum SentMessageStatus {
6+
@SerializedName("success")
7+
SUCCESS,
8+
9+
@SerializedName("error")
10+
ERROR
11+
}

src/test/java/com/mailjet/client/TransactionalEmailBuilderIT.java

Lines changed: 10 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,9 @@
33
import com.mailjet.client.errors.MailjetException;
44
import com.mailjet.client.transactional.*;
55
import com.mailjet.client.transactional.response.MessageResult;
6+
import com.mailjet.client.transactional.response.SendEmailError;
67
import com.mailjet.client.transactional.response.SendEmailsResponse;
8+
import com.mailjet.client.transactional.response.SentMessageStatus;
79
import org.junit.Assert;
810
import org.junit.Before;
911
import org.junit.Test;
@@ -56,7 +58,7 @@ public void SendEmailsRequest_SendsMessage() throws MailjetException, IOExceptio
5658

5759
MessageResult messageResult = response.getMessages()[0];
5860

59-
Assert.assertEquals("success", messageResult.getStatus());
61+
Assert.assertEquals(SentMessageStatus.SUCCESS, messageResult.getStatus());
6062
Assert.assertEquals("custom-id-value", messageResult.getCustomID());
6163
}
6264

@@ -84,10 +86,12 @@ public void SendEmailsRequest_InvalidToField_ReturnsAnError() throws MailjetExce
8486

8587
MessageResult messageResult = response.getMessages()[0];
8688

87-
Assert.assertEquals("error", messageResult.getStatus());
88-
Assert.assertEquals("mj-0013", messageResult.getErrors()[0].getErrorCode());
89-
Assert.assertEquals(400, messageResult.getErrors()[0].getStatusCode());
90-
Assert.assertEquals("\"invalid-email\" is an invalid email address.", messageResult.getErrors()[0].getErrorMessage());
91-
Assert.assertEquals("To[0].Email", messageResult.getErrors()[0].getErrorRelatedTo()[0]);
89+
Assert.assertEquals(SentMessageStatus.ERROR, messageResult.getStatus());
90+
SendEmailError error = messageResult.getErrors()[0];
91+
92+
Assert.assertEquals("mj-0013", error.getErrorCode());
93+
Assert.assertEquals(400, error.getStatusCode());
94+
Assert.assertEquals("\"invalid-email\" is an invalid email address.", error.getErrorMessage());
95+
Assert.assertEquals("To[0].Email", error.getErrorRelatedTo()[0]);
9296
}
9397
}

0 commit comments

Comments
 (0)