Improve benchmark implementation based on code review feedback

Co-authored-by: nomeguy <85475922+nomeguy@users.noreply.github.com>

Author: Copilot

src/test/java/org/casbin/jcasbin/main/benchmark/BenchmarkUtil.java

@@ -20,6 +20,12 @@
  */
 public class BenchmarkUtil {
 
+    /** Default number of warmup iterations */
+    private static final int DEFAULT_WARMUP_ITERATIONS = 3;
+    
+    /** Default number of measurement iterations */
+    private static final int DEFAULT_MEASUREMENT_ITERATIONS = 5;
+    
     /**
      * Functional interface for benchmark operations.
      */
@@ -67,6 +73,6 @@ public static void runBenchmark(String name, int warmupIterations, int measureme
      * @param operation The operation to benchmark
      */
     public static void runBenchmark(String name, BenchmarkOperation operation) {
-        runBenchmark(name, 3, 5, operation);
+        runBenchmark(name, DEFAULT_WARMUP_ITERATIONS, DEFAULT_MEASUREMENT_ITERATIONS, operation);
     }
 }

src/test/java/org/casbin/jcasbin/main/benchmark/CachedEnforcerBenchmarkTest.java

@@ -207,30 +207,30 @@ public void benchmarkCachedPriorityModel() {
 
     @Test
     public void benchmarkCachedRBACModelMediumParallel() {
-        CachedEnforcer e = new CachedEnforcer("examples/rbac_model.conf", "");
-
-        // 10000 roles, 1000 resources.
-        List<List<String>> pPolicies = new ArrayList<>();
-        for (int i = 0; i < 10000; i++) {
-            List<String> policy = new ArrayList<>();
-            policy.add(String.format("group%d", i));
-            policy.add(String.format("data%d", i / 10));
-            policy.add("read");
-            pPolicies.add(policy);
-        }
-        e.addPolicies(pPolicies);
-
-        // 100000 users.
-        List<List<String>> gPolicies = new ArrayList<>();
-        for (int i = 0; i < 100000; i++) {
-            List<String> policy = new ArrayList<>();
-            policy.add(String.format("user%d", i));
-            policy.add(String.format("group%d", i / 10));
-            gPolicies.add(policy);
-        }
-        e.addGroupingPolicies(gPolicies);
-
         BenchmarkUtil.runBenchmark("Cached RBAC Model Medium Parallel", () -> {
+            CachedEnforcer e = new CachedEnforcer("examples/rbac_model.conf", "");
+
+            // 10000 roles, 1000 resources.
+            List<List<String>> pPolicies = new ArrayList<>();
+            for (int i = 0; i < 10000; i++) {
+                List<String> policy = new ArrayList<>();
+                policy.add(String.format("group%d", i));
+                policy.add(String.format("data%d", i / 10));
+                policy.add("read");
+                pPolicies.add(policy);
+            }
+            e.addPolicies(pPolicies);
+
+            // 100000 users.
+            List<List<String>> gPolicies = new ArrayList<>();
+            for (int i = 0; i < 100000; i++) {
+                List<String> policy = new ArrayList<>();
+                policy.add(String.format("user%d", i));
+                policy.add(String.format("group%d", i / 10));
+                gPolicies.add(policy);
+            }
+            e.addGroupingPolicies(gPolicies);
+
             e.enforce("user5001", "data150", "read");
         });
     }

src/test/java/org/casbin/jcasbin/main/benchmark/RoleManagerBenchmarkTest.java

@@ -193,11 +193,10 @@ public void benchmarkHasLinkWithPatternAndDomainPatternLarge() {
 
     @Test
     public void benchmarkConcurrentHasLinkWithMatching() {
-        Enforcer e = new Enforcer("examples/rbac_with_pattern_model.conf", "examples/rbac_with_pattern_policy.csv");
-        e.addNamedMatchingFunc("g2", "keyMatch2", BuiltInFunctions::keyMatch2);
-        RoleManager rm = e.getRoleManager();
-
         BenchmarkUtil.runBenchmark("Concurrent HasLink With Matching", () -> {
+            Enforcer e = new Enforcer("examples/rbac_with_pattern_model.conf", "examples/rbac_with_pattern_policy.csv");
+            e.addNamedMatchingFunc("g2", "keyMatch2", BuiltInFunctions::keyMatch2);
+            RoleManager rm = e.getRoleManager();
             rm.hasLink("alice", "/book/123");
         });
     }