Spaces:
Sleeping
Sleeping
HonzysClawdbot
test: add coverage for pure utility modules, fix Vitest 60% threshold (#339)
eb47743 unverified | import { describe, it, expect } from 'vitest' | |
| import { resolveWithin } from '../paths' | |
| import path from 'node:path' | |
| describe('resolveWithin', () => { | |
| const base = '/tmp/sandbox' | |
| it('resolves a simple relative path within base', () => { | |
| const result = resolveWithin(base, 'file.txt') | |
| expect(result).toBe('/tmp/sandbox/file.txt') | |
| }) | |
| it('resolves nested relative path', () => { | |
| const result = resolveWithin(base, 'subdir/file.txt') | |
| expect(result).toBe('/tmp/sandbox/subdir/file.txt') | |
| }) | |
| it('throws when path escapes base with ..', () => { | |
| expect(() => resolveWithin(base, '../escape.txt')).toThrow('Path escapes base directory') | |
| }) | |
| it('throws when path tries deep escape', () => { | |
| expect(() => resolveWithin(base, '../../etc/passwd')).toThrow('Path escapes base directory') | |
| }) | |
| it('throws for absolute path outside base', () => { | |
| expect(() => resolveWithin(base, '/etc/passwd')).toThrow('Path escapes base directory') | |
| }) | |
| it('allows an absolute path within the base', () => { | |
| const result = resolveWithin(base, '/tmp/sandbox/file.txt') | |
| expect(result).toBe('/tmp/sandbox/file.txt') | |
| }) | |
| it('handles double slashes and normalizes', () => { | |
| const result = resolveWithin(base, 'subdir//file.txt') | |
| expect(result).toBe('/tmp/sandbox/subdir/file.txt') | |
| }) | |
| it('does not allow sibling directory access', () => { | |
| expect(() => resolveWithin(base, '../other/file.txt')).toThrow() | |
| }) | |
| it('handles base dir with trailing slash', () => { | |
| const result = resolveWithin('/tmp/sandbox/', 'file.txt') | |
| expect(result).toBe('/tmp/sandbox/file.txt') | |
| }) | |
| }) | |