fix(config): return error if --config file does not exist; fix test cases

Author: zipper-meng

config/src/meta/mod.rs

@@ -39,7 +39,14 @@ impl Opt {
         let mut figment = Figment::new();
         // Merge toml config file.
         if let Some(path) = path {
-            figment = figment.merge(Toml::file(path.as_ref()));
+            let path = path.as_ref();
+            if !path.exists() {
+                return Err(figment::error::Kind::Message(format!(
+                    "Config file not found: {path:?}"
+                ))
+                .into());
+            }
+            figment = figment.merge(Toml::file(path));
         }
         // Merge environment variables with prefix `CNOSDB_META_`.// Map field-keys into env-keys.
         let env_key_map = Self::env_keys();

config/src/tskv/mod.rs

@@ -89,7 +89,14 @@ impl Config {
         let mut figment = Figment::new();
         // Merge toml config file.
         if let Some(path) = path {
-            figment = figment.merge(Toml::file(path.as_ref()));
+            let path = path.as_ref();
+            if !path.exists() {
+                return Err(figment::error::Kind::Message(format!(
+                    "Config file not found: {path:?}"
+                ))
+                .into());
+            }
+            figment = figment.merge(Toml::file(path));
         }
         // Merge environment variables with prefix `CNOSDB_`.
         let env_key_map = Self::env_keys();

e2e_test/src/independent/replica_test.rs

@@ -10,8 +10,6 @@ use crate::utils::global::E2eContext;
 use crate::utils::Client;
 use crate::{check_response, cluster_def};
 
-// const SERVER_URL: &str = "http://127.0.0.1:8902/api/v1/sql?db=replica_test_db";
-
 fn replica_test(meta: Arc<TenantMeta>, server_url: &str) {
     let db_info = meta.get_db_info("replica_test_db").unwrap().unwrap();
     assert!(!db_info.buckets.is_empty());

main/src/main.rs

@@ -237,12 +237,14 @@ fn main() -> Result<(), std::io::Error> {
     });
     Ok(())
 }
+
 fn handle_error<T, E: std::fmt::Debug>(result: Result<T, E>, context: &str) -> T {
     result.unwrap_or_else(|e| {
         error!("{}: {:?}", context, e);
         process::exit(1);
     })
 }
+
 fn parse_config(run_args: &RunArgs) -> config::tskv::Config {
     println!("-----------------------------------------------------------");
     println!("Using Config File: {}\n", run_args.config);

query_server/sqllogicaltests/cases/function/gis/check_write.slt

@@ -112,12 +112,12 @@ select st_distance('LINESTRING (30 10, 10 30, 40 40)', 'GEOMETRYCOLLECTION (POIN
 
 
 
-query 
+query
 select st_distance('POLYGON ((30 10, 40 40, 20 40, 10 20, 30 10))', 'POLYGON ((30 10, 40 40, 20 40, 10 20, 30 10))');
 ----
 0.0
 
-query 
+query
 select st_distance('POLYGON ((30 10, 40 40, 20 40, 10 20, 30 10))', 'POLYGON ((35 10, 45 45, 15 40, 10 20, 35 10),(20 30, 35 35, 30 20, 20 30))');
 ----
 0.0

query_server/sqllogicaltests/cases/function/gis/st_distance.slt

@@ -27,7 +27,7 @@ PROJ_DIR=$(
 
 META="${PROJ_DIR}/meta/scripts/cluster.sh"
 DATA="${PROJ_DIR}/main/scripts/cluster.sh"
-PROFILE='release'
+PROFILE='test-ci'
 
 ## Run cluster.
 CLUSTER_STARTED=0