add new tests for editor,compact feature and description for editor , comoact feature test cases

Author: sayemaanjum0412

e2etests/tests/integration/test_editor_help_command.rs

@@ -13,6 +13,7 @@ const TEST_NAMES: &[&str] = &[
     "test_editor_h_command",
     "test_editor_command_interaction",
     "test_editor_command_error",
+    "test_editor_with_file_path",
 ];
 #[allow(dead_code)]
 const TOTAL_TESTS: usize = TEST_NAMES.len();
@@ -182,7 +183,7 @@ fn test_editor_h_command() -> Result<(), Box<dyn std::error::Error>> {
 #[test]
 #[cfg(all(feature = "editor", feature = "sanity"))]
 fn test_editor_command_interaction() -> Result<(), Box<dyn std::error::Error>> {
-    println!("🔍 Testing /editor command interaction...");
+    println!("🔍 Testing /editor command interaction... | Description: Test that the /editor command successfully launches the integrated editor interface");
 
     let session = get_chat_session();
     let mut chat = session.lock().unwrap_or_else(|poisoned| poisoned.into_inner());
@@ -233,7 +234,7 @@ fn test_editor_command_interaction() -> Result<(), Box<dyn std::error::Error>> {
 #[test]
 #[cfg(all(feature = "editor", feature = "sanity"))]
 fn test_editor_command_error() -> Result<(), Box<dyn std::error::Error>> {
-    println!("🔍 Testing /editor command error handling");
+    println!("🔍 Testing /editor command error handling ... | Description: Tests the /editor <non_exixt_filepath> command error handling when attempting to open a nonexistent file");
 
     let session = get_chat_session();
     let mut chat = session.lock().unwrap_or_else(|poisoned| poisoned.into_inner());
@@ -278,5 +279,62 @@ fn test_editor_command_error() -> Result<(), Box<dyn std::error::Error>> {
     // Cleanup only if this is the last test
     cleanup_if_last_test(&TEST_COUNT, TOTAL_TESTS)?;
 
+    Ok(())
+}
+
+#[test]
+#[cfg(all(feature = "editor", feature = "sanity"))]
+fn test_editor_with_file_path() -> Result<(), Box<dyn std::error::Error>> {
+    println!("🔍 Testing /editor <filepath> command... | Description: Tests the /editor <filepath> command to load an existing file into the editor and verify content loading");
+    
+    let home_dir = std::env::var("HOME").unwrap_or_else(|_| "/tmp".to_string());
+    let test_file_path = format!("{}/test_editor_file.txt", home_dir);
+    
+    // Create a test file
+    std::fs::write(&test_file_path, "Hello from test file\nThis is a test file for editor command.")?;
+    println!("✅ Created test file at {}", test_file_path);
+    
+    let session = get_chat_session();
+    let mut chat = session.lock().unwrap_or_else(|poisoned| poisoned.into_inner());
+    
+    // Execute /editor command with file path
+    let response = chat.execute_command(&format!("/editor {}", test_file_path))?;
+    
+    println!("📝 Editor with file response: {} bytes", response.len());
+    println!("📝 FULL OUTPUT:");
+    println!("{}", response);
+    println!("📝 END OUTPUT");
+    
+     // Press 'i' to enter insert mode
+    let insert_response = chat.execute_command("i")?;
+    println!("📝 Insert mode response: {} bytes", insert_response.len());
+    
+    
+    // Press Esc to exit insert mode
+    let esc_response = chat.execute_command("\x1b")?; // ESC key
+    println!("📝 Esc response: {} bytes", esc_response.len());
+    
+    // Execute :wq to save and quit
+    let wq_response = chat.execute_command(":wq")?;
+
+    println!("📝 Final wq response: {} bytes", wq_response.len());
+    println!("📝 WQ RESPONSE:");
+    println!("{}", wq_response);
+    println!("📝 END WQ RESPONSE");
+    
+    // Verify the file content is loaded in editor
+    assert!(wq_response.contains("Hello from test file"), "File content not loaded in editor");
+    println!("✅ File content loaded successfully in editor");
+    
+    // Clean up test file
+    std::fs::remove_file(test_file_path).ok();
+    println!("✅ Cleaned up test file");
+    
+    // Release the lock before cleanup
+    drop(chat);
+    
+    // Cleanup only if this is the last test
+    cleanup_if_last_test(&TEST_COUNT, TOTAL_TESTS)?;
+
     Ok(())
 }

e2etests/tests/session_mgmt/test_compact_command.rs

@@ -184,7 +184,7 @@ fn test_compact_h_command() -> Result<(), Box<dyn std::error::Error>> {
 #[test]
 #[cfg(all(feature = "compact", feature = "sanity"))]
 fn test_compact_truncate_true_command() -> Result<(), Box<dyn std::error::Error>> {
-    println!("🔍 Testing /compact --truncate-large-messages true command...");
+    println!("🔍 Testing /compact --truncate-large-messages true command... | Description: Test that the /compact  —truncate-large-messages truncates large messages");
 
     let session = get_chat_session();
     let mut chat = session.lock().unwrap_or_else(|poisoned| poisoned.into_inner());
@@ -228,7 +228,7 @@ fn test_compact_truncate_true_command() -> Result<(), Box<dyn std::error::Error>
 #[test]
 #[cfg(all(feature = "compact", feature = "sanity"))]
 fn test_compact_truncate_false_command() -> Result<(), Box<dyn std::error::Error>> {
-    println!("🔍 Testing /compact --truncate-large-messages false command...");
+    println!("🔍 Testing /compact --truncate-large-messages false command... | Description: Tests the /compact --truncate-large-messages false command to verify no message truncation occurs");
 
     let session = get_chat_session();
      let mut chat = session.lock().unwrap_or_else(|poisoned| poisoned.into_inner());
@@ -271,7 +271,7 @@ fn test_compact_truncate_false_command() -> Result<(), Box<dyn std::error::Error
 #[test]
 #[cfg(all(feature = "compact", feature = "sanity"))]
 fn test_show_summary() -> Result<(), Box<dyn std::error::Error>> {
-    println!("🔍 Testing /compact --show-summary command...");
+    println!("🔍 Testing /compact --show-summary command... | Description: Tests the /compact --show-summary command to display conversation summary after compaction");
 
     let session = get_chat_session();
     let mut chat = session.lock().unwrap_or_else(|poisoned| poisoned.into_inner());
@@ -322,7 +322,7 @@ fn test_show_summary() -> Result<(), Box<dyn std::error::Error>> {
 #[test]
 #[cfg(all(feature = "compact", feature = "sanity"))]
 fn test_max_message_truncate_true() -> Result<(), Box<dyn std::error::Error>> {
-    println!("🔍 Testing /compact --truncate-large-messages true --max-message-length command...");
+    println!("🔍 Testing /compact --truncate-large-messages true --max-message-length command... | Description: Test /compact --truncate-large-messages true  --max-message-length <MAX_MESSAGE_LENGTH> command compacts the conversation by summarizing it to free up context space, truncating large messages to a maximum of provided <MAX_MESSAGE_LENGTH>. ");
 
     let session = get_chat_session();
     let mut chat = session.lock().unwrap_or_else(|poisoned| poisoned.into_inner());
@@ -376,7 +376,7 @@ fn test_max_message_truncate_true() -> Result<(), Box<dyn std::error::Error>> {
 #[test]
 #[cfg(all(feature = "compact", feature = "sanity"))]
 fn test_max_message_truncate_false() -> Result<(), Box<dyn std::error::Error>> {
-    println!("🔍 Testing /compact --truncate-large-messages false --max-message-length command...");
+    println!("🔍 Testing /compact --truncate-large-messages false --max-message-length command... | Description: Test /compact --truncate-large-messages false --max-message-length <MAX_MESSAGE_LENGTH> command compacts the conversation by summarizing it to free up context space, but keeps large messages intact (no truncation) despite the max-message-length setting.");
 
     let session = get_chat_session();
     let mut chat = session.lock().unwrap_or_else(|poisoned| poisoned.into_inner());
@@ -427,7 +427,7 @@ fn test_max_message_truncate_false() -> Result<(), Box<dyn std::error::Error>> {
 #[test]
 #[cfg(all(feature = "compact", feature = "sanity"))]
 fn test_max_message_length_invalid() -> Result<(), Box<dyn std::error::Error>> {
-    println!("🔍 Testing /compact --max-message-length command...");
+    println!("🔍 Testing /compact --max-message-length command... | Description: Tests the /compact --max-message-length <MAX_MESSAGE_LENGTH> command with invalid subcommand to verify proper error handling and help display");
 
     let session = get_chat_session();
     let mut chat = session.lock().unwrap_or_else(|poisoned| poisoned.into_inner());
@@ -467,4 +467,53 @@ fn test_max_message_length_invalid() -> Result<(), Box<dyn std::error::Error>> {
     cleanup_if_last_test(&TEST_COUNT, TOTAL_TESTS)?;
 
     Ok(())
-}
+}
+
+#[test]
+#[cfg(all(feature = "compact", feature = "sanity"))]
+fn test_compact_messages_to_exclude_command() -> Result<(), Box<dyn std::error::Error>> {
+    println!("\n🔍 Testing /compact command... | Description: Test /compact --messages-to-exclude <MESSAGES_TO_EXCLUDE> command compacts the conversation by summarizing it to free up context space, excluding provided number of user-assistant message pair from the summarization process.");
+    
+    let session = get_chat_session();
+    let mut chat = session.lock().unwrap_or_else(|poisoned| poisoned.into_inner());
+     
+    let response = chat.execute_command("What is AWS?")?;
+    
+    println!("📝 AI response: {} bytes", response.len());
+    println!("📝 FULL OUTPUT:");
+    println!("{}", response);
+    println!("📝 END OUTPUT");
+
+    let response = chat.execute_command("What is fibonacci?")?;
+    
+    println!("📝 AI response: {} bytes", response.len());
+    println!("📝 FULL OUTPUT:");
+    println!("{}", response);
+    println!("📝 END OUTPUT");
+
+    let response = chat.execute_command("/compact --messages-to-exclude 1")?;
+    
+    println!("📝 Compact response: {} bytes", response.len());
+    println!("📝 FULL OUTPUT:");
+    println!("{}", response);
+    println!("📝 END OUTPUT");
+    
+    // Verify compact response - either success or too short
+    if response.contains("history") && response.contains("compacted") && response.contains("successfully") {
+        println!("✅ Found compact success message");
+    } else if response.contains("Conversation") && response.contains("short") {
+        println!("✅ Found conversation too short message");
+    } else {
+        panic!("Missing expected compact response");
+    }
+    
+    println!("✅ All compact content verified!");
+    
+    // Release the lock before cleanup
+    drop(chat);
+    
+    // Cleanup only if this is the last test
+    cleanup_if_last_test(&TEST_COUNT, TOTAL_TESTS)?;
+
+    Ok(())
+}