Re-enable SDK image forwarding test (#5934)
## Summary - re-enable the TypeScript SDK test that verifies local images are forwarded to `codex exec` ## Testing - `pnpm test` *(fails: unable to download pnpm 10.8.1 because external network access is blocked in the sandbox)* ------ https://chatgpt.com/codex/tasks/task_i_690289cb861083209fd006867e2adfb1
This commit is contained in:
parent
3429e82e45
commit
bf35105af6
4 changed files with 7 additions and 8 deletions
|
|
@ -1,6 +1,8 @@
|
|||
import path from "node:path";
|
||||
|
||||
export function codexPathOverride() {
|
||||
return process.env.CODEX_EXECUTABLE ??
|
||||
path.join(process.cwd(), "..", "..", "codex-rs", "target", "debug", "codex");
|
||||
return (
|
||||
process.env.CODEX_EXECUTABLE ??
|
||||
path.join(process.cwd(), "..", "..", "codex-rs", "target", "debug", "codex")
|
||||
);
|
||||
}
|
||||
|
|
|
|||
|
|
@ -115,7 +115,6 @@ export type TodoListItem = {
|
|||
items: TodoItem[];
|
||||
};
|
||||
|
||||
|
||||
/** Canonical union of thread items and their type-specific payloads. */
|
||||
export type ThreadItem =
|
||||
| AgentMessageItem
|
||||
|
|
|
|||
|
|
@ -21,8 +21,7 @@ export async function createOutputSchemaFile(schema: unknown): Promise<OutputSch
|
|||
const cleanup = async () => {
|
||||
try {
|
||||
await fs.rm(schemaDir, { recursive: true, force: true });
|
||||
}
|
||||
catch {
|
||||
} catch {
|
||||
// suppress
|
||||
}
|
||||
};
|
||||
|
|
|
|||
|
|
@ -3,7 +3,7 @@ import os from "node:os";
|
|||
import path from "node:path";
|
||||
|
||||
import { codexExecSpy } from "./codexExecSpy";
|
||||
import { describe, expect, it, xit } from "@jest/globals";
|
||||
import { describe, expect, it } from "@jest/globals";
|
||||
|
||||
import { Codex } from "../src/codex";
|
||||
|
||||
|
|
@ -308,8 +308,7 @@ describe("Codex", () => {
|
|||
await close();
|
||||
}
|
||||
});
|
||||
// TODO(pakrym): unskip the test
|
||||
xit("forwards images to exec", async () => {
|
||||
it("forwards images to exec", async () => {
|
||||
const { url, close } = await startResponsesTestProxy({
|
||||
statusCode: 200,
|
||||
responseBodies: [
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue