Skip to content

Commit f636df3

Browse files
icecrasher321claude
andcommitted
test(sandbox): drop explicit any from the new conformance stream mocks
The two new E2B mocks annotated their arguments as `any`, which both violates the repo's no-`any` rule and defeats the point of a mock: an invalid SDK shape would type-check. Matches the sibling mock a few lines above (`async (_code, options) =>`) and infers from the `vi.fn()` signature instead of naming a type, so the mock stays bound to whatever the adapter actually calls. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
1 parent 4413a7e commit f636df3

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

apps/sim/lib/execution/remote-sandbox/conformance.test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -379,7 +379,7 @@ describe.each(PROVIDERS)('sandbox conformance [%s]', (provider) => {
379379
// while producing no oversized result — the caller parses every chunk and keeps none of it.
380380
const oversized = 'x'.repeat(MAX_SANDBOX_PROCESS_OUTPUT_BYTES + 1024)
381381
if (provider === 'e2b') {
382-
mockE2BCommandsRun.mockImplementationOnce(async (_cmd: string, options: any) => {
382+
mockE2BCommandsRun.mockImplementationOnce(async (_cmd, options) => {
383383
options.onStdout(`${oversized}TAIL_MARKER`)
384384
return { stdout: `${oversized}TAIL_MARKER`, stderr: '', exitCode: 0 }
385385
})
@@ -416,7 +416,7 @@ describe.each(PROVIDERS)('sandbox conformance [%s]', (provider) => {
416416
it('still bounds a stream the caller does not consume', async () => {
417417
const oversized = 'x'.repeat(MAX_SANDBOX_PROCESS_OUTPUT_BYTES + 1)
418418
if (provider === 'e2b') {
419-
mockE2BCommandsRun.mockImplementationOnce(async (_cmd: string, options: any) => {
419+
mockE2BCommandsRun.mockImplementationOnce(async (_cmd, options) => {
420420
options.onStderr(oversized)
421421
})
422422
} else {

0 commit comments

Comments
 (0)