commit 6df7415aded0424b18b54f1412e7182437cdd68f
author Bertrand Delacretaz <bdelacretaz@apache.org> Mon Mar 29 13:14:20 2021 +0200
committer Bertrand Delacretaz <bdelacretaz@apache.org> Mon Mar 29 13:14:20 2021 +0200
tree beb89747ac1211709cb63052a4d9095750489364
parent 544823a8f748943887a820058c9150c58ce4ec8e

SLING-10192 - minor test coverage improvements

src/test/java/org/apache/sling/jcr/repoinit/SetPropertiesTest.java

diff --git a/src/test/java/org/apache/sling/jcr/repoinit/SetPropertiesTest.java b/src/test/java/org/apache/sling/jcr/repoinit/SetPropertiesTest.java
index 2914b5b..94e6713 100644
--- a/src/test/java/org/apache/sling/jcr/repoinit/SetPropertiesTest.java
+++ b/src/test/java/org/apache/sling/jcr/repoinit/SetPropertiesTest.java
@@ -30,6 +30,9 @@
 import javax.jcr.RepositoryException;
 import javax.jcr.ValueFactory;
 import javax.jcr.Value;
+
+import static org.junit.Assert.fail;
+
 import java.io.IOException;
 import java.util.UUID;
 
@@ -177,6 +180,41 @@
         assertAuthorizableSubTreePropertiesAgain(groupid);
     }
 
+    @Test
+    public void unknownAuthorizable() throws Exception {
+        final String setProps =
+            "set properties on authorizable(nonExistingUser)\n"
+            + "set one to oneA\n"
+            + "end"
+        ;
+        try {
+            U.parseAndExecute(setProps);
+            fail("Expecting execution to fail");
+        } catch(RuntimeException asExpected) {
+            // all good
+        }
+    }
+
+    @Test
+    public void typoInFunctionName() throws Exception {
+        String userid = "user" + UUID.randomUUID();
+        U.parseAndExecute("create user " + userid);
+
+        // The parser accepts any function name in paths,
+        // make sure we fail in the right way
+        final String setProps =
+            "set properties on UNauthorizable(" + userid + ")\n"
+            + "set one to oneA\n"
+            + "end"
+        ;
+        try {
+            U.parseAndExecute(setProps);
+            fail("Expecting execution to fail");
+        } catch(RuntimeException asExpected) {
+            // all good
+        }
+    }
+
     /**
      * Set properties on an authorizable and then verify that the values were set
      */
@@ -184,6 +222,7 @@
         final String setPropsA =
                 "set properties on authorizable(" +id + ")\n"
                         + "set one to oneA\n"
+                        + "set dbl{Double} to 3.14\n"
                         + "default two to twoA\n"
                         + "set nested/one to oneA\n"
                         + "default nested/two to twoA\n"
@@ -196,6 +235,7 @@
         U.parseAndExecute(setPropsA);
 
         U.assertAuthorizableSVPropertyExists(id, "one", vf.createValue("oneA"));
+        U.assertAuthorizableSVPropertyExists(id, "dbl", vf.createValue(3.14));
         U.assertAuthorizableSVPropertyExists(id, "nested/one", vf.createValue("oneA"));
         U.assertAuthorizableSVPropertyExists(id, "two", vf.createValue("twoA"));
         U.assertAuthorizableSVPropertyExists(id, "nested/two", vf.createValue("twoA"));