Ignore commas at the end of links, too

diff --git a/src/main/java/net/pterodactylus/sone/text/SoneTextParser.java b/src/main/java/net/pterodactylus/sone/text/SoneTextParser.java
index 6752c12..dbfa0f3 100644 (file)
--- a/src/main/java/net/pterodactylus/sone/text/SoneTextParser.java
+++ b/src/main/java/net/pterodactylus/sone/text/SoneTextParser.java
@@ -321,7 +321,7 @@ public class SoneTextParser implements Parser<SoneTextParserContext> {
        }
 
        private boolean isPunctuation(char character) {
        }
 
        private boolean isPunctuation(char character) {

-               return character == '.';
+               return (character == '.') || (character == ',');

        }
 
        private static class NextLink {
        }
 
        private static class NextLink {

diff --git a/src/test/java/net/pterodactylus/sone/text/SoneTextParserTest.java b/src/test/java/net/pterodactylus/sone/text/SoneTextParserTest.java
index fb66f86..1161b8d 100644 (file)
--- a/src/test/java/net/pterodactylus/sone/text/SoneTextParserTest.java
+++ b/src/test/java/net/pterodactylus/sone/text/SoneTextParserTest.java
@@ -140,6 +140,14 @@ public class SoneTextParserTest {
                assertThat("Part Text", "Some text and a link: [http://example.sone/abc|example.sone/abc|example.sone/abc]... Nice!", is(convertText(parts, PlainTextPart.class, LinkPart.class)));
        }
 
                assertThat("Part Text", "Some text and a link: [http://example.sone/abc|example.sone/abc|example.sone/abc]... Nice!", is(convertText(parts, PlainTextPart.class, LinkPart.class)));
        }
 

+       @Test
+       public void commasAreIgnoredAtEndOfLinkBeforeWhitespace() {
+               SoneTextParser soneTextParser = new SoneTextParser(null, null);
+               Iterable<Part> parts = soneTextParser.parse("Some text and a link: http://example.sone/abc, nice!", null);
+               assertThat("Parts", parts, notNullValue());
+               assertThat("Part Text", "Some text and a link: [http://example.sone/abc|example.sone/abc|example.sone/abc], nice!", is(convertText(parts, PlainTextPart.class, LinkPart.class)));
+       }
+

        /**
         * Converts all given {@link Part}s into a string, validating that the
         * part’s classes match only the expected classes.
        /**
         * Converts all given {@link Part}s into a string, validating that the
         * part’s classes match only the expected classes.