minor, add null case.

chenzhx · Apr 4, 2018 · febabc4 · febabc4
1 parent a6bfc9d
commit febabc4
Showing 1 changed file with 12 additions and 0 deletions.
diff --git a/server/src/test/java/org/apache/kylin/rest/util/AclUtilTest.java b/server/src/test/java/org/apache/kylin/rest/util/AclUtilTest.java
@@ -41,6 +41,18 @@ public class AclUtilTest extends ServiceTestBase {
     @Autowired
     AclUtil aclUtil;
 
+    @Test
+    public void testNull() {
+        // ADMIN will go into hasRole first.
+        swichUser("ANALYST", Constant.ROLE_ANALYST);
+        try {
+            aclUtil.hasProjectAdminPermission(null);
+            Assert.fail("expecting some AlreadyExistsException here");
+        } catch (Exception e) {
+            Assert.assertEquals("Access is denied", e.getMessage());
+        }
+    }
+
     @Test
     public void testBasic() throws IOException {
         final String PROJECT = "default";