Merge pull request #24 from denghuafeng/master

update: parse function type variable substringBeforeLast change subst…
fivesmallq · Mar 12, 2018 · f8fdd82 · f8fdd82
2 parents 7d70d64 + efe7386
commit f8fdd82
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/src/main/java/im/nll/data/extractor/parser/ExtractorParser.java b/src/main/java/im/nll/data/extractor/parser/ExtractorParser.java
@@ -48,7 +48,7 @@ public class ExtractorParser {
     }
 
     public static Extractor parse(String shortString) {
-        String type = StringUtils.substringBeforeLast(shortString, SPLIT_CHAR);
+        String type = StringUtils.substringBefore(shortString, SPLIT_CHAR);
         Class extractorClass = extractorMap.get(type);
         if (extractorClass == null) {
             throw new ParseException("parse extractor error! unsupport extractor:'" + type + "'. string:'" + shortString + "'");

diff --git a/src/test/java/im/nll/data/extractor/parser/ExtractorParserTest.java b/src/test/java/im/nll/data/extractor/parser/ExtractorParserTest.java
@@ -63,4 +63,10 @@ public void testParseError() throws Exception {
         String string = "json2:$..books";
         ExtractorParser.parse(string);
     }
+
+    @Test
+    public void testStringRangeParse() throws Exception {
+        String string = "stringRange:<li style=\"background: rgba(0, 0, 0, 0) none repeat scroll 0% 0%;\">,</li>";
+        ExtractorParser.parse(string);
+    }
 }