Uses into_* for value conversions and adds bool as an option

Author: joelgallant

src/env.rs

@@ -126,10 +126,14 @@ impl Source for Environment {
             }
 
             let value = if self.parse_numbers {
-                if let Ok(parsed) = value.parse() {
+                let string_value = Value::new(Some(&uri), ValueKind::String(value.clone()));
+
+                if let Ok(parsed) = string_value.clone().into_int() {
                     ValueKind::Integer(parsed)
-                } else if let Ok(parsed) = value.parse() {
+                } else if let Ok(parsed) = string_value.clone().into_float() {
                     ValueKind::Float(parsed)
+                } else if let Ok(parsed) = string_value.clone().into_bool() {
+                    ValueKind::Boolean(parsed)
                 } else {
                     ValueKind::String(value)
                 }

tests/env.rs

@@ -87,6 +87,7 @@ fn test_custom_separator_behavior() {
 fn test_parse_numbers() {
     env::set_var("INT_VAL", "42");
     env::set_var("FLOAT_VAL", "42.2");
+    env::set_var("BOOL_VAL", "true");
 
     let environment = Environment::new().parse_numbers(true);
     let values = environment.collect().unwrap();
@@ -99,7 +100,12 @@ fn test_parse_numbers() {
         values.get("float_val").unwrap().clone().into_float().ok(),
         Some(42.2)
     );
+    assert_eq!(
+        values.get("bool_val").unwrap().clone().into_bool().ok(),
+        Some(true)
+    );
 
     env::remove_var("INT_VAL");
     env::remove_var("FLOAT_VAL");
+    env::remove_var("BOOL_VAL");
 }