Files
Unlucky-Launcher/crates/cli/tests/adopt_test/collision_cases.rs
T
AskaEth 347a3cc90d fix(game): 复审 r2 三项——Win32 尾点/尾空格别名 + 写面与游戏树重叠 + 更高祖先断链
r1 五项里 R4/R5 与 slug/大小写等直接案例已由 r2 复审独立通过;这三条是它补证出的**剩余边界**,
只做限定修复,不推翻重写:

r2.1(未存在认领路径的 Win32 尾点/尾空格别名):比较键新增 `fspaths::comparison_real()`——
已存在前缀用 `canonicalize`,**尚不存在的尾段按平台语义归一**:Windows 普通路径折叠尾 `.`/空格
(`Foo.` 与 `Foo ` 都是 `Foo`,Win32 建目录时就这么做),**verbatim 路径(`\?\…`)不折叠**
(那是另一个物理目录,合法)。原先 `dir_key` 只统一分隔符与大小写,于是一个尚未创建的
`custom` 认领 `Foo.` 与缺省派生 `Foo` 键不相等却被判定「不撞」,两个实例共用一个运行目录。

r2.2(`data`/`.ULL` 整体映射到受保护游戏树时仍静默外写):新增
`fspaths::ensure_surfaces_clear_of_game_tree()`——默认写盘面(`.ULL`、`config/`、
`config/instances/`、`profiles.json`、`data/`,以及缺省档的运行目录)**解析后不得落进游戏根**;
`plan`(dry-run 也报)与 `adopt`(写盘前)各核一次,重叠即 `adopt_write_surface_in_game_tree`
具名拒绝并提示把数据目录挪到游戏树之外。整体迁移到游戏树**之外**照旧允许(判据是落点不是链接)。
配置写面判据同时收窄职责:`ensure_config_write_surface` 只管「在 `.ULL` 内」,
不再带「`.ULL` 在游戏根里」的豁免特例——重叠由上面那条统一负责。

r2.3(游戏根更高祖先断链仍当空列表):`scan` 的 `NotFound` 分支改为沿祖先找**最近的已存在入口**,
它能 `lstat` 到但 `canonicalize` 不出来(断链的符号链接/目录联接、目标被搬走)⇒
`adopt_versions_unreadable`;一路到盘根都不存在才是「真正的新数据根」,空结果仍允许。
原先只查 `versions` 与游戏根两层,迁移联接在更高层断开时会误报空库。

新错误码:`adopt_write_surface_in_game_tree`(CLI hint 与 help 拒绝清单同步)。

验证(持久日志在 `target/m221-r2/`,git 忽略、不入库;上一轮 `target/m221-r1/` 证据未覆盖):
- `cargo fmt --all --check` 0;`cargo check -p unlucky-core -p unlucky-cli --all-targets` 0;
- `cargo test -p unlucky-core --test adopt` **43 passed / 0 failed**(另跑一次
  `ULL_ADOPT_TEST_SYMLINK_REQUIRE=1` 同样 43 passed);
- `cargo test -p unlucky-cli --test adopt_test` **13 passed / 0 failed**;
- `cargo clippy -p unlucky-core -p unlucky-cli --all-targets -- -D warnings` 0;
- 额外:`cargo test -p unlucky-core --lib` **778 passed**、`cargo test -p unlucky-cli` 全部目标通过;
- 三次**实现-only 变异**(不折叠 Win32 尾点尾空格 / 不核游戏树重叠 / NotFound 直接当空)
  全部以**断言红**收场(1 / 4 / 2 条失败,非编译错误、非环境错误),每次还原后与基线逐文件一致,
  收尾再跑 core+cli 双绿。

界限:只覆盖 core+cli 的定向目标(不等于 workspace 全量);没有启动真实 Minecraft;GUI 未接;
判据是检查那一刻的快照(TOCTOU 看不到);Win32 8.3 短名与非 junction 重解析点未覆盖。
2026-11-09 23:56:02 +08:00

196 lines
6.1 KiB
Rust
Raw Blame History

This file contains ambiguous Unicode characters
This file contains Unicode characters that might be confused with other characters. If you think that this is intentional, you can safely ignore this warning. Use the Escape button to reveal them.
//! 运行目录**碰撞**的 CLI 面:slug 同名 / 大小写不敏感不许让两个实例共用一个目录(复审 r1 的 R1)。
//!
//! 从 `tests/adopt_test.rs` 拆出来只为守 400 行硬线;用例体未改。
use std::path::Path;
use crate::support::{parse_json, run_cli, TempDir};
use crate::fixture::foreign_data_root;
/// **slug 碰撞的两个展示名不能绑定同一个物理运行目录**(R1 的 CLI 面)。
///
/// 复审 r1 的实跑:`Better MC` 与 `Better-MC` 都成功、两个 UUID、却指向同一个目录,
/// 随后 `launch --dry-run` 的 `--gameDir` 也是同一个。这条把它钉住。
///
/// 抓得住:
/// - 登记侧不查已有实例占用的目录(两个 `run_dir` 相同,第一条断言红);
/// - `launch` 的 `game_dir` 与登记的目录不一致(第三条断言红)。
///
/// 抓不住:`pack_dir` 的 slug 规则本身(既有行为,本 mission 不改)。
#[test]
fn colliding_slug_names_get_separate_run_dirs() {
let dir = TempDir::new("adopt-slug");
let root = dir.path();
foreign_data_root(root);
let dir_arg = dir.arg();
let first = run_cli(&[
"instances",
"adopt",
"1.20.1",
"--name",
"Better MC",
"--json",
"--data-dir",
dir_arg.as_str(),
]);
assert_eq!(
first.0, 0,
"第一次登记应成功:stdout={} stderr={}",
first.1, first.2
);
let first = parse_json(&first.1);
let second = run_cli(&[
"instances",
"adopt",
"1.20.1",
"--name",
"Better-MC",
"--json",
"--data-dir",
dir_arg.as_str(),
]);
assert_eq!(
second.0, 0,
"第二次登记应成功:stdout={} stderr={}",
second.1, second.2
);
let second = parse_json(&second.1);
let first_dir = first["data"]["instance"]["run_dir"]
.as_str()
.expect("run_dir A")
.to_string();
let second_dir = second["data"]["instance"]["run_dir"]
.as_str()
.expect("run_dir B")
.to_string();
assert_ne!(first_dir, second_dir, "两个实例不许共用运行目录");
assert!(
Path::new(&first_dir).is_dir() && Path::new(&second_dir).is_dir(),
"两个运行目录都要真实存在"
);
// 真启动计划(dry-run)也要分开:两条实例各自的 game_dir 不同。
for (value, expect) in [(&first, &first_dir), (&second, &second_dir)] {
let id = value["data"]["instance"]["id"]
.as_str()
.expect("instance id");
let args = [
"launch",
id,
"--dry-run",
"--json",
"--data-dir",
dir_arg.as_str(),
];
let (code, stdout, stderr) = run_cli(&args);
assert_eq!(
code, 0,
"stdout={stdout}
stderr={stderr}"
);
assert_eq!(
parse_json(&stdout)["data"]["game_dir"]
.as_str()
.expect("game_dir"),
expect.as_str()
);
}
}
/// **Win32 尾点/尾空格**(r2 R2.1 的 CLI 面):两次登记 + 两条 `launch --dry-run` 不许落到同一物理目录。
///
/// 抓得住:认领目录的尾点/尾空格不归一(第二次默认登记仍拿到 `Foo`,`run_dir` 相同断言红)。
///
/// 抓不住:`\?\` verbatim 写法(那本来就不折叠,是另一个目录)。
#[cfg(windows)]
#[test]
fn win32_trailing_dot_or_space_claims_do_not_share_a_run_dir() {
for suffix in [".", " "] {
let dir = TempDir::new("adopt-win32-alias");
let root = dir.path();
foreign_data_root(root);
let dir_arg = dir.arg();
let claimed = root
.join(".ULL/data/instances")
.join(format!("Foo{suffix}"));
let claimed_arg = claimed.display().to_string();
let first = run_cli(&[
"instances",
"adopt",
"1.20.1",
"--name",
"First",
"--isolation",
"custom",
"--game-dir",
claimed_arg.as_str(),
"--json",
"--data-dir",
dir_arg.as_str(),
]);
assert_eq!(
first.0, 0,
"custom 登记应成功:stdout={} stderr={}",
first.1, first.2
);
let first = parse_json(&first.1);
let second = run_cli(&[
"instances",
"adopt",
"1.20.1",
"--name",
"Foo",
"--json",
"--data-dir",
dir_arg.as_str(),
]);
assert_eq!(
second.0, 0,
"默认登记应成功:stdout={} stderr={}",
second.1, second.2
);
let second = parse_json(&second.1);
let first_dir = first["data"]["instance"]["run_dir"]
.as_str()
.expect("run_dir A")
.to_string();
let second_dir = second["data"]["instance"]["run_dir"]
.as_str()
.expect("run_dir B")
.to_string();
assert_ne!(
first_dir, second_dir,
"尾点/尾空格别名不许共用运行目录(suffix={suffix:?})"
);
// 两条启动计划的 --gameDir 也必须分开(argv 已脱敏,比尾段)。
for (value, expect) in [(&first, &first_dir), (&second, &second_dir)] {
let id = value["data"]["instance"]["id"]
.as_str()
.expect("instance id");
let args = [
"launch",
id,
"--dry-run",
"--json",
"--data-dir",
dir_arg.as_str(),
];
let (code, stdout, stderr) = run_cli(&args);
assert_eq!(code, 0, "stdout={stdout} stderr={stderr}");
let plan = parse_json(&stdout);
assert_eq!(
plan["data"]["game_dir"].as_str().expect("game_dir"),
expect.as_str(),
"启动计划必须落在自己的运行目录(suffix={suffix:?})"
);
}
}
}