Delete redundant helper-spelling tests
This commit is contained in:
parent
391a5d7fd7
commit
f3360248fd
2 changed files with 8 additions and 54 deletions
|
|
@ -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();
|
||||
|
|
|
|||
|
|
@ -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");
|
||||
|
|
|
|||
Reference in a new issue