feat: support comment ("#",";") in model (testCommentModel) (#381)

Author: LMay001

examples/comment_model.conf

@@ -0,0 +1,12 @@
+[request_definition]
+r = sub, obj, act ; Request definition
+
+[policy_definition]
+p = sub, obj, act
+
+[policy_effect]
+e = some(where (p.eft == allow)) # This is policy effect.
+
+# Matchers
+[matchers]
+m = r.sub == p.sub && r.obj == p.obj && r.act == p.act

src/main/java/org/casbin/jcasbin/config/Config.java

@@ -120,6 +120,12 @@ private void parseBuffer(BufferedReader buf) throws IOException {
             } else if (line.startsWith("[") && line.endsWith("]")) {
                 section = line.substring(1, line.length() - 1);
             } else {
+                int index1 = line.indexOf(DEFAULT_COMMENT);
+                int index2 = line.indexOf(DEFAULT_COMMENT_SEM);
+                int index = (index1 != -1 && index2 != -1) ? Math.min(index1, index2) : (index1 != -1) ? index1 : (index2 != -1) ? index2 : -1;
+                if (index != -1)
+                    line = line.substring(0, index);
+
                 String[] optionVal = line.split("=", 2);
                 if (optionVal.length != 2) {
                     throw new IllegalArgumentException(String.format("parse the content error : line %d , %s = ? ", lineNum, optionVal[0]));

src/test/java/org/casbin/jcasbin/main/ModelUnitTest.java

@@ -787,6 +787,20 @@ public void testABACPolicy(){
         testEnforce(e, sub3, "/data2", "write", false);
     }
 
+    @Test
+    public void testCommentModel(){
+        Enforcer e = new Enforcer("examples/comment_model.conf", "examples/basic_policy.csv");
+
+        testEnforce(e, "alice", "data1", "read", true);
+        testEnforce(e, "alice", "data1", "write", false);
+        testEnforce(e, "alice", "data2", "read", false);
+        testEnforce(e, "alice", "data2", "write", false);
+        testEnforce(e, "bob", "data1", "read", false);
+        testEnforce(e, "bob", "data1", "write", false);
+        testEnforce(e, "bob", "data2", "read", false);
+        testEnforce(e, "bob", "data2", "write", true);
+    }
+
     @Test
     public void testSubjectPriorityWithDomain() {
         Enforcer e = new Enforcer("examples/subject_priority_model_with_domain.conf", "examples/subject_priority_policy_with_domain.csv");