Clean console output

Author: abhishekanne

e2etests/run_tests.py

@@ -61,7 +61,7 @@ def parse_test_results(stdout):
         clean_line = line.strip()
 
         # Look for test declaration lines
-        if clean_line.startswith('test ') and ' ... ' in clean_line:
+        if clean_line.startswith('test ') and ' ...' in clean_line:
             # Extract test name (everything between 'test ' and ' ... ')
             test_name = clean_line.split(' ... ')[0].replace('test ', '').strip()
 

e2etests/tests/agent/test_agent_commands.rs

@@ -25,7 +25,7 @@ const TOTAL_TESTS: usize = TEST_NAMES.len();
 #[test]
 #[cfg(all(feature = "agent", feature = "sanity"))]
 fn agent_without_subcommand() -> Result<(), Box<dyn std::error::Error>> {
-    println!("🔍 Testing /agent command... | Description: Tests the /agent command without subcommands to display help information. Verifies agent management description, usage, available subcommands, and options");
+    println!("\n🔍 Testing /agent command... | Description: Tests the /agent command without subcommands to display help information. Verifies agent management description, usage, available subcommands, and options");
 
     let session = get_chat_session();
     let mut chat = session.lock().unwrap();
@@ -78,7 +78,7 @@ fn agent_without_subcommand() -> Result<(), Box<dyn std::error::Error>> {
 #[test]
 #[cfg(all(feature = "agent", feature = "sanity"))]
 fn test_agent_create_command() -> Result<(), Box<dyn std::error::Error>> {
-    println!("🔍 Testing /agent create --name <agent_name> command... | Description: Tests the /agent create command to create a new agent with specified name. Verifies agent creation process, file system operations, and cleanup");
+    println!("\n🔍 Testing /agent create --name <agent_name> command... | Description: Tests the /agent create command to create a new agent with specified name. Verifies agent creation process, file system operations, and cleanup");
 
     let timestamp = std::time::SystemTime::now()
         .duration_since(std::time::UNIX_EPOCH)
@@ -147,7 +147,7 @@ fn test_agent_create_command() -> Result<(), Box<dyn std::error::Error>> {
 #[test]
 #[cfg(all(feature = "agent", feature = "sanity"))]
 fn test_agent_create_missing_args() -> Result<(), Box<dyn std::error::Error>> {
-    println!("🔍 Testing /agent create without required arguments... | Description: Tests the /agent create command without required arguments to verify error handling. Verifies proper error messages, usage information, and help suggestions");
+    println!("\n🔍 Testing /agent create without required arguments... | Description: Tests the /agent create command without required arguments to verify error handling. Verifies proper error messages, usage information, and help suggestions");
 
     let session = get_chat_session();
     let mut chat = session.lock().unwrap();
@@ -198,7 +198,7 @@ fn test_agent_create_missing_args() -> Result<(), Box<dyn std::error::Error>> {
 #[test]
 #[cfg(all(feature = "agent", feature = "sanity"))]
 fn test_agent_help_command() -> Result<(), Box<dyn std::error::Error>> {
-    println!("🔍 Testing /agent help... | Description: Tests the /agent help command to display comprehensive agent help information. Verifies agent descriptions, usage notes, launch instructions, and configuration paths");
+    println!("\n🔍 Testing /agent help... | Description: Tests the /agent help command to display comprehensive agent help information. Verifies agent descriptions, usage notes, launch instructions, and configuration paths");
 
     let session = get_chat_session();
     let mut chat = session.lock().unwrap();
@@ -255,7 +255,7 @@ fn test_agent_help_command() -> Result<(), Box<dyn std::error::Error>> {
 #[test]
 #[cfg(all(feature = "agent", feature = "sanity"))]
 fn test_agent_invalid_command() -> Result<(), Box<dyn std::error::Error>> {
-    println!("🔍 Testing /agent invalidcommand... | Description: Tests the /agent command with invalid subcommand to verify error handling. Verifies that invalid commands display help information with available commands and options");
+    println!("\n🔍 Testing /agent invalidcommand... | Description: Tests the /agent command with invalid subcommand to verify error handling. Verifies that invalid commands display help information with available commands and options");
 
     let session = get_chat_session();
     let mut chat = session.lock().unwrap();
@@ -294,7 +294,7 @@ fn test_agent_invalid_command() -> Result<(), Box<dyn std::error::Error>> {
 #[test]
 #[cfg(all(feature = "agent", feature = "sanity"))]
 fn test_agent_list_command() -> Result<(), Box<dyn std::error::Error>> {
-    println!("🔍 Testing /agent list command... | Description: Tests the /agent list command to display all available agents. Verifies agent listing format and presence of default agent");
+    println!("\n🔍 Testing /agent list command... | Description: Tests the /agent list command to display all available agents. Verifies agent listing format and presence of default agent");
 
     let session = get_chat_session();
     let mut chat = session.lock().unwrap();
@@ -328,7 +328,7 @@ fn test_agent_list_command() -> Result<(), Box<dyn std::error::Error>> {
 // #[test]
 // #[cfg(feature = "agent")]
 // fn test_agent_schema_command() -> Result<(), Box<dyn std::error::Error>> {
-//     println!("🔍 Testing /agent schema... | Description: Tests the /agent schema command to display agent configuration schema. Verifies JSON schema structure with required keys and properties");
+//     println!("\n🔍 Testing /agent schema... | Description: Tests the /agent schema command to display agent configuration schema. Verifies JSON schema structure with required keys and properties");
 
 //     let session = get_chat_session();
 //     let mut chat = session.lock().unwrap();
@@ -369,7 +369,7 @@ fn test_agent_list_command() -> Result<(), Box<dyn std::error::Error>> {
 #[test]
 #[cfg(all(feature = "agent", feature = "sanity"))]
 fn test_agent_set_default_command() -> Result<(), Box<dyn std::error::Error>> {
-    println!("🔍 Testing /agent set-default with valid arguments... | Description: Tests the /agent set-default command with valid arguments to set default agent. Verifies success messages and confirmation of default agent configuration");
+    println!("\n🔍 Testing /agent set-default with valid arguments... | Description: Tests the /agent set-default command with valid arguments to set default agent. Verifies success messages and confirmation of default agent configuration");
 
     let session = get_chat_session();
     let mut chat = session.lock().unwrap();
@@ -410,7 +410,7 @@ fn test_agent_set_default_command() -> Result<(), Box<dyn std::error::Error>> {
 #[test]
 #[cfg(all(feature = "agent", feature = "sanity"))]
 fn test_agent_set_default_missing_args() -> Result<(), Box<dyn std::error::Error>> {
-    println!("🔍 Testing /agent set-default without required arguments... | Description: Tests the /agent set-default command without required arguments to verify error handling. Verifies error messages, usage information, and available options display");
+    println!("\n🔍 Testing /agent set-default without required arguments... | Description: Tests the /agent set-default command without required arguments to verify error handling. Verifies error messages, usage information, and available options display");
 
     let session = get_chat_session();
     let mut chat = session.lock().unwrap();

e2etests/tests/ai_prompts/test_ai_prompt.rs

@@ -4,7 +4,7 @@ use q_cli_e2e_tests::q_chat_helper::QChatSession;
 #[test]
 #[cfg(all(feature = "ai_prompts", feature = "sanity"))]
 fn test_what_is_aws_prompt() -> Result<(), Box<dyn std::error::Error>> {
-    println!("🔍 [AI PROMPTS] Testing 'What is AWS?' AI prompt... | Description: Tests AI prompt functionality by sending 'What is AWS?' and verifying the response contains relevant AWS information and technical terms");
+    println!("\n🔍 [AI PROMPTS] Testing 'What is AWS?' AI prompt... | Description: Tests AI prompt functionality by sending 'What is AWS?' and verifying the response contains relevant AWS information and technical terms");
 
     let mut chat = QChatSession::new()?;
     println!("✅ Q Chat session started");
@@ -57,7 +57,7 @@ fn test_what_is_aws_prompt() -> Result<(), Box<dyn std::error::Error>> {
 #[test]
 #[cfg(all(feature = "ai_prompts", feature = "sanity"))]
 fn test_simple_greeting() -> Result<(), Box<dyn std::error::Error>> {
-    println!("🔍 Testing simple 'Hello' prompt... | Description: Tests basic AI interaction by sending a simple greeting and verifying the AI responds appropriately with greeting-related content");
+    println!("\n🔍 Testing simple 'Hello' prompt... | Description: Tests basic AI interaction by sending a simple greeting and verifying the AI responds appropriately with greeting-related content");
 
     let mut chat = QChatSession::new()?;
     println!("✅ Q Chat session started");

e2etests/tests/ai_prompts/test_prompts_commands.rs

@@ -19,7 +19,7 @@ const TOTAL_TESTS: usize = TEST_NAMES.len();
 #[test]
 #[cfg(all(feature = "ai_prompts", feature = "sanity"))]
 fn test_prompts_command() -> Result<(), Box<dyn std::error::Error>> {
-    println!("🔍 Testing /prompts command... | Description: Tests the /prompts command to display available prompts with usage instructions and argument requirements");
+    println!("\n🔍 Testing /prompts command... | Description: Tests the /prompts command to display available prompts with usage instructions and argument requirements");
 
     let session = get_chat_session();
     let mut chat = session.lock().unwrap();
@@ -58,7 +58,7 @@ fn test_prompts_command() -> Result<(), Box<dyn std::error::Error>> {
 #[test]
 #[cfg(all(feature = "ai_prompts", feature = "sanity"))]
 fn test_prompts_help_command() -> Result<(), Box<dyn std::error::Error>> {
-    println!("🔍 Testing /prompts --help command... | Description: Tests the /prompts --help command to display comprehensive help information about prompts functionality and MCP server integration");
+    println!("\n🔍 Testing /prompts --help command... | Description: Tests the /prompts --help command to display comprehensive help information about prompts functionality and MCP server integration");
 
     let session = get_chat_session();
     let mut chat = session.lock().unwrap();
@@ -119,7 +119,7 @@ fn test_prompts_help_command() -> Result<(), Box<dyn std::error::Error>> {
 #[test]
 #[cfg(all(feature = "ai_prompts", feature = "sanity"))]
 fn test_prompts_list_command() -> Result<(), Box<dyn std::error::Error>> {
-    println!("🔍 Testing /prompts list command... | Description: Tests the /prompts list command to display all available prompts with their arguments and usage information");
+    println!("\n🔍 Testing /prompts list command... | Description: Tests the /prompts list command to display all available prompts with their arguments and usage information");
 
     let session = get_chat_session();
     let mut chat = session.lock().unwrap();

e2etests/tests/context/test_context_command.rs

@@ -25,7 +25,7 @@ const TOTAL_TESTS: usize = TEST_NAMES.len();
 #[test]
 #[cfg(all(feature = "context", feature = "sanity"))]
 fn test_context_show_command() -> Result<(), Box<dyn std::error::Error>> {
-    println!("🔍 Testing /context show command... | Description: Tests the /context show command to display current context information including agent configuration and context files");
+    println!("\n🔍 Testing /context show command... | Description: Tests the /context show command to display current context information including agent configuration and context files");
 
     let session = get_chat_session();
     let mut chat = session.lock().unwrap_or_else(|poisoned| poisoned.into_inner());
@@ -59,7 +59,7 @@ fn test_context_show_command() -> Result<(), Box<dyn std::error::Error>> {
 #[test]
 #[cfg(all(feature = "context", feature = "sanity"))]
 fn test_context_help_command() -> Result<(), Box<dyn std::error::Error>> {
-    println!("🔍 Testing /context help command... | Description: Tests the /context help command to display comprehensive help information for context management including usage, commands, and options");
+    println!("\n🔍 Testing /context help command... | Description: Tests the /context help command to display comprehensive help information for context management including usage, commands, and options");
 
     let session = get_chat_session();
     let mut chat = session.lock().unwrap_or_else(|poisoned| poisoned.into_inner());
@@ -101,7 +101,7 @@ fn test_context_help_command() -> Result<(), Box<dyn std::error::Error>> {
 #[test]
 #[cfg(all(feature = "context", feature = "sanity"))]
 fn test_context_without_subcommand() -> Result<(), Box<dyn std::error::Error>> {
-    println!("🔍 Testing /context without sub command... | Description: Tests the /context command without subcommands to verify it displays help information with usage and available commands");
+    println!("\n🔍 Testing /context without sub command... | Description: Tests the /context command without subcommands to verify it displays help information with usage and available commands");
 
     let session = get_chat_session();
     let mut chat = session.lock().unwrap_or_else(|poisoned| poisoned.into_inner());
@@ -139,7 +139,7 @@ fn test_context_without_subcommand() -> Result<(), Box<dyn std::error::Error>> {
 #[test]
 #[cfg(all(feature = "context", feature = "sanity"))]
 fn test_context_invalid_command() -> Result<(), Box<dyn std::error::Error>> {
-    println!("🔍 Testing /context invalid command... | Description: Tests the /context command with invalid subcommand to verify proper error handling and help display");
+    println!("\n🔍 Testing /context invalid command... | Description: Tests the /context 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());
@@ -169,7 +169,7 @@ fn test_context_invalid_command() -> Result<(), Box<dyn std::error::Error>> {
 #[test]
 #[cfg(all(feature = "context", feature = "sanity"))]
 fn test_add_non_existing_file_context() -> Result<(), Box<dyn std::error::Error>> {
-    println!("🔍 Testing /context add non-existing file command... | Description: Tests the /context add command with non-existing file to verify proper error handling and force option suggestion");
+    println!("\n🔍 Testing /context add non-existing file command... | Description: Tests the /context add command with non-existing file to verify proper error handling and force option suggestion");
 
     let non_existing_file_path = "/tmp/non_existing_file.py";
 
@@ -200,7 +200,7 @@ fn test_add_non_existing_file_context() -> Result<(), Box<dyn std::error::Error>
 #[test]
 #[cfg(all(feature = "context", feature = "sanity"))]
 fn test_context_remove_command_of_non_existent_file() -> Result<(), Box<dyn std::error::Error>> {
-    println!("🔍 Testing /context remove non existing file command... | Description: Tests the /context remove command with non-existing file to verify proper error handling");
+    println!("\n🔍 Testing /context remove non existing file command... | Description: Tests the /context remove command with non-existing file to verify proper error handling");
 
     let session = get_chat_session();
     let mut chat = session.lock().unwrap_or_else(|poisoned| poisoned.into_inner());
@@ -228,7 +228,7 @@ fn test_context_remove_command_of_non_existent_file() -> Result<(), Box<dyn std:
 #[test]
 #[cfg(all(feature = "context", feature = "sanity"))]
 fn test_add_remove_file_context() -> Result<(), Box<dyn std::error::Error>> {
-    println!("🔍 Testing /context add <filename> command and /context remove <filename> command... | Description: Tests the complete workflow of adding a file to context, verifying it appears in context show, then removing it and verifying removal");
+    println!("\n🔍 Testing /context add <filename> command and /context remove <filename> command... | Description: Tests the complete workflow of adding a file to context, verifying it appears in context show, then removing it and verifying removal");
 
     let test_file_path = "/tmp/test_context_file_.py";
     // Create a test file
@@ -302,7 +302,7 @@ fn test_add_remove_file_context() -> Result<(), Box<dyn std::error::Error>> {
 #[test]
 #[cfg(all(feature = "context", feature = "sanity"))]
 fn test_add_glob_pattern_file_context()-> Result<(), Box<dyn std::error::Error>> {
-    println!("🔍 Testing /context add *.py glob pattern command... | Description: Tests the /context add command with glob patterns to add multiple files matching a pattern and verify pattern-based context management");
+    println!("\n🔍 Testing /context add *.py glob pattern command... | Description: Tests the /context add command with glob patterns to add multiple files matching a pattern and verify pattern-based context management");
 
     let test_file1_path = "/tmp/test_context_file1.py";
     let test_file2_path = "/tmp/test_context_file2.py";
@@ -384,7 +384,7 @@ fn test_add_glob_pattern_file_context()-> Result<(), Box<dyn std::error::Error>>
 #[test]
 #[cfg(all(feature = "context", feature = "sanity"))]
 fn test_add_remove_multiple_file_context()-> Result<(), Box<dyn std::error::Error>> {
-    println!("🔍 Testing /context add <filename1> <filename2> <filename3> command and /context remove <filename1> <filename2> <filename3>... | Description: Tests adding and removing multiple files in a single command to verify batch context operations");
+    println!("\n🔍 Testing /context add <filename1> <filename2> <filename3> command and /context remove <filename1> <filename2> <filename3>... | Description: Tests adding and removing multiple files in a single command to verify batch context operations");
 
     let test_file1_path = "/tmp/test_context_file1.py";
     let test_file2_path = "/tmp/test_context_file2.py";
@@ -469,7 +469,7 @@ fn test_add_remove_multiple_file_context()-> Result<(), Box<dyn std::error::Erro
 #[test]
 #[cfg(all(feature = "context", feature = "sanity"))]
 fn test_clear_context_command()-> Result<(), Box<dyn std::error::Error>> {
-    println!("🔍 Testing /context clear command... | Description: Tests the /context clear command to remove all files from context and verify the context is completely cleared");
+    println!("\n🔍 Testing /context clear command... | Description: Tests the /context clear command to remove all files from context and verify the context is completely cleared");
 
     let test_file_path = "/tmp/test_context_file.py";
 

e2etests/tests/core_session/test_clear_command.rs

@@ -4,27 +4,27 @@ use q_cli_e2e_tests::q_chat_helper::QChatSession;
 #[test]
 #[cfg(all(feature = "clear", feature = "sanity"))]
 fn test_clear_command() -> Result<(), Box<dyn std::error::Error>> {
-    println!("🔍 Testing /clear command... | Description: Tests the /clear command to clear conversation history and verify that previous context is no longer remembered by the AI");
+    println!("\n🔍 Testing /clear command... | Description: Tests the /clear command to clear conversation history and verify that previous context is no longer remembered by the AI");
 
     let mut chat = QChatSession::new()?;
     println!("✅ Q Chat session started");
 
     // Send initial message
-    println!("🔍 Sending prompt: 'My name is TestUser'");
+    println!("\n🔍 Sending prompt: 'My name is TestUser'");
     let _initial_response = chat.send_prompt("My name is TestUser")?;
     println!("📝 Initial response: {} bytes", _initial_response.len());
     println!("📝 INITIAL RESPONSE OUTPUT:");
     println!("{}", _initial_response);
     println!("📝 END INITIAL RESPONSE");
 
     // Execute clear command
-    println!("🔍 Executing command: '/clear'");
+    println!("\n🔍 Executing command: '/clear'");
     let _clear_response = chat.execute_command("/clear")?;
 
     println!("✅ Clear command executed");
 
     // Check if AI remembers previous conversation
-    println!("🔍 Sending prompt: 'What is my name?'");
+    println!("\n🔍 Sending prompt: 'What is my name?'");
     let test_response = chat.send_prompt("What is my name?")?;
     println!("📝 Test response: {} bytes", test_response.len());
     println!("📝 TEST RESPONSE OUTPUT:");