diff options
Diffstat (limited to 'src/tests/effect_test_helpers.cc')
| -rw-r--r-- | src/tests/effect_test_helpers.cc | 41 |
1 files changed, 15 insertions, 26 deletions
diff --git a/src/tests/effect_test_helpers.cc b/src/tests/effect_test_helpers.cc index a31c447..9250366 100644 --- a/src/tests/effect_test_helpers.cc +++ b/src/tests/effect_test_helpers.cc @@ -2,8 +2,6 @@ // It implements reusable test helpers for GPU effect testing. // Provides pixel validation and lifecycle testing utilities. -#if !defined(STRIP_ALL) // Test code only - zero size impact on final binary - #include "effect_test_helpers.h" #include "gpu/effect.h" #include <cassert> @@ -13,27 +11,24 @@ // ============================================================================ bool validate_pixels( - const std::vector<uint8_t>& pixels, - int width, - int height, + const std::vector<uint8_t>& pixels, int width, int height, std::function<bool(uint8_t r, uint8_t g, uint8_t b, uint8_t a)> predicate) { const size_t pixel_count = width * height; for (size_t i = 0; i < pixel_count; ++i) { - const size_t offset = i * 4; // BGRA8 = 4 bytes/pixel + const size_t offset = i * 4; // BGRA8 = 4 bytes/pixel const uint8_t b = pixels[offset + 0]; const uint8_t g = pixels[offset + 1]; const uint8_t r = pixels[offset + 2]; const uint8_t a = pixels[offset + 3]; if (predicate(r, g, b, a)) { - return true; // At least one pixel matches + return true; // At least one pixel matches } } - return false; // No pixels matched + return false; // No pixels matched } -bool has_rendered_content(const std::vector<uint8_t>& pixels, - int width, +bool has_rendered_content(const std::vector<uint8_t>& pixels, int width, int height) { return validate_pixels(pixels, width, height, [](uint8_t r, uint8_t g, uint8_t b, uint8_t a) { @@ -41,13 +36,9 @@ bool has_rendered_content(const std::vector<uint8_t>& pixels, }); } -bool all_pixels_match_color(const std::vector<uint8_t>& pixels, - int width, - int height, - uint8_t target_r, - uint8_t target_g, - uint8_t target_b, - uint8_t tolerance) { +bool all_pixels_match_color(const std::vector<uint8_t>& pixels, int width, + int height, uint8_t target_r, uint8_t target_g, + uint8_t target_b, uint8_t tolerance) { const size_t pixel_count = width * height; for (size_t i = 0; i < pixel_count; ++i) { const size_t offset = i * 4; @@ -61,10 +52,10 @@ bool all_pixels_match_color(const std::vector<uint8_t>& pixels, if (diff_r * diff_r + diff_g * diff_g + diff_b * diff_b > tolerance * tolerance) { - return false; // At least one pixel doesn't match + return false; // At least one pixel doesn't match } } - return true; // All pixels match + return true; // All pixels match } uint64_t hash_pixels(const std::vector<uint8_t>& pixels) { @@ -87,7 +78,7 @@ bool test_effect_lifecycle(Effect* effect, MainSequence* main_seq) { // Check initial state if (effect->is_initialized) { - return false; // Should not be initialized yet + return false; // Should not be initialized yet } // Initialize effect @@ -95,10 +86,10 @@ bool test_effect_lifecycle(Effect* effect, MainSequence* main_seq) { // Check initialized state if (!effect->is_initialized) { - return false; // Should be initialized now + return false; // Should be initialized now } - return true; // Lifecycle test passed + return true; // Lifecycle test passed } bool test_effect_render_smoke(Effect* effect) { @@ -108,14 +99,12 @@ bool test_effect_render_smoke(Effect* effect) { // If this doesn't crash, consider it a success // Note: This requires the effect to be initialized first if (!effect->is_initialized) { - return false; // Cannot render uninitialized effect + return false; // Cannot render uninitialized effect } // We cannot actually render without a full render pass setup // This is a placeholder for more sophisticated render testing // Real render tests should use OffscreenRenderTarget - return true; // Smoke test passed (no crash) + return true; // Smoke test passed (no crash) } - -#endif /* !defined(STRIP_ALL) */ |