You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by cz...@apache.org on 2013/03/06 17:59:40 UTC
svn commit: r1453423 -
/sling/trunk/launchpad/integration-tests/src/main/java/org/apache/sling/launchpad/webapp/integrationtest/servlets/post/PostServletCreateTest.java
Author: cziegeler
Date: Wed Mar 6 16:59:40 2013
New Revision: 1453423
URL: http://svn.apache.org/r1453423
Log:
SLING-2673 : Tweak PostServletCreateTest.testCreatingNodeUnderFile test to allow the presence of a PersistenceException
Modified:
sling/trunk/launchpad/integration-tests/src/main/java/org/apache/sling/launchpad/webapp/integrationtest/servlets/post/PostServletCreateTest.java
Modified: sling/trunk/launchpad/integration-tests/src/main/java/org/apache/sling/launchpad/webapp/integrationtest/servlets/post/PostServletCreateTest.java
URL: http://svn.apache.org/viewvc/sling/trunk/launchpad/integration-tests/src/main/java/org/apache/sling/launchpad/webapp/integrationtest/servlets/post/PostServletCreateTest.java?rev=1453423&r1=1453422&r2=1453423&view=diff
==============================================================================
--- sling/trunk/launchpad/integration-tests/src/main/java/org/apache/sling/launchpad/webapp/integrationtest/servlets/post/PostServletCreateTest.java (original)
+++ sling/trunk/launchpad/integration-tests/src/main/java/org/apache/sling/launchpad/webapp/integrationtest/servlets/post/PostServletCreateTest.java Wed Mar 6 16:59:40 2013
@@ -25,7 +25,6 @@ import java.util.Map;
import javax.servlet.http.HttpServletResponse;
-import org.apache.commons.httpclient.HttpMethod;
import org.apache.commons.httpclient.NameValuePair;
import org.apache.sling.commons.testing.integration.HttpTestBase;
import org.apache.sling.servlets.post.SlingPostConstants;
@@ -169,7 +168,7 @@ public class PostServletCreateTest exten
}
/**
- * SLING-1091: test create node with an exact node name (no filtering)
+ * SLING-1091: test create node with an exact node name (no filtering)
*/
public void testCreateNodeWithExactName() throws IOException {
Map<String,String> nodeProperties = new HashMap<String, String>();
@@ -186,8 +185,8 @@ public class PostServletCreateTest exten
}
/**
- * SLING-1091: test error reporting when attempting to create a node with an
- * invalid exact node name.
+ * SLING-1091: test error reporting when attempting to create a node with an
+ * invalid exact node name.
*/
public void testCreateNodeWithInvalidExactName() throws IOException {
String location = postUrl + SLASH;
@@ -198,12 +197,12 @@ public class PostServletCreateTest exten
}
/**
- * SLING-1091: test error reporting when attempting to create a node with an
- * already used node name.
+ * SLING-1091: test error reporting when attempting to create a node with an
+ * already used node name.
*/
public void testCreateNodeWithAlreadyUsedExactName() throws IOException {
String testNodeName = "alreadyUsedExactNodeName";
-
+
Map<String,String> nodeProperties = new HashMap<String, String>();
nodeProperties.put(SlingPostConstants.RP_NODE_NAME, testNodeName);
final String location = testClient.createNode(postUrl + SLASH, nodeProperties);
@@ -253,11 +252,10 @@ public class PostServletCreateTest exten
list.add(new NameValuePair("prop", "value"));
list.add(new NameValuePair("jcr:primaryType", "nt:unstructured"));
- final HttpMethod method = assertPostStatus(childUrl, 500, list,
+ assertPostStatus(childUrl, 500, list,
"Response to creating a child under nt:file should fail.");
- final String body = method.getResponseBodyAsString();
- assertTrue("Failure should result from a ConstraintViolationException",
- body.contains("javax.jcr.nodetype.ConstraintViolationException"));
+ // we shouldn't check for a specific exception as that one is implementation specific (see SLING-2763)
+ // a result of 500 is enough.
}
}
\ No newline at end of file