Skip to content

filter out part of codex installation to prevent codex test from timing out #762

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
lilly-luo merged 1 commit into from
Sep 19, 2026
+8 −3
Merged

filter out part of codex installation to prevent codex test from timing out #762

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions tests/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -78,6 +78,8 @@ varies
argument spelling or routing mode, never hides the agent/provider in the test name. Duplicate boot-only cases
are incorporated into the Databricks configuration TUI journeys.
Generated-file cleanup and strict app-server stdout assertions remain enforced.
Managed Codex state comparisons exclude `.codex/tmp/arg0`, the disposable executable
links recreated by Codex version checks; persistent agent files remain compared.

ug no longer runs a post-configure agent probe; the deprecated `--skip-validate`
flag is accepted as a no-op where older journeys still pass it. Tests retain
Expand Down
5 changes: 3 additions & 2 deletions tests/integration/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -299,8 +299,9 @@ module fetches the workspace's published config once, replaces Claude's static m
across all configured/fresh scenarios. The Codex module does the same with
`main.default.ci_e2e_openai_mps`. The tests verify Claude's admin header, native cache and real
model picker, Codex's exact app-server catalog, and both agents' rejection of personal source
overrides. In addition,
`test_ug_configure_managed_codex_catalog_fallback` injects the intentionally nonexistent
overrides. Codex state comparisons exclude `.codex/tmp/arg0`, the disposable executable links
recreated by version checks, while continuing to compare persistent agent files.
In addition, `test_ug_configure_managed_codex_catalog_fallback` injects the intentionally nonexistent
`system.ai.gpt-99`, keeping it out of the real workspace while launching Codex through that
workspace on the valid default model `system.ai.gpt-5-6-sol`. With smart routing enabled, it opens
the real Codex `/models` picker and requires that injected custom-catalog model to be listed. The
Expand Down
4 changes: 3 additions & 1 deletion tests/integration/test_ug_codex_managed_model_discovery.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -47,8 +47,10 @@ def _codex_state_and_agent_files(session):
for path in paths
if path.is_file()
# A fresh launch must retrieve and cache the control-plane input before it can reject an
# override. Exclude only that expected cache; every agent-owned state/file stays compared.
# override. Exclude that expected cache and Codex's disposable arg0 executable links,
# which even a version check recreates; persistent agent-owned files stay compared.
and not is_managed_config_control_plane_cache(session.home, path)
and not path.is_relative_to(session.home / ".codex" / "tmp" / "arg0")
}


Expand Down
Loading