[Th3D0c0]

tests/test_for_each.h and tests/test_set_var.h each memnew a Node and never free it. Every other test in the suite that creates one pairs it with a memdelete — for example test_check_var.h, test_check_trigger.h and test_call_method.h — so these two look like oversights rather than intent.

doctest re-enters a TEST_CASE body once per leaf SUBCASE, so the cost is one leaked Node per leaf: 4 from BTForEach and 18 from BTSetVar.

Running the suite in a tests=yes build reports:

WARNING: 22 ObjectDB instances were leaked at exit (run with `--verbose` for details).

and on a build with use_asan=yes, LeakSanitizer fails the run outright:

Direct leak of 14400 byte(s) in 18 object(s) allocated from:
    #3 TestSetVar::DOCTEST_ANON_FUNC_523() modules/limboai/tests/test_set_var.h:30
Direct leak of 3200 byte(s) in 4 object(s) allocated from:
    #3 TestForEach::DOCTEST_ANON_FUNC_364() modules/limboai/tests/test_for_each.h:24
SUMMARY: AddressSanitizer: 17600 byte(s) leaked in 22 allocation(s).

With both memdeletes added, the full Godot test suite reports 0 leaked instances and the same number of passing cases as before.

🤖 Generated with Claude Code