Revert "test: extend testing suite to include skeleton, raycast and animation_player"

[ViTeXFTW]

• Reverts test: extend testing suite to include skeleton, raycast and animation_player #38 because if failing test on linux systems

Greptile Overview

Greptile Summary

This PR reverts #38, which added comprehensive test coverage for skeleton, raycast, animation player, hierarchy parser, animation parser, and HLod parser components. The revert removes ~2,460 lines of test code due to test failures on Linux systems.

What was removed:

• Three new test executables: skeleton_tests, bounding_box_tests, and raycast_tests
• Six test files covering critical w3d parsing and rendering functionality
• CMake configuration for building and linking these test suites
• Vulkan Y-flip adjustments (proj[1][1] *= -1.0f) in two raycast test cases

Key observations:

• This is a clean revert that completely backs out PR test: extend testing suite to include skeleton, raycast and animation_player #38
• The revert is justified by platform-specific test failures on Linux
• No production code is affected - only test infrastructure
• The tests were following TDD practices per CLAUDE.md guidelines

Recommendations:

• Investigate the root cause of Linux test failures before re-introducing these tests
• Consider running tests in CI on multiple platforms to catch platform-specific issues earlier
• The test coverage loss is significant - these tests should be fixed and re-added when the Linux issues are resolved

Confidence Score: 5/5

• This PR is safe to merge as it cleanly reverts problematic test code
• This is a clean revert commit that only removes test code, not production functionality. The revert is justified due to platform-specific test failures on Linux. No logic errors or breaking changes introduced.
• No files require special attention - this is a straightforward revert

Important Files Changed

Filename	Overview
tests/CMakeLists.txt	Removes test target configurations for skeleton, bounding box, and raycast tests, and removes three test sources from w3d_tests
tests/render/raycast_test.cpp	Removes Vulkan Y-flip corrections from two test cases
tests/render/test_animation_player.cpp	Completely removes comprehensive animation player test suite (446 lines)
tests/render/test_skeleton_pose.cpp	Completely removes skeleton pose computation test suite (341 lines)
tests/w3d/test_animation_parser.cpp	Completely removes animation parser test suite (493 lines)
tests/w3d/test_hierarchy_parser.cpp	Completely removes hierarchy parser test suite (371 lines)
tests/w3d/test_hlod_parser.cpp	Completely removes HLod parser test suite (427 lines)

Context used:

• Context from dashboard - CLAUDE.md (source)
• Context from dashboard - CLAUDE.md (source)

[greptile-apps]

_{7 files reviewed, no comments}

_{Edit Code Review Agent Settings | Greptile}