enhance tests

Signed-off-by: OneSizeFitsQuorum <[email protected]>

Author: OneSizeFitsQuorum

iotdb-core/datanode/src/test/java/org/apache/iotdb/db/storageengine/dataregion/wal/allocation/FirstCreateStrategyTest.java

@@ -25,7 +25,6 @@
 import org.apache.iotdb.db.queryengine.plan.planner.plan.node.PlanNodeId;
 import org.apache.iotdb.db.queryengine.plan.planner.plan.node.write.InsertRowNode;
 import org.apache.iotdb.db.storageengine.dataregion.wal.node.IWALNode;
-import org.apache.iotdb.db.storageengine.dataregion.wal.node.WALFakeNode;
 import org.apache.iotdb.db.storageengine.dataregion.wal.node.WALNode;
 import org.apache.iotdb.db.storageengine.dataregion.wal.utils.WALFileUtils;
 import org.apache.iotdb.db.utils.EnvironmentUtils;
@@ -47,7 +46,6 @@
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertNotEquals;
 import static org.junit.Assert.assertNotNull;
-import static org.junit.Assert.assertNull;
 import static org.junit.Assert.assertTrue;
 
 public class FirstCreateStrategyTest {
@@ -224,60 +222,6 @@ public void testReInitializeAfterDiskSpaceCleaned() throws IllegalPathException,
     }
   }
 
-  @Test
-  public void testReInitializeFailsWhenDiskStillFull() throws IllegalPathException, IOException {
-    // Create unique temporary directory for testing
-    Path tempDir = Files.createTempDirectory("iotdb_wal_reinit_fail_test_");
-
-    // Use non-existent directories that will cause re-initialization to fail
-    String[] nonExistentWalDirs =
-        new String[] {
-          tempDir.resolve("non_existent_dir1").toString(),
-          tempDir.resolve("non_existent_dir2").toString(),
-          tempDir.resolve("non_existent_dir3").toString()
-        };
-
-    String[] originalWalDirs = commonConfig.getWalDirs();
-    commonConfig.setWalDirs(nonExistentWalDirs);
-
-    try {
-      // Create strategy - this will fail to initialize folderManager
-      FirstCreateStrategy strategy = new FirstCreateStrategy();
-
-      // Verify that folderManager is null due to initialization failure
-      try {
-        java.lang.reflect.Field folderManagerField =
-            AbstractNodeAllocationStrategy.class.getDeclaredField("folderManager");
-        folderManagerField.setAccessible(true);
-        Object folderManager = folderManagerField.get(strategy);
-        assertNull("folderManager should be null due to initialization failure", folderManager);
-      } catch (NoSuchFieldException | IllegalAccessException e) {
-        throw new RuntimeException("Failed to access folderManager field", e);
-      }
-
-      // Now apply for WAL node - re-initialization should also fail and return WALFakeNode
-      IWALNode walNode = strategy.applyForWALNode("test_reinit_fail_identifier");
-
-      // Verify that WALFakeNode is returned when re-initialization fails
-      assertNotNull("WAL node should be returned even when re-initialization fails", walNode);
-      assertTrue(
-          "Should return WALFakeNode when re-initialization fails", walNode instanceof WALFakeNode);
-
-      // WALFakeNode should be the failure instance
-      WALFakeNode fakeNode = (WALFakeNode) walNode;
-      assertTrue(
-          "WALFakeNode should contain failure information",
-          fakeNode.toString().contains("DiskSpaceInsufficientException")
-              || fakeNode.toString().contains("Failed to create WAL node"));
-
-    } finally {
-      // Clean up temp directory
-      EnvironmentUtils.cleanDir(tempDir.toString());
-      // Restore original WAL directories
-      commonConfig.setWalDirs(originalWalDirs);
-    }
-  }
-
   private InsertRowNode getInsertRowNode() throws IllegalPathException {
     long time = 110L;
     TSDataType[] dataTypes =