mirror of
https://github.com/openai/codex.git
synced 2026-06-01 19:02:59 +00:00
Remove unnecessary tests (#17395)
# External (non-OpenAI) Pull Request Requirements Before opening this Pull Request, please read the dedicated "Contributing" markdown file or your PR may be closed: https://github.com/openai/codex/blob/main/docs/contributing.md If your PR conforms to our contribution guidelines, replace this text with a detailed and high quality description of your changes. Include a link to a bug report or enhancement request.
This commit is contained in:
@@ -53,58 +53,6 @@ fn build_stage_one_input_message_uses_default_limit_when_model_context_window_mi
|
||||
assert!(message.contains(&expected_truncated));
|
||||
}
|
||||
|
||||
#[test]
|
||||
fn build_consolidation_prompt_renders_embedded_template() {
|
||||
let temp = tempdir().unwrap();
|
||||
let memories_dir = temp.path().join("memories");
|
||||
|
||||
let prompt = build_consolidation_prompt(&memories_dir, &Phase2InputSelection::default());
|
||||
|
||||
assert!(prompt.contains(&format!(
|
||||
"Folder structure (under {}/):",
|
||||
memories_dir.display()
|
||||
)));
|
||||
assert!(!prompt.contains("Memory extensions (under"));
|
||||
assert!(!prompt.contains("<extension_name>/instructions.md"));
|
||||
assert!(prompt.contains("**Diff since last consolidation:**"));
|
||||
assert!(prompt.contains("- selected inputs this run: 0"));
|
||||
}
|
||||
|
||||
#[tokio::test]
|
||||
async fn build_consolidation_prompt_points_to_extensions_without_inlining_them() {
|
||||
let temp = tempdir().unwrap();
|
||||
let memories_dir = temp.path().join("memories");
|
||||
let extension_dir = temp.path().join("memories_extensions/tape_recorder");
|
||||
tokio_fs::create_dir_all(extension_dir.join("resources"))
|
||||
.await
|
||||
.unwrap();
|
||||
tokio_fs::write(
|
||||
extension_dir.join("instructions.md"),
|
||||
"source-specific instructions\n",
|
||||
)
|
||||
.await
|
||||
.unwrap();
|
||||
tokio_fs::write(
|
||||
extension_dir.join("resources/notes.md"),
|
||||
"source-specific resource\n",
|
||||
)
|
||||
.await
|
||||
.unwrap();
|
||||
|
||||
let prompt = build_consolidation_prompt(&memories_dir, &Phase2InputSelection::default());
|
||||
let memory_extensions_dir = temp.path().join("memories_extensions");
|
||||
|
||||
assert!(prompt.contains(&format!(
|
||||
"Memory extensions (under {}/)",
|
||||
memory_extensions_dir.display()
|
||||
)));
|
||||
assert!(prompt.contains(&format!("Under `{}/`:", memory_extensions_dir.display())));
|
||||
assert!(prompt.contains("<extension_name>/instructions.md"));
|
||||
assert!(prompt.contains("Optional source-specific inputs:"));
|
||||
assert!(!prompt.contains("source-specific instructions"));
|
||||
assert!(!prompt.contains("source-specific resource"));
|
||||
}
|
||||
|
||||
#[tokio::test]
|
||||
async fn build_memory_tool_developer_instructions_renders_embedded_template() {
|
||||
let temp = tempdir().unwrap();
|
||||
|
||||
Reference in New Issue
Block a user