test: fix readonly reviewer sandbox assertions
This commit is contained in:
@@ -138,8 +138,10 @@ describe('claude reviewer runtime guard', () => {
|
||||
'best-effort',
|
||||
);
|
||||
|
||||
expect(sandbox).toBeDefined();
|
||||
expect(sandbox?.failIfUnavailable).toBe(false);
|
||||
if (!sandbox) {
|
||||
throw new Error('expected sandbox settings');
|
||||
}
|
||||
expect(sandbox.failIfUnavailable).toBe(false);
|
||||
});
|
||||
|
||||
it('keeps non-linux reviewers in best-effort sandbox mode', () => {
|
||||
@@ -153,8 +155,10 @@ describe('claude reviewer runtime guard', () => {
|
||||
'best-effort',
|
||||
);
|
||||
|
||||
expect(sandbox).toBeDefined();
|
||||
expect(sandbox?.failIfUnavailable).toBe(false);
|
||||
if (!sandbox) {
|
||||
throw new Error('expected sandbox settings');
|
||||
}
|
||||
expect(sandbox.failIfUnavailable).toBe(false);
|
||||
});
|
||||
|
||||
it('flags mutating shell commands', () => {
|
||||
|
||||
Reference in New Issue
Block a user