Skip to content

Commit 2e54ffb

Browse files
facontidavideclaude
andcommitted
Add Phase 4 edge case tests for LoopNode and Reactive nodes
New tests: - gtest_loop.cpp: 15 tests for LoopNode decorator - Static queue parsing (int, double, string, bool) - Empty queue handling (if_empty parameter) - Dynamic queue from blackboard - Vector input conversion (Issue #969) - Child failure behavior - Restart after completion - convertFromString tests for SharedQueue types - gtest_reactive.cpp: 7 additional tests - ReactiveSequence first child fails - ReactiveSequence halt on condition change - ReactiveFallback first child succeeds - ReactiveFallback all children fail - ReactiveSequence all children succeed - ReactiveSequence re-evaluates on every tick Coverage improvements: - loop_node.h: 65.9% -> 98.8% (+32.9%) - reactive_fallback.cpp: 69.2% -> 74.4% (+5.2%) - Overall: 76.6% -> 77.1% - Tests: 420 -> 442 (+22) Co-Authored-By: Claude Opus 4.5 <[email protected]>
1 parent f3c490c commit 2e54ffb

File tree

3 files changed

+683
-0
lines changed

3 files changed

+683
-0
lines changed

tests/CMakeLists.txt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@ set(BT_TESTS
2424
gtest_name_validation.cpp
2525
gtest_json.cpp
2626
gtest_loggers.cpp
27+
gtest_loop.cpp
2728
gtest_reactive.cpp
2829
gtest_reactive_backchaining.cpp
2930
gtest_sequence.cpp

0 commit comments

Comments
 (0)