chore: Remove lib.rs and move behavior tests from integration to unit tests

.github/workflows/ci.yml

@@ -34,7 +34,8 @@ jobs:
       - uses: actions/checkout@v4
 
       - name: Test
-        run: cargo test --workspace --lib --bins
+        run: cargo test --workspace --bins
         env:
+          STORIFY_SKIP_BEHAVIOR: "1"
           RUST_LOG: DEBUG
           RUST_BACKTRACE: full

.github/workflows/test_behavior.yml

@@ -31,7 +31,7 @@ jobs:
           echo "MinIO not ready" >&2; exit 1
 
       - name: Run Behavior Tests
-        run: cargo test --test behavior --quiet
+        run: cargo test tests::behavior_suite -- --quiet
         env:
           RUST_LOG: info
 

Cargo.toml

@@ -50,8 +50,3 @@ predicates = "3.1.3"
 rand = "0.9.2"
 tracing-subscriber = { version = "0.3.20", features = ["env-filter", "tracing-log"] }
 tempfile = "3.14.0"
-
-[[test]]
-harness = false
-name = "behavior"
-path = "tests/behavior/main.rs"

src/cli/mod.rs

@@ -4,6 +4,4 @@ pub mod entry;
 pub mod prompts;
 pub mod storage;
 
-pub use context::CliContext;
-pub use entry::{Args, Command, ConfigCommand, GlobalOptions, run, run_with_prompt};
-pub use prompts::Prompt;
+pub use entry::{Args, run};

src/cli/prompts.rs

@@ -6,9 +6,10 @@ use tokio::task;
 use crate::error::{Error, Result};
 
 /// Interactive prompt mode for CLI operations
-#[derive(Debug, Clone, Copy)]
+#[derive(Debug, Clone, Copy, Default)]
 pub enum Prompt {
     /// Console-based interactive prompts using dialoguer
+    #[default]
     Console,
     /// Non-interactive mode that uses defaults or fails
     NonInteractive,
@@ -77,12 +78,6 @@ impl Prompt {
     }
 }
 
-impl Default for Prompt {
-    fn default() -> Self {
-        Self::Console
-    }
-}
-
 fn join_error(err: task::JoinError) -> Error {
     Error::Io {
         source: io::Error::other(err.to_string()),

src/config/loader.rs

@@ -332,28 +332,6 @@ pub fn resolve(request: ConfigRequest) -> Result<ResolvedConfig> {
     Ok(resolved)
 }
 
-/// Load storage configuration using environment variables only.
-pub fn load_storage_config() -> Result<StorageConfig> {
-    let resolved = resolve(ConfigRequest {
-        profile: None,
-        profile_store_path: None,
-        non_interactive: false,
-        require_storage: true,
-        master_password: None,
-    })?;
-
-    let storage = resolved.storage.ok_or_else(|| Error::NoConfiguration {
-        profiles: "none".to_string(),
-    })?;
-    Ok(storage)
-}
-
-pub fn load_env_storage_config(provider_hint: Option<String>) -> Result<StorageConfig> {
-    load_env_config(&env_value, provider_hint)
-        .and_then(build_config)
-        .map_err(with_config_hint)
-}
-
 fn load_env_config(
     get: &dyn Fn(&str) -> Option<String>,
     provider_hint: Option<String>,

src/config/profile_store.rs

@@ -121,17 +121,6 @@ impl ProfileStore {
         default_store_path()
     }
 
-    pub fn open_default() -> Result<Self> {
-        Self::open_with_options(ProfileStoreOpenOptions::default())
-    }
-
-    pub fn open(path: Option<PathBuf>) -> Result<Self> {
-        Self::open_with_options(ProfileStoreOpenOptions {
-            path,
-            ..ProfileStoreOpenOptions::default()
-        })
-    }
-
     pub fn open_with_password(
         path: Option<PathBuf>,
         master_password: Option<SecretString>,
@@ -350,15 +339,6 @@ impl ProfileStore {
         self.persist()
     }
 
-    /// Re-derive encryption key (for key rotation or master password change)
-    pub fn set_encryption(&mut self, master_password: Option<SecretString>) -> Result<()> {
-        let password = resolve_master_password(master_password, &self.path);
-        let salt = generate_salt();
-        let key = derive_master_key(&password, &salt)?;
-        self.encryption = EncryptionMetadata::new(key, salt.to_vec());
-        self.persist()
-    }
-
     fn persist(&mut self) -> Result<()> {
         let mut payload = self.file.clone();
         payload.normalize_default();
@@ -389,7 +369,7 @@ impl ProfileStore {
         write_atomic(&self.path, serialized.as_bytes())?;
 
         // Synchronize salt file to ensure it matches the in-memory state
-        // This is critical for operations like set_encryption() that generate a new salt
+        // This is critical for operations that regenerate the encryption metadata
         let salt_path = Self::salt_file_path(&self.path);
         Self::write_salt_file(&salt_path, salt)?;
 

src/main.rs

@@ -1,6 +1,15 @@
+mod cli;
+mod config;
+mod error;
+mod storage;
+mod utils;
+
+#[cfg(test)]
+mod tests;
+
 use clap::Parser;
 
-use storify::cli::{Args, run};
+use crate::cli::{Args, run};
 
 #[tokio::main]
 async fn main() {

src/storage.rs

@@ -47,10 +47,14 @@ impl StorageClient {
         Ok(Self { operator, provider })
     }
 
+    #[cfg(test)]
+    #[allow(dead_code)]
     pub fn provider(&self) -> StorageProvider {
         self.provider
     }
 
+    #[cfg(test)]
+    #[allow(dead_code)]
     pub fn operator(&self) -> &Operator {
         &self.operator
     }

src/storage/constants.rs

@@ -10,4 +10,3 @@ pub const PROGRESS_UPDATE_INTERVAL: u64 = 100;
 pub const DEFAULT_FS_ROOT: &str = "./storage";
 pub const DEFAULT_HDFS_ROOT: &str = "/";
 pub const DEFAULT_COS_ENDPOINT: &str = "https://cos.myqcloud.com";
-pub const CAT_CONFIRM_SIZE_THRESHOLD: u64 = 10 * 1024 * 1024;

src/tests/behavior/macros.rs

@@ -0,0 +1,21 @@
+macro_rules! register_behavior_tests {
+    ($($test:ident),+ $(,)?) => {
+        pub fn tests(client: &StorageClient, tests: &mut Vec<Trial>) {
+            tests.extend(async_trials!(client, $( $test ),+));
+        }
+
+        #[cfg(test)]
+        mod case_tests {
+            $(
+            #[test]
+            fn $test() {
+                if let Err(err) = crate::tests::behavior::run_behavior_case(
+                    concat!("behavior::", stringify!($test)),
+                ) {
+                    panic!("behavior case {} failed: {err}", stringify!($test));
+                }
+            }
+            )*
+        }
+    };
+}