mirror of
https://github.com/microsoft/mscclpp.git
synced 2026-05-13 09:46:00 +00:00
Address review comments: 1. Ensure TearDown() is always called if SetUp() succeeds, even when TestBody() throws. This prevents resource leaks and maintains MPI synchronization between tests. 2. Replace assert() in fifo_perf_tests.cu with proper return false on validation failure, ensuring consistent test failure reporting. Fixes: - test/framework.cc: Track SetUp success and call TearDown in finally-style - test/unit/fifo_perf_tests.cu: Replace assert with explicit check Co-authored-by: chhwang <8018170+chhwang@users.noreply.github.com>