Author: fmeschbe
Date: Thu Feb 20 10:52:38 2014
New Revision: 1570139

URL: http://svn.apache.org/r1570139
Log:
SLING-3403 Improved Unit test for parameter lists to make sure
   duplicate parameter names are not collated in non-request order

Modified:
    
sling/trunk/bundles/engine/src/test/java/org/apache/sling/engine/impl/parameters/UtilTest.java

Modified: 
sling/trunk/bundles/engine/src/test/java/org/apache/sling/engine/impl/parameters/UtilTest.java
URL: 
http://svn.apache.org/viewvc/sling/trunk/bundles/engine/src/test/java/org/apache/sling/engine/impl/parameters/UtilTest.java?rev=1570139&r1=1570138&r2=1570139&view=diff
==============================================================================
--- 
sling/trunk/bundles/engine/src/test/java/org/apache/sling/engine/impl/parameters/UtilTest.java
 (original)
+++ 
sling/trunk/bundles/engine/src/test/java/org/apache/sling/engine/impl/parameters/UtilTest.java
 Thu Feb 20 10:52:38 2014
@@ -100,19 +100,25 @@ public class UtilTest extends TestCase {
 
     public void test_decode_query() throws IllegalArgumentException, 
UnsupportedEncodingException, IOException {
         final ParameterMap map = new ParameterMap();
-        final String query = "a=1&b=2&c=3";
+        final String query = "a=1&b=2&c=3&a=1&b=2&c=3";
         Util.parseQueryString(new 
ByteArrayInputStream(query.getBytes(Util.ENCODING_DIRECT)), 
Util.ENCODING_DIRECT, map, false);
 
         assertEquals(3, map.size());
 
         List<RequestParameter> pars = map.getRequestParameterList();
-        assertEquals(3, pars.size());
+        assertEquals(6, pars.size());
         assertEquals("a", pars.get(0).getName());
         assertEquals("1", pars.get(0).getString());
         assertEquals("b", pars.get(1).getName());
         assertEquals("2", pars.get(1).getString());
         assertEquals("c", pars.get(2).getName());
         assertEquals("3", pars.get(2).getString());
+        assertEquals("a", pars.get(3).getName());
+        assertEquals("1", pars.get(3).getString());
+        assertEquals("b", pars.get(4).getName());
+        assertEquals("2", pars.get(4).getString());
+        assertEquals("c", pars.get(5).getName());
+        assertEquals("3", pars.get(5).getString());
     }
 
     public void test_getParameter_with_space() throws Exception {


Reply via email to