Skip to content

Commit fc89c73

Browse files
committed
refactor util tests: remove redundant OpenMP thread control checks to simplify test logic
1 parent 51b9f39 commit fc89c73

File tree

1 file changed

+0
-4
lines changed

1 file changed

+0
-4
lines changed

modules/util/tests/util.cpp

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -22,10 +22,6 @@ TEST(UtilTest, GetNumThreads_WithOpenMPEnvironment_HandlesThreadControlCorrectly
2222
if (num_threads_env_var.has_value()) {
2323
// When PPC_NUM_THREADS is set, GetNumThreads() should return that value
2424
EXPECT_EQ(ppc::util::GetNumThreads(), num_threads_env_var.value());
25-
26-
// And after setting OpenMP threads, it should match
27-
omp_set_num_threads(num_threads_env_var.value());
28-
EXPECT_EQ(ppc::util::GetNumThreads(), omp_get_max_threads());
2925
} else {
3026
// When PPC_NUM_THREADS is not set, GetNumThreads() should return 1
3127
// This is independent of OpenMP's thread count

0 commit comments

Comments
 (0)