Address code review comments

- Remove duplicate static getMPIRank() and getMPISize() functions
- Add full namespace qualification to GTEST_SKIP macro

Co-authored-by: chhwang <8018170+chhwang@users.noreply.github.com>

Author: Copilot

test/framework.cc

@@ -37,10 +37,6 @@ static void finalizeMPI() {
   g_mpi_initialized = false;
 }
 
-static int getMPIRank() { return g_mpi_rank; }
-
-static int getMPISize() { return g_mpi_size; }
-
 static bool isMainProcess() { return g_mpi_rank == 0; }
 
 // Public utility functions for test output

test/framework.hpp

@@ -397,7 +397,7 @@ class SkipHelper {
   std::ostringstream message_;
 };
 
-#define GTEST_SKIP() ::SkipHelper(__FILE__, __LINE__)
+#define GTEST_SKIP() ::mscclpp::test::SkipHelper(__FILE__, __LINE__)
 
 // Create a namespace alias for compatibility with GTest code
 namespace testing = ::mscclpp::test;