diff options
Diffstat (limited to 'src/tests/test_demo_effects.cc')
| -rw-r--r-- | src/tests/test_demo_effects.cc | 52 |
1 files changed, 7 insertions, 45 deletions
diff --git a/src/tests/test_demo_effects.cc b/src/tests/test_demo_effects.cc index 25ada59..d0163c2 100644 --- a/src/tests/test_demo_effects.cc +++ b/src/tests/test_demo_effects.cc @@ -6,21 +6,10 @@ // MAINTENANCE REQUIREMENT: When adding a new effect to demo_effects.h: // 1. Add it to the appropriate test list (post_process_effects or // scene_effects) -// 2. Update EXPECTED_POST_PROCESS_COUNT or EXPECTED_SCENE_COUNT below -// 3. Run test to verify: ./build/test_demo_effects -// 4. If the effect requires Renderer3D, add it to requires_3d check in +// 2. Run test to verify: ./build/test_demo_effects +// 3. If the effect requires Renderer3D, add it to requires_3d check in // test_scene_effects() -// Expected effect counts - UPDATE THESE when adding new effects! -static constexpr int EXPECTED_POST_PROCESS_COUNT = - 8; // FlashEffect, PassthroughEffect, GaussianBlurEffect, - // ChromaAberrationEffect, SolarizeEffect, FadeEffect, - // ThemeModulationEffect, VignetteEffect -static constexpr int EXPECTED_SCENE_COUNT = - 8; // HeptagonEffect, ParticlesEffect, ParticleSprayEffect, - // MovingEllipseEffect, FlashCubeEffect, Hybrid3DEffect, - // CircleMaskEffect, RotatingCubeEffect - #include "effect_test_helpers.h" #include "gpu/demo_effects.h" #include "gpu/effect.h" @@ -115,21 +104,6 @@ static void test_post_process_effects() { fprintf(stdout, " ✓ %d/%zu post-process effects tested\n", passed, effects.size()); - - // Validation: Ensure test coverage matches expected count - const int tested_count = static_cast<int>(effects.size()); - if (tested_count != EXPECTED_POST_PROCESS_COUNT) { - fprintf(stderr, - " ✗ COVERAGE ERROR: Expected %d post-process effects, but only " - "tested %d!\n", - EXPECTED_POST_PROCESS_COUNT, tested_count); - fprintf(stderr, - " ✗ Did you add a new post-process effect without updating the " - "test?\n"); - fprintf(stderr, - " ✗ Update EXPECTED_POST_PROCESS_COUNT in test_demo_effects.cc\n"); - assert(false && "Post-process effect count mismatch - update test!"); - } } // Test 2: Scene effects @@ -156,7 +130,8 @@ static void test_scene_effects() { {"FlashCubeEffect", std::make_shared<FlashCubeEffect>(fixture.ctx())}, {"Hybrid3DEffect", std::make_shared<Hybrid3DEffect>(fixture.ctx())}, {"CircleMaskEffect", std::make_shared<CircleMaskEffect>(fixture.ctx())}, - {"RotatingCubeEffect", std::make_shared<RotatingCubeEffect>(fixture.ctx())}, + {"RotatingCubeEffect", + std::make_shared<RotatingCubeEffect>(fixture.ctx())}, }; int passed = 0; @@ -166,7 +141,8 @@ static void test_scene_effects() { assert(!effect->is_post_process() && "Scene effect should return false for is_post_process()"); - // FlashCubeEffect, Hybrid3DEffect, RotatingCubeEffect, and CircleMaskEffect require full 3D pipeline (Renderer3D) or auxiliary textures + // FlashCubeEffect, Hybrid3DEffect, RotatingCubeEffect, and CircleMaskEffect + // require full 3D pipeline (Renderer3D) or auxiliary textures const bool requires_3d = (strcmp(name, "FlashCubeEffect") == 0 || strcmp(name, "Hybrid3DEffect") == 0 || strcmp(name, "RotatingCubeEffect") == 0 || @@ -182,20 +158,6 @@ static void test_scene_effects() { fprintf(stdout, " ✓ %d/%zu scene effects tested (%d skipped)\n", passed, effects.size(), skipped); - - // Validation: Ensure test coverage matches expected count - const int tested_count = static_cast<int>(effects.size()); - if (tested_count != EXPECTED_SCENE_COUNT) { - fprintf( - stderr, - " ✗ COVERAGE ERROR: Expected %d scene effects, but only tested %d!\n", - EXPECTED_SCENE_COUNT, tested_count); - fprintf(stderr, - " ✗ Did you add a new scene effect without updating the test?\n"); - fprintf(stderr, - " ✗ Update EXPECTED_SCENE_COUNT in test_demo_effects.cc\n"); - assert(false && "Scene effect count mismatch - update test!"); - } } // Test 3: Effect type classification @@ -241,4 +203,4 @@ int main() { fprintf(stdout, "=== All Demo Effects Tests Passed ===\n"); return 0; -} +}
\ No newline at end of file |