Skip to content

Commit 03ed702

Browse files
committed
#482: fixed issue with invalid recipients coming from Outlook parser
1 parent f2003c1 commit 03ed702

File tree

3 files changed

+13
-2
lines changed

3 files changed

+13
-2
lines changed

modules/outlook-module/pom.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@
3535
<dependency>
3636
<groupId>org.simplejavamail</groupId>
3737
<artifactId>outlook-message-parser</artifactId>
38-
<version>1.11.1</version>
38+
<version>1.12.0</version>
3939
<scope>compile</scope>
4040
</dependency>
4141
</dependencies>

modules/simple-java-mail/src/test/java/org/simplejavamail/converter/EmailConverterTest.java

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,6 @@
3131
import static org.assertj.core.api.Assertions.assertThat;
3232
import static org.simplejavamail.api.email.ContentTransferEncoding.BIT7;
3333
import static org.simplejavamail.internal.util.MiscUtil.normalizeNewlines;
34-
import static org.simplejavamail.internal.util.Preconditions.assumeTrue;
3534

3635
public class EmailConverterTest {
3736

@@ -55,6 +54,18 @@ public void testOutlookBasicConversions() {
5554
assertThat(normalizeNewlines(msg.getPlainText())).isEqualTo("Just a test to get an email with one cc recipient.\n");
5655
}
5756

57+
@Test
58+
public void testOutlookBasicConversionsGithubIssue482() {
59+
final Recipient ramonFrom = new Recipient("Boss Ramon", "[email protected]", null);
60+
final Recipient ramonTo = new Recipient("Boss Ramon", "[email protected]", TO);
61+
62+
@NotNull Email msg = EmailConverter.outlookMsgToEmail(new File(RESOURCE_TEST_MESSAGES + "/#482 emailAddressList_is_required.msg"));
63+
EmailAssert.assertThat(msg).hasFromRecipient(ramonFrom);
64+
EmailAssert.assertThat(msg).hasSubject("subj");
65+
EmailAssert.assertThat(msg).hasOnlyRecipients(ramonTo);
66+
EmailAssert.assertThat(msg).hasNoAttachments();
67+
}
68+
5869
@Test
5970
public void testOutlookUnicode() {
6071
final Recipient kalejs = new Recipient("[email protected]", "[email protected]", null);

0 commit comments

Comments
 (0)