Skip to content

tests: cover Vulkan RTX world extraction#265

Draft
cursor[bot] wants to merge 1 commit into
next-gen-5from
cursor/missing-test-coverage-d4fa
Draft

tests: cover Vulkan RTX world extraction#265
cursor[bot] wants to merge 1 commit into
next-gen-5from
cursor/missing-test-coverage-d4fa

Conversation

@cursor

@cursor cursor Bot commented Jun 16, 2026

Copy link
Copy Markdown

Risky behavior now covered

  • Vulkan RTX world BLAS CPU extraction now has deterministic unit coverage for supported BSP face and triangle-soup surfaces.
  • The tests cover malformed/unsupported surfaces being skipped, primitive count capping, cap-limited packing, and stable vertex/index emission order.

Test files added/updated

  • Added tests/unit/test_vk_rtx_world.c.
  • Registered unit_vk_rtx_world in CMakeLists.txt with USE_VULKAN_RTX=1.
  • Updated tests/README.md so the new target is discoverable.

Why this materially reduces regression risk

  • RTX BLAS extraction is renderer-critical and feeds acceleration-structure geometry; off-by-one cap handling or bad surface parsing can produce missing geometry, corrupt ray hits, or GPU-side faults.
  • The new test exercises the production helper directly without requiring Vulkan hardware or game data, keeping coverage deterministic for CI/headless agents.

Validation

  • cmake -S . -B build-coverage-tests -DBUILD_TESTING=ON -DBUILD_UNIT_TESTS=ON -DUSE_VULKAN_RTX=ON && cmake --build build-coverage-tests --target unit_vk_rtx_world
  • ctest --test-dir build-coverage-tests -R unit_vk_rtx_world --output-on-failure
  • cmake --build build-coverage-tests --target unit_vm_native_module unit_vk_rtx_world && ctest --test-dir build-coverage-tests -R 'unit_(vm_native_module|vk_rtx_world)' --output-on-failure
Open in Web View Automation 

Co-authored-by: Tim Fox <timfox@outlook.com>
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.

1 participant