From 8fe21b5009a6ed3bca94c0e7df38b39b59ccafb3 Mon Sep 17 00:00:00 2001
From: Tatsiana Luchonak <Tatsiana_Luchonak@epam.com>
Date: Tue, 20 Aug 2019 15:57:29 +0300
Subject: [PATCH] Update arguments parsing from expressions/ add tests to
 increase coverage

---
 .../github/javafaker/service/FakeValuesService.java    | 10 +++++++---
 src/test/java/com/github/javafaker/LoremTest.java      |  9 ++++++++-
 .../javafaker/service/FakeValuesServiceTest.java       |  7 +++++++
 3 files changed, 22 insertions(+), 4 deletions(-)

diff --git a/src/main/java/com/github/javafaker/service/FakeValuesService.java b/src/main/java/com/github/javafaker/service/FakeValuesService.java
index fc9fbf2ad..02c2041c5 100644
--- a/src/main/java/com/github/javafaker/service/FakeValuesService.java
+++ b/src/main/java/com/github/javafaker/service/FakeValuesService.java
@@ -22,7 +22,9 @@
 import java.util.regex.Pattern;
 
 public class FakeValuesService {
-    private static final Pattern EXPRESSION_PATTERN = Pattern.compile("#\\{([a-z0-9A-Z_.]+)\\s?(?:'([^']+)')?(?:,'([^']+)')*\\}");
+
+    private static final Pattern EXPRESSION_PATTERN = Pattern.compile("#\\{([a-z0-9A-Z_.]+)\\s?((?:,?'([^']+)')*)\\}");
+    private static final Pattern EXPRESSION_ARGUMENTS_PATTERN = Pattern.compile("(?:'(.*?)')");
 
     private final Logger log = Logger.getLogger("faker");
 
@@ -345,9 +347,11 @@ protected String resolveExpression(String expression, Object current, Faker root
         while (matcher.find()) {
             final String escapedDirective = matcher.group(0);
             final String directive = matcher.group(1);
+            final String arguments = matcher.group(2);
+            final Matcher argsMatcher = EXPRESSION_ARGUMENTS_PATTERN.matcher(arguments);
             List<String> args = new ArrayList<String>();
-            for (int i = 2; i < matcher.groupCount() + 1 && matcher.group(i) != null; i++) {
-                args.add(matcher.group(i));
+            while (argsMatcher.find()) {
+                args.add(argsMatcher.group(1));
             }
 
             // resolve the expression and reprocess it to handle recursive templates
diff --git a/src/test/java/com/github/javafaker/LoremTest.java b/src/test/java/com/github/javafaker/LoremTest.java
index 4899b6eae..8a5688900 100644
--- a/src/test/java/com/github/javafaker/LoremTest.java
+++ b/src/test/java/com/github/javafaker/LoremTest.java
@@ -1,12 +1,15 @@
 package com.github.javafaker;
 
 import static com.github.javafaker.matchers.MatchesRegularExpression.matchesRegularExpression;
+import static org.hamcrest.Matchers.greaterThanOrEqualTo;
+import static org.hamcrest.Matchers.hasSize;
 import static org.hamcrest.Matchers.isEmptyOrNullString;
 import static org.hamcrest.Matchers.isEmptyString;
 import static org.hamcrest.Matchers.not;
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertThat;
 
+import org.hamcrest.Matchers;
 import org.junit.Test;
 
 public class LoremTest extends AbstractFakerTest {
@@ -71,7 +74,7 @@ public void testCharactersMinimumMaximumLength() {
 
     @Test
     public void testCharactersMinimumMaximumLengthIncludeUppercase() {
-        assertThat(faker.lorem().characters(1, 10), matchesRegularExpression("[a-zA-Z\\d]{1,10}"));
+        assertThat(faker.lorem().characters(1, 10, true), matchesRegularExpression("[a-zA-Z\\d]{1,10}"));
     }
 
     @Test
@@ -100,4 +103,8 @@ public void testSentenceFixedNumberOfWords() {
         assertThat(faker.lorem().sentence(10, 0), matchesRegularExpression("(\\w+\\s?){10}\\."));
     }
 
+    @Test
+    public void testWords() {
+        assertThat(faker.lorem().words(), hasSize(greaterThanOrEqualTo(1)));
+    }
 }
diff --git a/src/test/java/com/github/javafaker/service/FakeValuesServiceTest.java b/src/test/java/com/github/javafaker/service/FakeValuesServiceTest.java
index 5b9e68bb3..cfc470ace 100644
--- a/src/test/java/com/github/javafaker/service/FakeValuesServiceTest.java
+++ b/src/test/java/com/github/javafaker/service/FakeValuesServiceTest.java
@@ -258,6 +258,13 @@ public void pastDateExpression() throws ParseException {
         assertThat( date.getTime(), lessThan( now.getTime() ));
     }
 
+    @Test
+    public void expressionWithFourArguments() throws ParseException {
+
+        assertThat(fakeValuesService.expression("#{Internet.password '5','8','true','true'}", faker),
+            matchesRegularExpression("[\\w\\d\\!%#$@_\\^&\\*]{5,8}"));
+    }
+
     /**
      * Two things are important here:
      * 1) the message in the exception should be USEFUL