This is an automated email from the ASF dual-hosted git repository.

jamesbognar pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/juneau.git


The following commit(s) were added to refs/heads/master by this push:
     new c5d7dad72 RequestPathParams should extend from List.
c5d7dad72 is described below

commit c5d7dad72561a354824cea10504175d223e3110c
Author: JamesBognar <[email protected]>
AuthorDate: Tue Aug 16 14:21:52 2022 -0400

    RequestPathParams should extend from List.
---
 .../java/org/apache/juneau/rest/arg/PathArg.java   |   2 +-
 .../juneau/rest/httppart/RequestFormParams.java    |   6 +
 .../juneau/rest/httppart/RequestPathParams.java    | 281 +++++++++++----------
 .../juneau/rest/httppart/RequestQueryParams.java   |   6 +
 .../apache/juneau/rest/annotation/Path_Test.java   |  50 ++--
 5 files changed, 187 insertions(+), 158 deletions(-)

diff --git 
a/juneau-rest/juneau-rest-server/src/main/java/org/apache/juneau/rest/arg/PathArg.java
 
b/juneau-rest/juneau-rest-server/src/main/java/org/apache/juneau/rest/arg/PathArg.java
index 69b80d384..227bd2d1c 100644
--- 
a/juneau-rest/juneau-rest-server/src/main/java/org/apache/juneau/rest/arg/PathArg.java
+++ 
b/juneau-rest/juneau-rest-server/src/main/java/org/apache/juneau/rest/arg/PathArg.java
@@ -116,7 +116,7 @@ public class PathArg implements RestOpArg {
                RestRequest req = opSession.getRequest();
                if (name.equals("*")) {
                        JsonMap m = new JsonMap();
-                       req.getPathParams().getAll().stream().forEach(x -> 
m.put(x.getName(), x.getValue()));
+                       req.getPathParams().stream().forEach(x -> 
m.put(x.getName(), x.getValue()));
                        return req.getBeanSession().convertToType(m, type);
                }
                HttpPartParserSession ps = partParser == null ? 
req.getPartParserSession() : partParser.getPartSession();
diff --git 
a/juneau-rest/juneau-rest-server/src/main/java/org/apache/juneau/rest/httppart/RequestFormParams.java
 
b/juneau-rest/juneau-rest-server/src/main/java/org/apache/juneau/rest/httppart/RequestFormParams.java
index 62e97d7ce..217d084af 100644
--- 
a/juneau-rest/juneau-rest-server/src/main/java/org/apache/juneau/rest/httppart/RequestFormParams.java
+++ 
b/juneau-rest/juneau-rest-server/src/main/java/org/apache/juneau/rest/httppart/RequestFormParams.java
@@ -554,6 +554,12 @@ public class RequestFormParams extends 
ArrayList<RequestFormParam> {
                return new RequestFormParams(this);
        }
 
+       private boolean eq(String s1, String s2) {
+               if (caseSensitive)
+                       return StringUtils.eq(s1, s2);
+               return StringUtils.eqic(s1, s2);
+       }
+
        @Override /* Object */
        public String toString() {
                JsonMap m = new JsonMap();
diff --git 
a/juneau-rest/juneau-rest-server/src/main/java/org/apache/juneau/rest/httppart/RequestPathParams.java
 
b/juneau-rest/juneau-rest-server/src/main/java/org/apache/juneau/rest/httppart/RequestPathParams.java
index 19e223b4a..b152e5d8b 100644
--- 
a/juneau-rest/juneau-rest-server/src/main/java/org/apache/juneau/rest/httppart/RequestPathParams.java
+++ 
b/juneau-rest/juneau-rest-server/src/main/java/org/apache/juneau/rest/httppart/RequestPathParams.java
@@ -16,9 +16,11 @@ import static org.apache.juneau.internal.ArgUtils.*;
 import static org.apache.juneau.internal.ClassUtils.*;
 import static org.apache.juneau.internal.CollectionUtils.*;
 import static org.apache.juneau.internal.StringUtils.*;
+import static java.util.stream.Collectors.*;
 import static org.apache.juneau.httppart.HttpPartType.*;
 
 import java.util.*;
+import java.util.stream.*;
 
 import org.apache.http.*;
 import org.apache.juneau.httppart.*;
@@ -29,6 +31,7 @@ import org.apache.juneau.svl.*;
 import org.apache.juneau.*;
 import org.apache.juneau.collections.*;
 import org.apache.juneau.http.*;
+import org.apache.juneau.http.part.*;
 
 /**
  * Represents the path parameters in an HTTP request.
@@ -63,11 +66,10 @@ import org.apache.juneau.http.*;
  *     <ul class='spaced-list'>
  *             <li>Methods for retrieving path parameters:
  *             <ul class='javatreec'>
- *                     <li class='jm'>{@link 
RequestPathParams#contains(String...) contains(String...)}
+ *                     <li class='jm'>{@link 
RequestPathParams#contains(String) contains(String)}
  *                     <li class='jm'>{@link 
RequestPathParams#containsAny(String...) containsAny(String...)}
  *                     <li class='jm'>{@link RequestPathParams#get(Class) 
get(Class)}
  *                     <li class='jm'>{@link RequestPathParams#get(String) 
get(String)}
- *                     <li class='jm'>{@link RequestPathParams#getAll() 
getAll()}
  *                     <li class='jm'>{@link RequestPathParams#getAll(String) 
getAll(String)}
  *                     <li class='jm'>{@link 
RequestPathParams#getFirst(String) getFirst(String)}
  *                     <li class='jm'>{@link RequestPathParams#getLast(String) 
getLast(String)}
@@ -80,8 +82,7 @@ import org.apache.juneau.http.*;
  *                     <li class='jm'>{@link 
RequestPathParams#add(String,Object) add(String,Object)}
  *                     <li class='jm'>{@link 
RequestPathParams#addDefault(List) addDefault(List)}
  *                     <li class='jm'>{@link 
RequestPathParams#addDefault(NameValuePair...) addDefault(NameValuePair...)}
- *                     <li class='jm'>{@link 
RequestPathParams#remove(NameValuePair...) remove(NameValuePair...)}
- *                     <li class='jm'>{@link 
RequestPathParams#remove(String...) remove(String...)}
+ *                     <li class='jm'>{@link RequestPathParams#remove(String) 
remove(String)}
  *                     <li class='jm'>{@link 
RequestPathParams#set(NameValuePair...) set(NameValuePair...)}
  *                     <li class='jm'>{@link 
RequestPathParams#set(String,Object) set(String,Object)}
  *             </ul>
@@ -100,17 +101,16 @@ import org.apache.juneau.http.*;
  *     <li class='extlink'>{@source}
  * </ul>
 */
-public class RequestPathParams {
+public class RequestPathParams extends ArrayList<RequestPathParam> {
+
+       private static final long serialVersionUID = 1L;
 
        private final RestSession session;
        private final RestRequest req;
-       private final boolean caseSensitive;
+       private boolean caseSensitive;
        private HttpPartParserSession parser;
        private final VarResolverSession vs;
 
-       private List<RequestPathParam> list = new LinkedList<>();
-       private Map<String,List<RequestPathParam>> map = new TreeMap<>();
-
        /**
         * Constructor.
         *
@@ -150,11 +150,23 @@ public class RequestPathParams {
                req = copyFrom.req;
                caseSensitive = copyFrom.caseSensitive;
                parser = copyFrom.parser;
-               list.addAll(copyFrom.list);
-               map.putAll(copyFrom.map);
+               addAll(copyFrom);
                vs = copyFrom.vs;
        }
 
+       /**
+        * Subset constructor.
+        */
+       private RequestPathParams(RequestPathParams copyFrom, String...names) {
+               this.req = copyFrom.req;
+               this.session = copyFrom.session;
+               caseSensitive = copyFrom.caseSensitive;
+               parser = copyFrom.parser;
+               vs = copyFrom.vs;
+               for (String n : names)
+                       copyFrom.stream().filter(x -> eq(x.getName(), 
n)).forEach(x -> add(x));
+       }
+
        /**
         * Sets the parser to use for part values.
         *
@@ -163,8 +175,18 @@ public class RequestPathParams {
         */
        public RequestPathParams parser(HttpPartParserSession value) {
                this.parser = value;
-               for (RequestPathParam p : list)
-                       p.parser(parser);
+               forEach(x -> x.parser(parser));
+               return this;
+       }
+
+       /**
+        * Sets case sensitivity for names in this list.
+        *
+        * @param value The new value for this setting.
+        * @return This object (for method chaining).
+        */
+       public RequestPathParams caseSensitive(boolean value) {
+               this.caseSensitive = value;
                return this;
        }
 
@@ -186,15 +208,11 @@ public class RequestPathParams {
        public RequestPathParams addDefault(List<NameValuePair> pairs) {
                for (NameValuePair p : pairs) {
                        String name = p.getName();
-                       String key = key(name);
-                       List<RequestPathParam> l = map.get(key);
-                       boolean hasAllBlanks = l != null && 
l.stream().allMatch(x -> StringUtils.isEmpty(x.getValue()));
-                       if (l == null || hasAllBlanks) {
-                               if (hasAllBlanks)
-                                       list.removeAll(l);
-                               RequestPathParam x = new RequestPathParam(req, 
name, vs.resolve(p.getValue()));
-                               list.add(x);
-                               map.put(key, list(x));
+                       Stream<RequestPathParam> l = stream(name);
+                       boolean hasAllBlanks = l.allMatch(x -> 
StringUtils.isEmpty(x.getValue()));
+                       if (hasAllBlanks) {
+                               removeAll(getAll(name));
+                               add(new RequestPathParam(req, name, 
vs.resolve(p.getValue())));
                        }
                }
                return this;
@@ -216,61 +234,14 @@ public class RequestPathParams {
        }
 
        /**
-        * Returns all the parameters with the specified name.
-        *
-        * @param name The parameter name.
-        * @return The list of all parameters with the specified name, or an 
empty list if none are found.
-        */
-       public List<RequestPathParam> getAll(String name) {
-               assertArgNotNull("name", name);
-               List<RequestPathParam> l = map.get(key(name));
-               return l == null ? emptyList() : unmodifiable(l);
-       }
-
-       /**
-        * Returns all the parameters on this request.
+        * Adds a default entry to the query parameters.
         *
-        * @return All the parameters on this request.
-        */
-       public List<RequestPathParam> getAll() {
-               return unmodifiable(list);
-       }
-
-       /**
-        * Returns <jk>true</jk> if the parameters with the specified names are 
present.
-        *
-        * @param names The parameter names.  Must not be <jk>null</jk>.
-        * @return <jk>true</jk> if the parameters with the specified names are 
present.
-        */
-       public boolean contains(String...names) {
-               assertArgNotNull("names", names);
-               for (String n : names)
-                       if (! map.containsKey(key(n)))
-                               return false;
-               return true;
-       }
-
-       /**
-        * Returns <jk>true</jk> if the parameter with any of the specified 
names are present.
-        *
-        * @param names The parameter names.  Must not be <jk>null</jk>.
-        * @return <jk>true</jk> if the parameter with any of the specified 
names are present.
-        */
-       public boolean containsAny(String...names) {
-               assertArgNotNull("names", names);
-               for (String n : names)
-                       if (map.containsKey(key(n)))
-                               return true;
-               return false;
-       }
-
-       /**
-        * Returns <jk>true</jk> if these parameters are empty.
-        *
-        * @return <jk>true</jk> if these parameters are empty.
+        * @param name The name.
+        * @param value The value.
+        * @return This object.
         */
-       public boolean isEmpty() {
-               return list.isEmpty();
+       public RequestPathParams addDefault(String name, String value) {
+               return addDefault(BasicStringPart.of(name, value));
        }
 
        /**
@@ -286,13 +257,7 @@ public class RequestPathParams {
         */
        public RequestPathParams add(String name, Object value) {
                assertArgNotNull("name", name);
-               String key = key(name);
-               RequestPathParam h = new RequestPathParam(req, name, 
stringify(value)).parser(parser);
-               if (map.containsKey(key))
-                       map.get(key).add(h);
-               else
-                       map.put(key, list(h));
-               list.add(h);
+               add(new RequestPathParam(req, name, 
stringify(value)).parser(parser));
                return this;
        }
 
@@ -308,10 +273,9 @@ public class RequestPathParams {
         */
        public RequestPathParams add(NameValuePair...parameters) {
                assertArgNotNull("parameters", parameters);
-               for (NameValuePair p : parameters) {
+               for (NameValuePair p : parameters)
                        if (p != null)
                                add(p.getName(), p.getValue());
-               }
                return this;
        }
 
@@ -331,12 +295,7 @@ public class RequestPathParams {
         */
        public RequestPathParams set(String name, Object value) {
                assertArgNotNull("name", name);
-               String key = key(name);
-               RequestPathParam p = new RequestPathParam(req, name, 
stringify(value)).parser(parser);
-               if (map.containsKey(key))
-                       
list.removeIf(x->caseSensitive?x.getName().equals(name):x.getName().equalsIgnoreCase(name));
-               list.add(p);
-               map.put(key, list(p));
+               set(new RequestPathParam(req, name, 
stringify(value)).parser(parser));
                return this;
        }
 
@@ -363,36 +322,97 @@ public class RequestPathParams {
        /**
         * Remove parameters.
         *
-        * @param name The parameter names.  Must not be <jk>null</jk>.
+        * @param name The parameter name.  Must not be <jk>null</jk>.
         * @return This object.
         */
-       public RequestPathParams remove(String...name) {
+       public RequestPathParams remove(String name) {
                assertArgNotNull("name", name);
-               for (String n : name) {
-                       String key = key(n);
-                       if (map.containsKey(key))
-                               list.removeAll(map.get(key));
-                       map.remove(key);
-               }
+               removeIf(x -> eq(x.getName(), name));
                return this;
        }
 
        /**
-        * Remove parameters.
+        * Returns a copy of this object but only with the specified param 
names copied.
         *
-        * @param parameters The parameters to remove.  Must not be 
<jk>null</jk>.
-        * @return This object.
+        * @param names The list to include in the copy.
+        * @return A new list object.
         */
-       public RequestPathParams remove(NameValuePair...parameters) {
-               for (NameValuePair p : parameters)
-                       remove(p.getName());
-               return this;
+       public RequestPathParams subset(String...names) {
+               return new RequestPathParams(this, names);
        }
 
        
//-----------------------------------------------------------------------------------------------------------------
        // Convenience getters.
        
//-----------------------------------------------------------------------------------------------------------------
 
+       /**
+        * Returns <jk>true</jk> if the parameters with the specified name is 
present.
+        *
+        * @param name The parameter name.  Must not be <jk>null</jk>.
+        * @return <jk>true</jk> if the parameters with the specified name is 
present.
+        */
+       public boolean contains(String name) {
+               assertArgNotNull("names", name);
+               return stream(name).findAny().isPresent();
+       }
+
+       /**
+        * Returns <jk>true</jk> if the parameter with any of the specified 
names are present.
+        *
+        * @param names The parameter names.  Must not be <jk>null</jk>.
+        * @return <jk>true</jk> if the parameter with any of the specified 
names are present.
+        */
+       public boolean containsAny(String...names) {
+               assertArgNotNull("names", names);
+               for (String n : names)
+                       if (stream(n).findAny().isPresent())
+                               return true;
+               return false;
+       }
+
+       /**
+        * Returns all the parameters with the specified name.
+        *
+        * @param name The parameter name.
+        * @return The list of all parameters with the specified name, or an 
empty list if none are found.
+        */
+       public List<RequestPathParam> getAll(String name) {
+               assertArgNotNull("name", name);
+               return stream(name).collect(toList());
+       }
+
+       /**
+        * Returns all headers with the specified name.
+        *
+        * @param name The header name.
+        * @return The stream of all headers with matching names.  Never 
<jk>null</jk>.
+        */
+       public Stream<RequestPathParam> stream(String name) {
+               return stream().filter(x -> eq(x.getName(), name));
+       }
+
+       /**
+        * Returns all headers in sorted order.
+        *
+        * @return The stream of all headers in sorted order.
+        */
+       public Stream<RequestPathParam> getSorted() {
+               Comparator<RequestPathParam> x;
+               if (caseSensitive)
+                       x = (x1,x2) -> x1.getName().compareTo(x2.getName());
+               else
+                       x = (x1,x2) -> 
String.CASE_INSENSITIVE_ORDER.compare(x1.getName(), x2.getName());
+               return stream().sorted(x);
+       }
+
+       /**
+        * Returns all the unique header names in this list.
+        * @return The list of all unique header names in this list.
+        */
+       public List<String> getNames() {
+               return stream().map(x -> x.getName()).map(x -> caseSensitive ? 
x : x.toLowerCase()).distinct().collect(toList());
+       }
+
        /**
         * Returns the first parameter with the specified name.
         *
@@ -405,8 +425,7 @@ public class RequestPathParams {
         */
        public RequestPathParam getFirst(String name) {
                assertArgNotNull("name", name);
-               List<RequestPathParam> l = map.get(key(name));
-               return (l == null || l.isEmpty() ? new RequestPathParam(req, 
name, null).parser(parser) : l.get(0));
+               return stream(name).findFirst().orElseGet(()->new 
RequestPathParam(req, name, null).parser(parser));
        }
 
        /**
@@ -421,8 +440,9 @@ public class RequestPathParams {
         */
        public RequestPathParam getLast(String name) {
                assertArgNotNull("name", name);
-               List<RequestPathParam> l = map.get(key(name));
-               return (l == null || l.isEmpty() ? new RequestPathParam(req, 
name, null).parser(parser) : l.get(l.size()-1));
+               Value<RequestPathParam> v = Value.empty();
+               stream(name).forEach(x -> v.set(x));
+               return v.orElseGet(() -> new RequestPathParam(req, name, 
null).parser(parser));
        }
 
        /**
@@ -435,7 +455,18 @@ public class RequestPathParams {
         * @return The parameter value, or {@link Optional#empty()} if it 
doesn't exist.
         */
        public RequestPathParam get(String name) {
-               return getLast(name);
+               List<RequestPathParam> l = getAll(name);
+               if (l.isEmpty())
+                       return new RequestPathParam(req, name, 
null).parser(parser);
+               if (l.size() == 1)
+                       return l.get(0);
+               StringBuilder sb = new StringBuilder(128);
+               for (int i = 0, j = l.size(); i < j; i++) {
+                       if (i > 0)
+                               sb.append(", ");
+                       sb.append(l.get(i).getValue());
+               }
+               return new RequestPathParam(req, name, 
sb.toString()).parser(parser);
        }
 
        /**
@@ -539,31 +570,17 @@ public class RequestPathParams {
                return get("/**");
        }
 
-       /**
-        * Converts the parameters to a readable string.
-        *
-        * @param sorted Sort the parameters by name.
-        * @return A JSON string containing the contents of the parameters.
-        */
-       public String toString(boolean sorted) {
-               JsonMap m = JsonMap.create();
-               if (sorted) {
-                       for (List<RequestPathParam> p1 : map.values())
-                               for (RequestPathParam p2 : p1)
-                                       m.append(p2.getName(), p2.getValue());
-               } else {
-                       for (RequestPathParam p : list)
-                               m.append(p.getName(), p.getValue());
-               }
-               return m.toString();
-       }
-
-       private String key(String name) {
-               return caseSensitive ? name : name.toLowerCase();
+       private boolean eq(String s1, String s2) {
+               if (caseSensitive)
+                       return StringUtils.eq(s1, s2);
+               return StringUtils.eqic(s1, s2);
        }
 
        @Override /* Object */
        public String toString() {
-               return toString(false);
+               JsonMap m = new JsonMap();
+               for (String n : getNames())
+                       m.put(n, get(n).asString().orElse(null));
+               return m.asJson();
        }
 }
diff --git 
a/juneau-rest/juneau-rest-server/src/main/java/org/apache/juneau/rest/httppart/RequestQueryParams.java
 
b/juneau-rest/juneau-rest-server/src/main/java/org/apache/juneau/rest/httppart/RequestQueryParams.java
index c2015c9df..a58901e4e 100644
--- 
a/juneau-rest/juneau-rest-server/src/main/java/org/apache/juneau/rest/httppart/RequestQueryParams.java
+++ 
b/juneau-rest/juneau-rest-server/src/main/java/org/apache/juneau/rest/httppart/RequestQueryParams.java
@@ -569,6 +569,12 @@ public class RequestQueryParams extends 
ArrayList<RequestQueryParam> {
                return 
ofNullable(PageArgs.create(get("p").asInteger().orElse(null), 
get("l").asInteger().orElse(null)));
        }
 
+       private boolean eq(String s1, String s2) {
+               if (caseSensitive)
+                       return StringUtils.eq(s1, s2);
+               return StringUtils.eqic(s1, s2);
+       }
+
        @Override /* Object */
        public String toString() {
                JsonMap m = new JsonMap();
diff --git 
a/juneau-utest/src/test/java/org/apache/juneau/rest/annotation/Path_Test.java 
b/juneau-utest/src/test/java/org/apache/juneau/rest/annotation/Path_Test.java
index 43db36a0e..1a1056a11 100644
--- 
a/juneau-utest/src/test/java/org/apache/juneau/rest/annotation/Path_Test.java
+++ 
b/juneau-utest/src/test/java/org/apache/juneau/rest/annotation/Path_Test.java
@@ -386,27 +386,27 @@ public class Path_Test {
        public static class F  {
                @RestGet(path="/")
                public String a(RequestPathParams path) {
-                       return format("a: {0}", path.toString(true));
+                       return format("a: {0}", path.toString());
                }
                @RestGet(path="/*")
                public String b(RequestPathParams path) {
-                       return format("b: {0}", path.toString(true));
+                       return format("b: {0}", path.toString());
                }
                @RestGet(path="/fc")
                public String c(RequestPathParams path) {
-                       return format("c: {0}", path.toString(true));
+                       return format("c: {0}", path.toString());
                }
                @RestGet(path="/fd/{c}/{d}")
                public String d(RequestPathParams path) {
-                       return format("d: {0}", path.toString(true));
+                       return format("d: {0}", path.toString());
                }
                @RestGet(path="/fe/{a}/{b}")
                public String e(RequestPathParams path) {
-                       return format("e: {0}", path.toString(true));
+                       return format("e: {0}", path.toString());
                }
                @RestGet(path="/ff/{c}/{d}/*")
                public String f(RequestPathParams path) {
-                       return format("f: {0}", path.toString(true));
+                       return format("f: {0}", path.toString());
                }
                private String format(String msg, Object...args) {
                        return StringUtils.format(msg, args);
@@ -436,7 +436,7 @@ public class Path_Test {
                        .assertStatus(404);
                f.get("http://localhost/f/x1/x2/foo";)
                        .run()
-                       .assertContent("b: 
{'/*':'foo','/**':'foo',a:'x1',b:'x2'}");
+                       .assertContent("b: 
{a:'x1',b:'x2','/**':'foo','/*':'foo'}");
                f.get("http://localhost/f///foo";)
                        .run()
                        .assertStatus(404);
@@ -478,16 +478,16 @@ public class Path_Test {
                        .assertStatus(404);
                f.get("http://localhost/f/x1/x2/fe/x3/x4";)
                        .run()
-                       .assertContent("e: {a:'x3',b:'x4'}");
+                       .assertContent("e: {a:'x1, x3',b:'x2, x4'}");
                f.get("http://localhost/f/x1/x2/ff/x3/x4";)
                        .run()
                        .assertContent("f: {a:'x1',b:'x2',c:'x3',d:'x4'}");
                f.get("http://localhost/f/x1/x2/ff/x3/x4/";)
                        .run()
-                       .assertContent("f: 
{'/*':'','/**':'',a:'x1',b:'x2',c:'x3',d:'x4'}");
+                       .assertContent("f: 
{a:'x1',b:'x2',c:'x3',d:'x4','/**':'','/*':''}");
                f.get("http://localhost/f/x1/x2/ff/x3/x4/foo/bar";)
                        .run()
-                       .assertContent("f: 
{'/*':'foo/bar','/**':'foo/bar',a:'x1',b:'x2',c:'x3',d:'x4'}");
+                       .assertContent("f: 
{a:'x1',b:'x2',c:'x3',d:'x4','/**':'foo/bar','/*':'foo/bar'}");
        }
 
        
//------------------------------------------------------------------------------------------------------------------
@@ -551,16 +551,16 @@ public class Path_Test {
                        .assertStatus(404);
                g.get("http://localhost/f/x1/x2/fe/x3/x4";)
                        .run()
-                       .assertContent("e: {a:'x3',b:'x4'}");
+                       .assertContent("e: {a:'x1, x3',b:'x2, x4'}");
                g.get("http://localhost/f/x1/x2/ff/x3/x4";)
                        .run()
                        .assertContent("f: {a:'x1',b:'x2',c:'x3',d:'x4'}");
                g.get("http://localhost/f/x1/x2/ff/x3/x4/";)
                        .run()
-                       .assertContent("f: 
{'/*':'','/**':'',a:'x1',b:'x2',c:'x3',d:'x4'}");
+                       .assertContent("f: 
{a:'x1',b:'x2',c:'x3',d:'x4','/**':'','/*':''}");
                g.get("http://localhost/f/x1/x2/ff/x3/x4/foo/bar";)
                        .run()
-                       .assertContent("f: 
{'/*':'foo/bar','/**':'foo/bar',a:'x1',b:'x2',c:'x3',d:'x4'}");
+                       .assertContent("f: 
{a:'x1',b:'x2',c:'x3',d:'x4','/**':'foo/bar','/*':'foo/bar'}");
        }
 
        
//------------------------------------------------------------------------------------------------------------------
@@ -609,7 +609,7 @@ public class Path_Test {
                        .assertStatus(404);
                h.get("http://localhost/h/ha1/hb1/f/x1/x2/foo";)
                        .run()
-                       .assertContent("b: 
{'/*':'foo','/**':'foo',a:'x1',b:'x2',ha:'ha1',hb:'hb1'}");
+                       .assertContent("b: 
{a:'x1',b:'x2',ha:'ha1',hb:'hb1','/**':'foo','/*':'foo'}");
                h.get("http://localhost/h//hb1/f/x1/x2/foo";)
                        .run()
                        .assertStatus(404);
@@ -645,19 +645,19 @@ public class Path_Test {
                        .assertStatus(404);
                h.get("http://localhost/h/ha1/hb1/f/x1/x2/fd/x3/x4";)
                        .run()
-                       .assertContent("d: 
{a:'x1',b:'x2',c:'x3',d:'x4',ha:'ha1',hb:'hb1'}");
+                       .assertContent("d: 
{a:'x1',b:'x2',ha:'ha1',hb:'hb1',c:'x3',d:'x4'}");
                h.get("http://localhost/h/ha1/hb1/f/x1/x2/fe/x3/x4";)
                        .run()
-                       .assertContent("e: {a:'x3',b:'x4',ha:'ha1',hb:'hb1'}");
+                       .assertContent("e: {a:'x1, x3',b:'x2, 
x4',ha:'ha1',hb:'hb1'}");
                h.get("http://localhost/h/ha1/hb1/f/x1/x2/ff/x3/x4";)
                        .run()
-                       .assertContent("f: 
{a:'x1',b:'x2',c:'x3',d:'x4',ha:'ha1',hb:'hb1'}");
+                       .assertContent("f: 
{a:'x1',b:'x2',ha:'ha1',hb:'hb1',c:'x3',d:'x4'}");
                h.get("http://localhost/h/ha1/hb1/f/x1/x2/ff/x3/x4/";)
                        .run()
-                       .assertContent("f: 
{'/*':'','/**':'',a:'x1',b:'x2',c:'x3',d:'x4',ha:'ha1',hb:'hb1'}");
+                       .assertContent("f: 
{a:'x1',b:'x2',ha:'ha1',hb:'hb1',c:'x3',d:'x4','/**':'','/*':''}");
                h.get("http://localhost/h/ha1/hb1/f/x1/x2/ff/x3/x4/foo/bar";)
                        .run()
-                       .assertContent("f: 
{'/*':'foo/bar','/**':'foo/bar',a:'x1',b:'x2',c:'x3',d:'x4',ha:'ha1',hb:'hb1'}");
+                       .assertContent("f: 
{a:'x1',b:'x2',ha:'ha1',hb:'hb1',c:'x3',d:'x4','/**':'foo/bar','/*':'foo/bar'}");
        }
 
        
//------------------------------------------------------------------------------------------------------------------
@@ -720,25 +720,25 @@ public class Path_Test {
                        .assertStatus(404);
                i.get("http://localhost/i/ia1/ib1/h/ha1/hb1/f/x1/x2/foo";)
                        .run()
-                       .assertContent("b: 
{'/*':'foo','/**':'foo',a:'x1',b:'x2',ha:'ha1',hb:'hb1',ia:'ia1',ib:'ib1'}");
+                       .assertContent("b: 
{a:'x1',b:'x2',ha:'ha1',hb:'hb1',ia:'ia1',ib:'ib1','/**':'foo','/*':'foo'}");
                i.get("http://localhost/i/ia1/ib1/h/ha1/hb1/f/x1/x2/fc";)
                        .run()
                        .assertContent("c: 
{a:'x1',b:'x2',ha:'ha1',hb:'hb1',ia:'ia1',ib:'ib1'}");
                i.get("http://localhost/i/ia1/ib1/h/ha1/hb1/f/x1/x2/fd/x3/x4";)
                        .run()
-                       .assertContent("d: 
{a:'x1',b:'x2',c:'x3',d:'x4',ha:'ha1',hb:'hb1',ia:'ia1',ib:'ib1'}");
+                       .assertContent("d: 
{a:'x1',b:'x2',ha:'ha1',hb:'hb1',ia:'ia1',ib:'ib1',c:'x3',d:'x4'}");
                i.get("http://localhost/i/ia1/ib1/h/ha1/hb1/f/x1/x2/fe/x3/x4";)
                        .run()
-                       .assertContent("e: 
{a:'x3',b:'x4',ha:'ha1',hb:'hb1',ia:'ia1',ib:'ib1'}");
+                       .assertContent("e: {a:'x1, x3',b:'x2, 
x4',ha:'ha1',hb:'hb1',ia:'ia1',ib:'ib1'}");
                i.get("http://localhost/i/ia1/ib1/h/ha1/hb1/f/x1/x2/ff/x3/x4";)
                        .run()
-                       .assertContent("f: 
{a:'x1',b:'x2',c:'x3',d:'x4',ha:'ha1',hb:'hb1',ia:'ia1',ib:'ib1'}");
+                       .assertContent("f: 
{a:'x1',b:'x2',ha:'ha1',hb:'hb1',ia:'ia1',ib:'ib1',c:'x3',d:'x4'}");
                i.get("http://localhost/i/ia1/ib1/h/ha1/hb1/f/x1/x2/ff/x3/x4/";)
                        .run()
-                       .assertContent("f: 
{'/*':'','/**':'',a:'x1',b:'x2',c:'x3',d:'x4',ha:'ha1',hb:'hb1',ia:'ia1',ib:'ib1'}");
+                       .assertContent("f: 
{a:'x1',b:'x2',ha:'ha1',hb:'hb1',ia:'ia1',ib:'ib1',c:'x3',d:'x4','/**':'','/*':''}");
                
i.get("http://localhost/i/ia1/ib1/h/ha1/hb1/f/x1/x2/ff/x3/x4/foo/bar";)
                        .run()
-                       .assertContent("f: 
{'/*':'foo/bar','/**':'foo/bar',a:'x1',b:'x2',c:'x3',d:'x4',ha:'ha1',hb:'hb1',ia:'ia1',ib:'ib1'}");
+                       .assertContent("f: 
{a:'x1',b:'x2',ha:'ha1',hb:'hb1',ia:'ia1',ib:'ib1',c:'x3',d:'x4','/**':'foo/bar','/*':'foo/bar'}");
        }
 
        
//------------------------------------------------------------------------------------------------------------------

Reply via email to