From febabc400944ab71f4bb3106394b37b1be21f49d Mon Sep 17 00:00:00 2001 From: tttMelody <245915794@qq.com> Date: Mon, 26 Mar 2018 13:06:32 +0800 Subject: [PATCH] minor, add null case. --- .../java/org/apache/kylin/rest/util/AclUtilTest.java | 12 ++++++++++++ 1 file changed, 12 insertions(+) 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 index 18e5bf528cf..217abe8e9ce 100644 --- 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";