Remove disable_skip (#5752)

Do you still need it?

Author: wujingyue

tests/cpp/multidevice.cpp

@@ -9,7 +9,6 @@
 
 #include <sys/types.h>
 #include <unistd.h>
-#include <mutex>
 
 #ifdef NVFUSER_DISTRIBUTED
 #include <torch/csrc/distributed/c10d/debug.h>
@@ -43,7 +42,6 @@ MultiDeviceTest::MultiDeviceTest() {
   tensor_options_ =
       at::TensorOptions().dtype(at::kFloat).device(communicator_->device());
   debug_print = getNvFuserEnv("MULTIDEVICE_DEBUG_PRINT") != nullptr;
-  disable_skip = getNvFuserEnv("MULTIDEVICE_DISABLE_SKIP") != nullptr;
 }
 
 MultiDeviceTest::~MultiDeviceTest() {
@@ -57,10 +55,9 @@ MultiDeviceTest::~MultiDeviceTest() {
 }
 
 void MultiDeviceTest::SetUp() {
-  // Set the same random seed for all processes.
   NVFuserTest::SetUp();
 
-  if (!disable_skip && !communicator_->is_available()) {
+  if (!communicator_->is_available()) {
     GTEST_SKIP() << "This test needs an available communicator.";
   }
 }

tests/cpp/multidevice.h

@@ -49,7 +49,6 @@ class MultiDeviceTest : public NVFuserTest {
   Communicator* communicator_;
   c10::TensorOptions tensor_options_;
   bool debug_print;
-  bool disable_skip;
 };
 
 // This macro is supposed to be used in a test case of a MultiDeviceTest or its

tests/cpp/test_multidevice_pipeline.cpp

@@ -287,7 +287,7 @@ TEST_P(PipelineTestTwoStages, Communication) {
        do_reduction,
        sharded_dim,
        use_fusion_executor_cache] = GetParam();
-  if (!disable_skip && !communicator_->isBackendAvailable(backend)) {
+  if (!communicator_->isBackendAvailable(backend)) {
     GTEST_SKIP() << "Backend not available";
   }
   communicator_->setDefaultBackend(backend);

tests/cpp/test_multidevice_sharding.cpp

@@ -19,7 +19,6 @@
 
 namespace nvfuser {
 
-using testing::Contains;
 using testing::Each;
 using testing::ElementsAre;
 using testing::Not;
@@ -1283,4 +1282,5 @@ TEST_F(MultiDeviceTest, MultipleIncompatibleReshapes) {
     EXPECT_FALSE(runtime->isSegmented());
   }
 }
+
 } // namespace nvfuser