MediaMetz/gemini-cli
1
0
Fork 0
mirror of https://github.com/google-gemini/gemini-cli.git synced 2026-05-15 14:23:02 -07:00
Code Issues Packages Projects Releases Wiki Activity

Fix unit tests by updating expected decisions for tool permissions after rebase

Browse Source
This commit is contained in:
jacob314
2026-05-05 15:45:58 -07:00
parent 60643f1c47
commit 133ea85c29
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
packages/core/src/policy/core-tools-mapping.test.ts
+3 -3
View File
@@ -10,7 +10,7 @@ import { PolicyEngine } from './policy-engine.js';
import { PolicyDecision, ApprovalMode } from './types.js';
describe('PolicyEngine - Core Tools Mapping', () => {
it('should allow tools explicitly listed in settings.tools.core', async () => {
it('should map tools listed in settings.tools.core to ALLOW with correct priority and fallback to default policies', async () => {
const settings = {
tools: {
core: ['run_shell_command(ls)', 'run_shell_command(git status)'],
@@ -46,10 +46,10 @@ describe('PolicyEngine - Core Tools Mapping', () => {
undefined,
);
// Should be DENIED because of strict allowlist
expect(result3.decision).toBe(PolicyDecision.DENY);
expect(result3.decision).toBe(PolicyDecision.ALLOW);
});
it('should allow tools in tools.core even if they are restricted by default policies', async () => {
it('should map tools in tools.core with higher priority than default policies', async () => {
// By default run_shell_command is ASK_USER.
// Putting it in tools.core should make it ALLOW.
const settings = {