Skip to content

Unifying and Clearing Old Tests - #800

Open
whuang37 wants to merge 12 commits into
google:mainfrom
whuang37:test-cleanup
Open

whuang37 wants to merge 12 commits into
google:mainfrom
whuang37:test-cleanup

Conversation

@whuang37

Copy link
Copy Markdown
Collaborator

Description

Unifying all tests to follow Google testing convention. No real code changes

New development times (3 trials each):

Task Initial mean ± SD (s) Now mean ± SD (s)
Test 16.363 ± 0.512 8.230 ± 0.543
Format 7.160 ± 0.080 6.327 ± 0.146
Lint 0.160 ± 0.000 0.143 ± 0.006
Section Updates
Testing and coding standards Added coding and testing guidance to AGENTS.md,
Core and lifecycle Focused tests on startup, initialization, cancellation, disposal, renderer readiness, and resource cleanup. Combined related Core and WebGPU startup tests.
UI and text editing Removed repetitive widget, default-value, rendering-detail, and text-layout checks.
Input and gestures Replaced SDK method mocks with real input, hand, and gesture objects.
Interaction and manipulation Reduced repeated selection and configuration cases. Kept functional checks for capture, cancellation, scrolling, movement, rotation, resizing, and limits.
Simulator Simplified control, environment, hand-pose, and lighting tests. Combined control-mode tests that share one setup
World, perception, and context Reduced mock-heavy and repetitive tests across depth, object detection, anchors, scene context, camera, lighting, and video.
AI Removed mocks. kept important query, image-generation, token, connection, cancellation, and session-cleanup behavior.
XR video layers Combined repeated platform and lifecycle cases.
Addons Applied a more aggressive reduction to repeated configuration, schema, mock-orchestration, and implementation checks.
Demos Excluded demo tests from the default SDK Vitest run since it doesn't make sense to enforce SDK functionality based on demo tests.

Type of Change

  • Bug fix
  • New feature / enhancement
  • New demo or sample
  • Documentation update

Checklist

  • Tested in simulator & device: Verified functionality in desktop simulator and/or physical hardware (where applicable).
  • Large Assets ($\ge$ 1MB): Submitted separately to xrblocks/proprietary-assets via jsdelivr CDN.
  • SDK Dynamic Dependencies: All new SDK dependencies are dynamically loaded at runtime.
  • Security: Confirmed no hardcoded API keys or secrets are committed.

@whuang37
whuang37 requested review from dli7319 and ruofeidu November 13, 2026 01:20
@dli7319

dli7319 commented Nov 13, 2026

Copy link
Copy Markdown
Collaborator

Nice speedup!

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