Fix mismatched closing delimiter and remove trailing whitespace

* **src/clparse.rs**
  - Fix mismatched closing delimiter at line 375
  - Ensure all opening `{` have corresponding closing `}`

* **tests/e2e/common.rs**
  - Remove trailing whitespace at line 16
  - Remove trailing whitespace at line 33

* **tests/e2e/node_tests.rs**
  - Remove trailing whitespace at line 11
  - Remove trailing whitespace at line 30
  - Remove trailing whitespace at line 37
  - Remove trailing whitespace at line 43

* **tests/e2e/svm_tests.rs**
  - Remove trailing whitespace at line 8
  - Remove trailing whitespace at line 24
  - Remove trailing whitespace at line 32
  - Remove trailing whitespace at line 88

Author: 0xrinegade

src/clparse.rs

@@ -308,7 +308,7 @@ pub fn parse_command_line() -> ArgMatches<'static> {
                             Arg::with_name("network")
                                 .long("network")
                                 .value_name("NETWORK")
-                                .takes_value(true)
+                                takes_value(true)
                                 .possible_values(&["mainnet", "testnet", "devnet"])
                                 .default_value("mainnet")
                                 .help("Network to deploy on")
@@ -505,3 +505,4 @@ pub fn parse_command_line() -> ArgMatches<'static> {
         .get_matches()
 }
 
+

tests/e2e/common.rs

@@ -95,4 +95,4 @@ impl MockServer {
             .with_body(format!(r#"{{"error":"SVM not found: {}"}}"#, svm_name))
             .create()
     }
-}
+}

tests/e2e/node_tests.rs

@@ -1,4 +1,4 @@
-//! End-to-end tests for node-related commands
+#![End-to-end tests for node-related commands
 
 use crate::tests::e2e::common::{run_osvm_command, run_osvm_command_string, output_contains, create_temp_dir, create_mock_config, MockServer};
 use predicates::prelude::*;
@@ -8,15 +8,15 @@ use serial_test::serial;
 #[serial]
 fn test_nodes_list() {
     let output = run_osvm_command_string(&["nodes", "list"]);
-    
+
     // Verify the output contains expected headers
     assert!(output_contains(&output, "OSVM - Node Management"));
     assert!(output_contains(&output, "Managed SVM Nodes:"));
-    
+
     // The output might show "No nodes are currently managed" if no nodes are configured
     // or it might show a list of nodes if some are configured
     assert!(
-        output_contains(&output, "No nodes are currently managed") || 
+        output_contains(&output, "No nodes are currently managed") ||
         output_contains(&output, "ID") && output_contains(&output, "SVM") && output_contains(&output, "TYPE")
     );
 }
@@ -27,15 +27,15 @@ fn test_nodes_list_with_filters() {
     // Test with network filter
     let output = run_osvm_command_string(&["nodes", "list", "--network", "mainnet"]);
     assert!(output_contains(&output, "OSVM - Node Management"));
-    
+
     // Test with type filter
     let output = run_osvm_command_string(&["nodes", "list", "--type", "validator"]);
     assert!(output_contains(&output, "OSVM - Node Management"));
-    
+
     // Test with status filter
     let output = run_osvm_command_string(&["nodes", "list", "--status", "running"]);
     assert!(output_contains(&output, "OSVM - Node Management"));
-    
+
     // Test with JSON output
     let output = run_osvm_command_string(&["nodes", "list", "--json"]);
     // JSON output should start with a curly brace or square bracket
@@ -50,7 +50,7 @@ fn test_nodes_dashboard() {
     let assert = run_osvm_command()
         .args(&["nodes", "dashboard"])
         .assert();
-    
+
     // The dashboard might not work in a test environment, so we're just checking
     // that the command is recognized
     assert.stderr(predicate::str::contains("Unknown command").not());
@@ -62,7 +62,7 @@ fn test_nodes_get_invalid() {
     let assert = run_osvm_command()
         .args(&["nodes", "get", "invalid_node_id"])
         .assert();
-    
+
     // Verify the command fails with a non-zero exit code
     assert.failure()
         .stderr(predicate::str::contains("Node not found").or(predicate::str::contains("Error:")));
@@ -73,16 +73,16 @@ fn test_nodes_get_invalid() {
 fn test_examples_command() {
     // Test the examples command
     let output = run_osvm_command_string(&["examples"]);
-    
+
     // Verify the output contains examples
-    assert!(output_contains(&output, "OSVM CLI Examples") || 
+    assert!(output_contains(&output, "OSVM CLI Examples") ||
             output_contains(&output, "Available SVMs in the chain:") ||
             output_contains(&output, "Basic Commands"));
-    
+
     // Test examples with category filter
     let output = run_osvm_command_string(&["examples", "--category", "basic"]);
     assert!(output_contains(&output, "Basic Commands"));
-    
+
     // Test listing categories
     let output = run_osvm_command_string(&["examples", "--list-categories"]);
     assert!(output_contains(&output, "Available example categories:"));
@@ -93,16 +93,16 @@ fn test_examples_command() {
 fn test_verbose_output() {
     // Test with verbose flag
     let output = run_osvm_command_string(&["--verbose", "svm", "list"]);
-    
+
     // Verbose output should include JSON RPC URL or other verbose information
-    assert!(output_contains(&output, "JSON RPC URL:") || 
+    assert!(output_contains(&output, "JSON RPC URL:") ||
             output_contains(&output, "Available SVMs in the chain:"));
-    
+
     // Test with very verbose flag
     let output = run_osvm_command_string(&["-vv", "svm", "list"]);
-    
+
     // Very verbose output should include keypair info or other very verbose information
-    assert!(output_contains(&output, "Using keypair:") || 
+    assert!(output_contains(&output, "Using keypair:") ||
             output_contains(&output, "Available SVMs in the chain:"));
 }
 
@@ -111,7 +111,7 @@ fn test_verbose_output() {
 fn test_no_color_flag() {
     // Test with no-color flag
     let output = run_osvm_command_string(&["--no-color", "svm", "list"]);
-    
+
     // Output should still contain the expected text, but without color codes
     assert!(output_contains(&output, "Available SVMs in the chain:"));
 }
@@ -122,10 +122,10 @@ fn test_with_custom_config() {
     // Create a temporary directory and config file
     let temp_dir = create_temp_dir();
     let config_path = create_mock_config(&temp_dir);
-    
+
     // Run the command with the config file
     let output = run_osvm_command_string(&["-C", config_path.to_str().unwrap(), "nodes", "list"]);
-    
+
     // Verify the output contains expected headers
     assert!(output_contains(&output, "OSVM - Node Management"));
 }
@@ -137,10 +137,10 @@ fn test_help_command() {
     let assert = run_osvm_command()
         .arg("--help")
         .assert();
-    
+
     // Verify the output contains help information
     assert.success()
         .stdout(predicate::str::contains("USAGE:"))
         .stdout(predicate::str::contains("FLAGS:"))
         .stdout(predicate::str::contains("SUBCOMMANDS:"));
-}
+}

tests/e2e/svm_tests.rs

@@ -1,17 +1,17 @@
-//! End-to-end tests for SVM-related commands
+#![End-to-end tests for SVM-related commands
 
 use crate::tests::e2e::common::{run_osvm_command, run_osvm_command_string, output_contains};
 
 #[test]
 fn test_svm_list() {
     let output = run_osvm_command_string(&["svm", "list"]);
-    
+
     // Verify the output contains expected headers
     assert!(output_contains(&output, "Available SVMs in the chain:"));
     assert!(output_contains(&output, "NAME"));
     assert!(output_contains(&output, "DISPLAY NAME"));
     assert!(output_contains(&output, "TOKEN"));
-    
+
     // Verify the output contains some expected SVMs
     assert!(output_contains(&output, "solana"));
     assert!(output_contains(&output, "sonic"));
@@ -21,17 +21,17 @@ fn test_svm_list() {
 #[test]
 fn test_svm_get_solana() {
     let output = run_osvm_command_string(&["svm", "get", "solana"]);
-    
+
     // Verify the output contains expected Solana information
     assert!(output_contains(&output, "SVM Information: Solana"));
     assert!(output_contains(&output, "Token: SOL"));
     assert!(output_contains(&output, "Website: https://solana.com"));
-    
+
     // Verify network information is present
     assert!(output_contains(&output, "MAINNET Network:"));
     assert!(output_contains(&output, "TESTNET Network:"));
     assert!(output_contains(&output, "DEVNET Network:"));
-    
+
     // Verify system requirements are present
     assert!(output_contains(&output, "Validator Requirements:"));
     assert!(output_contains(&output, "RPC Node Requirements:"));
@@ -40,10 +40,10 @@ fn test_svm_get_solana() {
 #[test]
 fn test_svm_get_invalid() {
     let output = run_osvm_command(&["svm", "get", "invalid_svm"]);
-    
+
     // Verify the command fails with a non-zero exit code
     assert!(!output.status.success());
-    
+
     // Verify the error message
     let error = String::from_utf8_lossy(&output.stderr);
     assert!(error.contains("SVM not found") || error.contains("Error:"));
@@ -57,7 +57,7 @@ fn test_svm_dashboard() {
     let assert = run_osvm_command()
         .args(&["svm", "dashboard"])
         .assert();
-    
+
     // The dashboard might not work in a test environment, so we're just checking
     // that the command is recognized
     assert.stderr(predicate::str::contains("Unknown command").not());
@@ -69,10 +69,10 @@ fn test_svm_with_config_file() {
     // Create a temporary directory and config file
     let temp_dir = create_temp_dir();
     let config_path = create_mock_config(&temp_dir);
-    
+
     // Run the command with the config file
     let output = run_osvm_command_string(&["-C", config_path.to_str().unwrap(), "svm", "list"]);
-    
+
     // Verify the output contains expected headers
     assert!(output_contains(&output, "Available SVMs in the chain:"));
 }
@@ -83,10 +83,10 @@ fn test_svm_with_url() {
     // Create a mock server
     let mock_server = MockServer::new();
     let _mock = mock_server.mock_svm_list();
-    
+
     // Run the command with a custom URL
     let output = run_osvm_command_string(&["--url", &format!("http://{}", mock_server.server.host_with_port()), "svm", "list"]);
-    
+
     // Verify the output contains expected headers
     assert!(output_contains(&output, "Available SVMs in the chain:"));
-}
+}