feat: add network validator and fix warnings

Author: dotandev

Cargo.toml

@@ -23,6 +23,7 @@ anyhow = "1.0"
 thiserror = "1.0"
 regex = "1.10"
 serde_json = "1.0"
+reqwest = { version = "0.11", features = ["blocking", "rustls-tls"], default-features = false }
 
 [dev-dependencies]
 tempfile = "3.10"

src/config.rs

@@ -14,6 +14,8 @@ pub struct Config {
     pub ports: Vec<u16>,
     #[serde(default)]
     pub files: Vec<FileCheck>,
+    #[serde(default)]
+    pub network: Vec<NetworkCheck>,
 }
 
 #[derive(Debug, Deserialize, Serialize, Clone)]
@@ -45,6 +47,13 @@ pub struct FileCheck {
     pub permissions: Option<u32>,
 }
 
+#[derive(Debug, Deserialize, Serialize, Clone)]
+pub struct NetworkCheck {
+    pub url: String,
+    #[serde(default)]
+    pub status_code: Option<u16>,
+}
+
 fn default_true() -> bool {
     true
 }

src/main.rs

@@ -71,6 +71,9 @@ ports:
 files:
   - path: .env
     required: false
+network:
+  - url: https://github.com
+    status_code: 200
 "#;
 
     std::fs::write(config_path, default_config)?;

src/validators/mod.rs

@@ -6,6 +6,7 @@ pub mod tool;
 pub mod env;
 pub mod port;
 pub mod file;
+pub mod network;
 
 #[derive(Debug, Clone, Serialize)]
 #[serde(rename_all = "lowercase")]
@@ -79,5 +80,11 @@ pub fn run_all_validations(config: &Config) -> Result<Vec<ValidationResult>> {
         results.extend(validator.validate()?);
     }
 
+    // Validate network
+    for network_check in &config.network {
+        let validator = network::NetworkValidator::new(network_check.clone());
+        results.extend(validator.validate()?);
+    }
+
     Ok(results)
 }

src/validators/network.rs

@@ -0,0 +1,56 @@
+use crate::config::NetworkCheck;
+use crate::validators::{ValidationResult, Validator};
+use anyhow::Result;
+use reqwest::blocking::Client;
+use std::time::Duration;
+
+pub struct NetworkValidator {
+    check: NetworkCheck,
+    client: Client,
+}
+
+impl NetworkValidator {
+    pub fn new(check: NetworkCheck) -> Self {
+        let client = Client::builder()
+            .timeout(Duration::from_secs(5))
+            .build()
+            .unwrap_or_else(|_| Client::new());
+            
+        Self { check, client }
+    }
+}
+
+impl Validator for NetworkValidator {
+    fn validate(&self) -> Result<Vec<ValidationResult>> {
+        let mut results = Vec::new();
+        
+        let url = &self.check.url;
+        let expected_status = self.check.status_code.unwrap_or(200);
+        
+        match self.client.get(url).send() {
+            Ok(response) => {
+                let status = response.status().as_u16();
+                
+                if status == expected_status {
+                    results.push(ValidationResult::success(format!(
+                        "Successfully connected to {} (Status: {})", 
+                        url, status
+                    )));
+                } else {
+                    results.push(ValidationResult::error(
+                        format!("Connected to {} but got status {}", url, status),
+                        Some(format!("Expected status code {}", expected_status)),
+                    ));
+                }
+            }
+            Err(e) => {
+                results.push(ValidationResult::error(
+                    format!("Failed to connect to {}", url),
+                    Some(format!("Error: {}", e)),
+                ));
+            }
+        }
+
+        Ok(results)
+    }
+}

tests/integration_tests.rs

@@ -5,7 +5,7 @@ use std::io::Write;
 
 #[test]
 fn test_cli_help() {
-    let mut cmd = Command::cargo_bin("envcheck").unwrap();
+    let mut cmd = Command::new(env!("CARGO_BIN_EXE_envcheck"));
     cmd.arg("--help");
     cmd.assert()
         .success()
@@ -14,7 +14,7 @@ fn test_cli_help() {
 
 #[test]
 fn test_cli_config_not_found() {
-    let mut cmd = Command::cargo_bin("envcheck").unwrap();
+    let mut cmd = Command::new(env!("CARGO_BIN_EXE_envcheck"));
     cmd.arg("--config").arg("non_existent_file.yaml");
     cmd.assert()
         .failure()
@@ -41,7 +41,7 @@ files:
         path_str
     ).unwrap();
 
-    let mut cmd = Command::cargo_bin("envcheck").unwrap();
+    let mut cmd = Command::new(env!("CARGO_BIN_EXE_envcheck"));
     cmd.arg("--config").arg(file.path());
 
     // Node check might fail if not installed in CI environment, but PATH and file should pass
@@ -61,7 +61,7 @@ ports:
 "#
     ).unwrap();
 
-    let mut cmd = Command::cargo_bin("envcheck").unwrap();
+    let mut cmd = Command::new(env!("CARGO_BIN_EXE_envcheck"));
     cmd.arg("--config").arg(file.path());
     cmd.assert()
         .success()
@@ -73,7 +73,7 @@ fn test_cli_init() {
     let temp_dir = tempfile::tempdir().unwrap();
     let config_path = temp_dir.path().join(".envcheck.yaml");
 
-    let mut cmd = Command::cargo_bin("envcheck").unwrap();
+    let mut cmd = Command::new(env!("CARGO_BIN_EXE_envcheck"));
     cmd.current_dir(temp_dir.path()).arg("init");
     cmd.assert().success();
 
@@ -95,7 +95,7 @@ env_vars:
 "#
     ).unwrap();
 
-    let mut cmd = Command::cargo_bin("envcheck").unwrap();
+    let mut cmd = Command::new(env!("CARGO_BIN_EXE_envcheck"));
     cmd.arg("--config").arg(file.path()).arg("--json");
 
     let output = cmd.assert().success().get_output().stdout.clone();
@@ -122,15 +122,15 @@ env_vars:
     ).unwrap();
 
     // Test failure
-    let mut cmd = Command::cargo_bin("envcheck").unwrap();
+    let mut cmd = Command::new(env!("CARGO_BIN_EXE_envcheck"));
     cmd.arg("--config").arg(file.path())
        .env("TEST_REGEX_VAR", "abc");
     cmd.assert()
         .failure()
         .stdout(predicate::str::contains("TEST_REGEX_VAR is set but does not match pattern"));
 
     // Test success
-    let mut cmd = Command::cargo_bin("envcheck").unwrap();
+    let mut cmd = Command::new(env!("CARGO_BIN_EXE_envcheck"));
     cmd.arg("--config").arg(file.path())
        .env("TEST_REGEX_VAR", "123");
     cmd.assert()
@@ -155,7 +155,7 @@ files:
         dir_path
     ).unwrap();
 
-    let mut cmd = Command::cargo_bin("envcheck").unwrap();
+    let mut cmd = Command::new(env!("CARGO_BIN_EXE_envcheck"));
     cmd.arg("--config").arg(file.path());
     cmd.assert()
         .success()