Skip to content

test(gamepad): use the default runtime context - #1343

Merged
ldm0 merged 1 commit into
mainfrom
codex/fix-gamepad-test-runtime
Oct 9, 2026
Merged

ldm0 merged 1 commit into
mainfrom
codex/fix-gamepad-test-runtime

Conversation

@lanyue-llk

Copy link
Copy Markdown
Contributor

问题

新增 gamepad 事件载荷测试沿用了已移除的 page_default_context 字段。测试夹具现在通过 page_default_runtime.context 保存默认页面 context,因此 main 及基于 main 的临时合并检查在编译测试时触发 E0609;Clippy 和 Nextest 都无法运行。

修复

将测试中的 context 指针取址改为 page_default_runtime.context,与相邻的 TimeRanges、SVG geometry、canvas 和 DOMException 测试使用同一接口。保留原生品牌、代理品牌及事件载荷身份的全部断言;不改变运行时实现。

验证

  • 原失败日志:Clippy、Nextest。二者均在 gamepad_interfaces.rs:23 编译失败。
  • cargo fmt --all --check 通过。
  • cargo test -p moli-renderer-v8 --locked --lib script_vm::tests::gamepad_interfaces:::两个原有场景通过,8972 个其他测试未执行。
  • 完整仓库测试由 CI 执行。

此修复基于 main 的 19f388b4f1ac11164b21969c3342789012125e09,与内容转换 #1334–#1337 独立。

@ldm0
ldm0 merged commit b75fdf8 into main Oct 9, 2026
3 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants