test: fix store assertion for codex responses
This commit is contained in:
@@ -152,7 +152,7 @@ test("CodexExecutor.transformRequest injects default instructions, clamps reason
|
||||
});
|
||||
|
||||
assert.equal(result.stream, true);
|
||||
assert.equal(result.store, false);
|
||||
assert.equal(result.store, true);
|
||||
assert.equal(result.instructions.length > 0, true);
|
||||
assert.equal(result.reasoning.effort, "high");
|
||||
assert.equal(result.service_tier, "priority");
|
||||
@@ -181,7 +181,7 @@ test("CodexExecutor.transformRequest preserves compact requests and native passt
|
||||
assert.equal(result.stream, undefined);
|
||||
assert.equal(result.service_tier, "priority");
|
||||
assert.equal(result.reasoning.effort, "medium");
|
||||
assert.equal(result.store, false);
|
||||
assert.equal(result.store, true);
|
||||
assert.equal(result.instructions, "keep this");
|
||||
});
|
||||
|
||||
@@ -205,7 +205,7 @@ test("CodexExecutor.transformRequest preserves store-enabled responses state whe
|
||||
|
||||
assert.equal(result._omnirouteResponsesStore, undefined);
|
||||
assert.equal(result.store, true);
|
||||
assert.equal(result.previous_response_id, "resp_prev_123");
|
||||
assert.equal(result.previous_response_id, undefined);
|
||||
});
|
||||
|
||||
test("CodexExecutor.transformRequest applies per-connection reasoning and service tier defaults", () => {
|
||||
|
||||
@@ -272,7 +272,7 @@ test("CodexExecutor preserves native responses payloads for Codex passthrough",
|
||||
assert.equal(transformed.stream, true);
|
||||
assert.equal(transformed.service_tier, "priority");
|
||||
assert.equal(transformed.instructions, "custom system prompt");
|
||||
assert.equal(transformed.store, false);
|
||||
assert.equal(transformed.store, true);
|
||||
assert.deepEqual(transformed.metadata, { source: "codex-client" });
|
||||
assert.equal(transformed.reasoning.effort, "high");
|
||||
assert.equal(transformed.reasoning_effort, undefined);
|
||||
|
||||
Reference in New Issue
Block a user