fix(tests): port tests to v2 API, fix FATAL_CHECK logic

- Port test_effect_base to EffectV2/SequenceV2
- Port test_demo_effects to v2 effects only
- Remove v1 lifecycle helpers from effect_test_helpers
- Fix cnn_test to not depend on cnn_v1_effect.h
- Fix test_sequence_v2_e2e node redeclaration

Known issue: test_sequence_v2_e2e still fails with bind group error
(needs source/sink texture views set)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

1 files changed, 1 insertions, 42 deletions

diff --git a/src/tests/common/effect_test_helpers.cc b/src/tests/common/effect_test_helpers.cc
index 9250366..d776609 100644
--- a/src/tests/common/effect_test_helpers.cc
+++ b/src/tests/common/effect_test_helpers.cc
@@ -1,9 +1,8 @@
 // This file is part of the 64k demo project.
 // It implements reusable test helpers for GPU effect testing.
-// Provides pixel validation and lifecycle testing utilities.
+// Provides pixel validation utilities.
 
 #include "effect_test_helpers.h"
-#include "gpu/effect.h"
 #include <cassert>
 
 // ============================================================================
@@ -68,43 +67,3 @@ uint64_t hash_pixels(const std::vector<uint8_t>& pixels) {
   return hash;
 }
 
-// ============================================================================
-// Effect Lifecycle Helpers
-// ============================================================================
-
-bool test_effect_lifecycle(Effect* effect, MainSequence* main_seq) {
-  assert(effect && "Effect pointer is null");
-  assert(main_seq && "MainSequence pointer is null");
-
-  // Check initial state
-  if (effect->is_initialized) {
-    return false; // Should not be initialized yet
-  }
-
-  // Initialize effect
-  effect->init(main_seq);
-
-  // Check initialized state
-  if (!effect->is_initialized) {
-    return false; // Should be initialized now
-  }
-
-  return true; // Lifecycle test passed
-}
-
-bool test_effect_render_smoke(Effect* effect) {
-  assert(effect && "Effect pointer is null");
-
-  // Smoke test: Just call render with dummy parameters
-  // 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
-  }
-
-  // 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)
-}