chore: get cargo test to pass

Author: brandonskiser

crates/chat-cli/src/auth/builder_id.rs

@@ -69,7 +69,7 @@ impl std::fmt::Display for OAuthFlow {
     fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
         match *self {
             OAuthFlow::DeviceCode => write!(f, "DeviceCode"),
-            OAuthFlow::Pkce => write!(f, "PKCE"),
+            OAuthFlow::Pkce => write!(f, "Pkce"),
         }
     }
 }
@@ -593,7 +593,7 @@ mod tests {
     #[test]
     fn test_oauth_flow_ser_deser() {
         test_ser_deser!(OAuthFlow, OAuthFlow::DeviceCode, "DeviceCode");
-        test_ser_deser!(OAuthFlow, OAuthFlow::Pkce, "PKCE");
+        test_ser_deser!(OAuthFlow, OAuthFlow::Pkce, "Pkce");
     }
 
     #[test]

crates/chat-cli/src/cli/chat/conversation_state.rs

@@ -987,7 +987,7 @@ mod tests {
 
     #[tokio::test]
     async fn test_conversation_state_additional_context() {
-        tracing_subscriber::fmt::try_init().ok();
+        // tracing_subscriber::fmt::try_init().ok();
 
         let mut tool_manager = ToolManager::default();
         let ctx = Context::builder().with_test_home().await.unwrap().build_fake();

crates/chat-cli/src/cli/chat/mod.rs

@@ -3384,7 +3384,7 @@ mod tests {
 
     #[tokio::test]
     async fn test_flow() {
-        let _ = tracing_subscriber::fmt::try_init();
+        // let _ = tracing_subscriber::fmt::try_init();
         let ctx = Context::builder().with_test_home().await.unwrap().build_fake();
         let test_client = create_stream(serde_json::json!([
             [
@@ -3438,7 +3438,7 @@ mod tests {
 
     #[tokio::test]
     async fn test_flow_tool_permissions() {
-        let _ = tracing_subscriber::fmt::try_init();
+        // let _ = tracing_subscriber::fmt::try_init();
         let ctx = Context::builder().with_test_home().await.unwrap().build_fake();
         let test_client = create_stream(serde_json::json!([
             [
@@ -3584,7 +3584,7 @@ mod tests {
 
     #[tokio::test]
     async fn test_flow_multiple_tools() {
-        let _ = tracing_subscriber::fmt::try_init();
+        // let _ = tracing_subscriber::fmt::try_init();
         let ctx = Context::builder().with_test_home().await.unwrap().build_fake();
         let test_client = create_stream(serde_json::json!([
             [
@@ -3678,7 +3678,7 @@ mod tests {
 
     #[tokio::test]
     async fn test_flow_tools_trust_all() {
-        let _ = tracing_subscriber::fmt::try_init();
+        // let _ = tracing_subscriber::fmt::try_init();
         let ctx = Context::builder().with_test_home().await.unwrap().build_fake();
         let test_client = create_stream(serde_json::json!([
             [

crates/chat-cli/src/cli/chat/parser.rs

@@ -320,7 +320,7 @@ mod tests {
 
     #[tokio::test]
     async fn test_parse() {
-        let _ = tracing_subscriber::fmt::try_init();
+        // let _ = tracing_subscriber::fmt::try_init();
         let tool_use_id = "TEST_ID".to_string();
         let tool_name = "execute_bash".to_string();
         let tool_args = serde_json::json!({

crates/chat-cli/src/platform/env.rs

@@ -52,10 +52,7 @@ mod inner {
 
 impl Env {
     pub fn new() -> Self {
-        match cfg!(test) {
-            true => Self(inner::Inner::Fake(Arc::new(Mutex::new(inner::Fake::default())))),
-            false => Self::default(),
-        }
+        Self::default()
     }
 
     /// Create a fake process environment from a slice of tuples.
@@ -213,7 +210,7 @@ mod tests {
 
     #[test]
     fn test_default_current_dir() {
-        let env = Env::new();
+        let env = Env::from_slice(&[]);
         assert_eq!(env.current_dir().unwrap(), PathBuf::from("/"));
     }
 }

crates/chat-cli/src/platform/fs.rs

@@ -39,10 +39,7 @@ mod inner {
 
 impl Fs {
     pub fn new() -> Self {
-        match cfg!(test) {
-            true => Self(inner::Inner::Fake(Arc::new(Mutex::new(HashMap::new())))),
-            false => Self::default(),
-        }
+        Self::default()
     }
 
     pub fn new_chroot() -> Self {

crates/chat-cli/src/settings/sqlite.rs

@@ -326,7 +326,7 @@ mod tests {
         );
 
         // Assert all the files in migrations/ are in the list
-        let migration_folder = std::path::Path::new(env!("CARGO_MANIFEST_DIR")).join("src/sqlite/migrations");
+        let migration_folder = std::path::Path::new(env!("CARGO_MANIFEST_DIR")).join("src/settings/sqlite_migrations");
         let migration_count = std::fs::read_dir(migration_folder).unwrap().count();
         assert_eq!(MIGRATIONS.len(), migration_count);
     }