diff --git a/epiphany-openai-runtime/src/bin/epiphany-openai-runtime.rs b/epiphany-openai-runtime/src/bin/epiphany-openai-runtime.rs index fc6dd53b..381d4433 100644 --- a/epiphany-openai-runtime/src/bin/epiphany-openai-runtime.rs +++ b/epiphany-openai-runtime/src/bin/epiphany-openai-runtime.rs @@ -1482,17 +1482,6 @@ mod tests { ) } - #[test] - fn same_nonempty_tool_request_round_requires_real_repetition() { - let first = vec!["source::read_file::{\"path\":\"README.md\"}".to_string()]; - let second = vec!["source::git_show::{\"commit\":\"abc\"}".to_string()]; - - assert!(!same_nonempty_tool_request_round(None, &first)); - assert!(!same_nonempty_tool_request_round(Some(&first), &[])); - assert!(!same_nonempty_tool_request_round(Some(&first), &second)); - assert!(same_nonempty_tool_request_round(Some(&first), &first)); - } - #[test] fn tool_loop_guard_stalls_only_after_repeated_identical_rounds() { let mut guard = ToolLoopGuard::default(); diff --git a/epiphany-release-construction/src/lib.rs b/epiphany-release-construction/src/lib.rs index 90dee423..23442239 100644 --- a/epiphany-release-construction/src/lib.rs +++ b/epiphany-release-construction/src/lib.rs @@ -312,7 +312,14 @@ impl ReleaseSourceGuard { run_git_checked( &path, - &release_submodule_update_args(), + &[ + "-c", + "core.longpaths=true", + "submodule", + "update", + "--init", + "--recursive", + ], "failed to initialize exact release submodules", )?; reset_and_clean_cached_submodules(&path, "clean exact release submodules")?; @@ -341,17 +348,6 @@ impl ReleaseSourceGuard { } } -fn release_submodule_update_args() -> [&'static str; 6] { - [ - "-c", - "core.longpaths=true", - "submodule", - "update", - "--init", - "--recursive", - ] -} - fn reset_and_clean_cached_submodules(repo: &Path, context: &str) -> Result<()> { run_git_checked( repo, @@ -924,20 +920,6 @@ mod tests { assert!(linux.iter().all(|(_, name)| !name.ends_with(".exe"))); } - #[test] - fn release_bundle_cache_is_owned_by_build_graph() { - let root = Path::new("release-build-cache"); - let first = release_bundle_target_dir(root, b"lock-v1", "target-a", "toolchain-a"); - let second = release_bundle_target_dir(root, b"lock-v1", "target-a", "toolchain-a"); - assert_eq!(first, second); - assert!( - first - .file_name() - .and_then(|name| name.to_str()) - .is_some_and(|name| name.starts_with("graph-")) - ); - } - #[test] fn release_bundle_cache_is_stable_across_lockfile_edits() { let root = Path::new("release-build-cache"); @@ -948,23 +930,6 @@ mod tests { ); } - #[test] - fn cached_source_update_does_not_force_refresh_unchanged_submodules() { - let args = release_submodule_update_args(); - assert_eq!( - args, - [ - "-c", - "core.longpaths=true", - "submodule", - "update", - "--init", - "--recursive", - ] - ); - assert!(!args.contains(&"--force")); - } - #[test] fn release_bundle_cache_separates_targets_and_toolchains() { let root = Path::new("release-build-cache");