Fix TestSavePolicyPreventsDuplicates to test adapter directly

Copilot · hsluoyz · Copilot · commit 34640bba3a27 · 2025-11-07T09:58:27.000Z
Co-authored-by: hsluoyz &lt;3787410+hsluoyz@users.noreply.github.com&gt;
diff --git a/adapter_test.go b/adapter_test.go
@@ -688,8 +688,10 @@ func TestSavePolicyPreventsDuplicates(t *testing.T) {
 		t.Errorf("Expected SavePolicy() to be successful; got %v", err)
 	}
 
-	// Try to add a duplicate via AddPolicy - should be prevented by unique index
-	_, err = e.AddPolicy("alice", "data1", "read")
+	// Try to add a duplicate directly via adapter (bypassing enforcer's in-memory check)
+	// This should be prevented by the unique index
+	adapter := a.(*adapter)
+	err = adapter.AddPolicy("p", "p", []string{"alice", "data1", "read"})
 	if err == nil {
 		t.Error("Expected AddPolicy of duplicate to fail due to unique index, but it succeeded")
 	}

Original file line number	Diff line number	Diff line change
`@@ -688,8 +688,10 @@ func TestSavePolicyPreventsDuplicates(t *testing.T) {`
`688`	`688`	`t.Errorf("Expected SavePolicy() to be successful; got %v", err)`
`689`	`689`	`}`
`690`	`690`
`691`		`- // Try to add a duplicate via AddPolicy - should be prevented by unique index`
`692`		`- _, err = e.AddPolicy("alice", "data1", "read")`
	`691`	`+ // Try to add a duplicate directly via adapter (bypassing enforcer's in-memory check)`
	`692`	`+ // This should be prevented by the unique index`
	`693`	`+ adapter := a.(*adapter)`
	`694`	`+ err = adapter.AddPolicy("p", "p", []string{"alice", "data1", "read"})`
`693`	`695`	`if err == nil {`
`694`	`696`	`t.Error("Expected AddPolicy of duplicate to fail due to unique index, but it succeeded")`
`695`	`697`	`}`