fix(config): fix ARM64 test failure by adding config validation

The test failed on ARM64 because Storage.getDbEngineFromConfig()
silently overrode the user's config to ROCKSDB, which is unreasonable
as it hides incompatible configuration from the user.

Replace the silent override with an explicit validateConfig() check
in Args that fails fast with IllegalArgumentException when LevelDB
is configured on ARM64. This makes the incompatibility visible
instead of silently swallowed.

Author: vividcoder

common/src/main/java/org/tron/core/config/args/Storage.java

@@ -29,7 +29,6 @@
 import org.apache.commons.lang3.StringUtils;
 import org.iq80.leveldb.CompressionType;
 import org.iq80.leveldb.Options;
-import org.tron.common.arch.Arch;
 import org.tron.common.cache.CacheStrategies;
 import org.tron.common.cache.CacheType;
 import org.tron.common.utils.DbOptionalsUtils;
@@ -90,7 +89,6 @@ public class Storage {
    * Default values of directory
    */
   private static final String DEFAULT_DB_ENGINE = "LEVELDB";
-  private static final String ROCKS_DB_ENGINE = "ROCKSDB";
   private static final boolean DEFAULT_DB_SYNC = false;
   private static final boolean DEFAULT_EVENT_SUBSCRIBE_CONTRACT_PARSE = true;
   private static final String DEFAULT_DB_DIRECTORY = "database";
@@ -175,11 +173,6 @@ public class Storage {
   private final Map<String, Sha256Hash> dbRoots = Maps.newConcurrentMap();
 
   public static String getDbEngineFromConfig(final Config config) {
-    if (Arch.isArm64()) {
-      // if is arm64 but config is leveldb, should throw exception?
-      logger.warn("Arm64 architecture detected, using RocksDB as db engine, ignore config.");
-      return ROCKS_DB_ENGINE;
-    }
     return config.hasPath(DB_ENGINE_CONFIG_KEY)
         ? config.getString(DB_ENGINE_CONFIG_KEY) : DEFAULT_DB_ENGINE;
   }

framework/src/main/java/org/tron/core/config/args/Args.java

@@ -130,6 +130,20 @@ public static void setParam(final String[] args, final String confFileName) {
 
     // 4. Init witness (depends on CLI witness flag)
     initLocalWitnesses(config, cmd);
+
+    // 5. Validate final configuration
+    validateConfig();
+  }
+
+  /**
+   * Validate the final configuration after all layers (defaults, config, CLI) are applied.
+   * Fails fast on incompatible configurations.
+   */
+  private static void validateConfig() {
+    if (Arch.isArm64() && "LEVELDB".equalsIgnoreCase(PARAMETER.storage.getDbEngine())) {
+      throw new IllegalArgumentException(
+          "LevelDB is not supported on ARM64 architecture, please use ROCKSDB");
+    }
   }
 
   /**

framework/src/test/java/org/tron/core/config/args/ArgsTest.java

@@ -38,6 +38,7 @@
 import org.tron.common.utils.DecodeUtil;
 import org.tron.common.utils.LocalWitnesses;
 import org.tron.common.utils.PublicMethod;
+import org.tron.common.arch.Arch;
 import org.tron.core.config.Configuration;
 
 @Slf4j
@@ -323,15 +324,21 @@ public void testCliOverridesStorageConfig() {
    *
    * <p>config-test.conf defines: db.directory = "database", db.engine = "LEVELDB".
    * Without any CLI storage arguments, the Storage object should use these config values.
+   * On ARM64, LEVELDB is not supported and validateConfig() should throw.
    */
   @Test
   public void testConfigStorageDefaults() {
-    Args.setParam(new String[] {}, TestConstants.TEST_CONF);
+    if (Arch.isArm64()) {
+      Assert.assertThrows(IllegalArgumentException.class,
+          () -> Args.setParam(new String[] {}, TestConstants.TEST_CONF));
+    } else {
+      Args.setParam(new String[] {}, TestConstants.TEST_CONF);
 
-    CommonParameter parameter = Args.getInstance();
+      CommonParameter parameter = Args.getInstance();
 
-    Assert.assertEquals("database", parameter.getStorage().getDbDirectory());
-    Assert.assertEquals("LEVELDB", parameter.getStorage().getDbEngine());
+      Assert.assertEquals("database", parameter.getStorage().getDbDirectory());
+      Assert.assertEquals("LEVELDB", parameter.getStorage().getDbEngine());
+    }
 
     Args.clearParam();
   }