Author: fmeschbe
Date: Mon May 19 02:07:44 2008
New Revision: 657763

URL: http://svn.apache.org/viewvc?rev=657763&view=rev
Log:
SLING-394 Test whether :saveParamPrefix is really ignored

Modified:
    
incubator/sling/trunk/launchpad/webapp/src/test/java/org/apache/sling/launchpad/webapp/integrationtest/servlets/post/PostServletCreateTest.java

Modified: 
incubator/sling/trunk/launchpad/webapp/src/test/java/org/apache/sling/launchpad/webapp/integrationtest/servlets/post/PostServletCreateTest.java
URL: 
http://svn.apache.org/viewvc/incubator/sling/trunk/launchpad/webapp/src/test/java/org/apache/sling/launchpad/webapp/integrationtest/servlets/post/PostServletCreateTest.java?rev=657763&r1=657762&r2=657763&view=diff
==============================================================================
--- 
incubator/sling/trunk/launchpad/webapp/src/test/java/org/apache/sling/launchpad/webapp/integrationtest/servlets/post/PostServletCreateTest.java
 (original)
+++ 
incubator/sling/trunk/launchpad/webapp/src/test/java/org/apache/sling/launchpad/webapp/integrationtest/servlets/post/PostServletCreateTest.java
 Mon May 19 02:07:44 2008
@@ -119,8 +119,8 @@
         assertJavascript("undefined", content, "out.println(typeof data.c)");
     }
 
-    /** Use a custom "save prefix" on some parameters, and check that only 
those
-     *  who have the prefix are saved.
+    /** SLING-394 removed :saveParamPrefix support. We check whether this is
+     * really ignored
      */
     public void testCustomSavePrefix() throws IOException {
         final Map <String, String> props = new HashMap <String, String> ();
@@ -130,21 +130,10 @@
         props.put(":saveParamPrefix","STUFF_");
         final String createdNodeUrl = testClient.createNode(postUrl + 
SlingPostConstants.DEFAULT_CREATE_SUFFIX, props,null,false);
         final String content = getContent(createdNodeUrl + ".json", 
CONTENT_TYPE_JSON);
-        assertJavascript("123456", content, "out.println(data.a + data.b)");
-        assertJavascript("undefined", content, "out.println(typeof data.c)");
+        assertJavascript("undefined", content, "out.println(typeof data.a)");
+        assertJavascript("undefined", content, "out.println(typeof data.b)");
+        assertJavascript("123456", content, "out.println(data.STUFF_a + 
data.STUFF_b)");
+        assertJavascript("string", content, "out.println(typeof data.c)");
     }
 
-    public void TODO_FAILS_testCustomSavePrefixPlusPlus() throws IOException {
-        // for some reason, ++ as a custom save prefix fails
-        // might indicate a weirdness in parameters processing
-        final Map <String, String> props = new HashMap <String, String> ();
-        props.put("++a","123");
-        props.put("++b","456");
-        props.put("c","not saved");
-        props.put(":saveParamPrefix","++");
-        final String createdNodeUrl = testClient.createNode(postUrl + 
SlingPostConstants.DEFAULT_CREATE_SUFFIX, props,null,false);
-        final String content = getContent(createdNodeUrl + ".json", 
CONTENT_TYPE_JSON);
-        assertJavascript("123456", content, "out.println(data.a + data.b)");
-        assertJavascript("undefined", content, "out.println(typeof data.c)");
-    }
  }
\ No newline at end of file


Reply via email to