Use ConfigBuilder in env.rs tests

Author: szarykott

src/config.rs

@@ -1,7 +1,7 @@
 use std::collections::HashMap;
 use std::fmt::Debug;
 
-use builder::ConfigBuilder;
+use crate::builder::ConfigBuilder;
 use serde::de::Deserialize;
 use serde::ser::Serialize;
 

tests/env.rs

@@ -105,11 +105,13 @@ fn test_parse_int() {
     env::set_var("INT_VAL", "42");
 
     let environment = Environment::new().try_parsing(true);
-    let mut config = Config::default();
 
-    config.set("tag", "Int").unwrap();
-
-    config.merge(environment).unwrap();
+    let config = Config::builder()
+        .set_default("tag", "Int")
+        .unwrap()
+        .add_source(environment)
+        .build()
+        .unwrap();
 
     let config: TestIntEnum = config.try_into().unwrap();
 
@@ -135,11 +137,13 @@ fn test_parse_float() {
     env::set_var("FLOAT_VAL", "42.3");
 
     let environment = Environment::new().try_parsing(true);
-    let mut config = Config::default();
-
-    config.set("tag", "Float").unwrap();
 
-    config.merge(environment).unwrap();
+    let config = Config::builder()
+        .set_default("tag", "Float")
+        .unwrap()
+        .add_source(environment)
+        .build()
+        .unwrap();
 
     let config: TestFloatEnum = config.try_into().unwrap();
 
@@ -168,11 +172,13 @@ fn test_parse_bool() {
     env::set_var("BOOL_VAL", "true");
 
     let environment = Environment::new().try_parsing(true);
-    let mut config = Config::default();
-
-    config.set("tag", "Bool").unwrap();
 
-    config.merge(environment).unwrap();
+    let config = Config::builder()
+        .set_default("tag", "Bool")
+        .unwrap()
+        .add_source(environment)
+        .build()
+        .unwrap();
 
     let config: TestBoolEnum = config.try_into().unwrap();
 
@@ -202,11 +208,13 @@ fn test_parse_off_int() {
     env::set_var("INT_VAL_1", "42");
 
     let environment = Environment::new().try_parsing(false);
-    let mut config = Config::default();
 
-    config.set("tag", "Int").unwrap();
-
-    config.merge(environment).unwrap();
+    let config = Config::builder()
+        .set_default("tag", "Int")
+        .unwrap()
+        .add_source(environment)
+        .build()
+        .unwrap();
 
     env::remove_var("INT_VAL_1");
 
@@ -231,11 +239,13 @@ fn test_parse_off_float() {
     env::set_var("FLOAT_VAL_1", "42.3");
 
     let environment = Environment::new().try_parsing(false);
-    let mut config = Config::default();
-
-    config.set("tag", "Float").unwrap();
 
-    config.merge(environment).unwrap();
+    let config = Config::builder()
+        .set_default("tag", "Float")
+        .unwrap()
+        .add_source(environment)
+        .build()
+        .unwrap();
 
     env::remove_var("FLOAT_VAL_1");
 
@@ -260,11 +270,13 @@ fn test_parse_off_bool() {
     env::set_var("BOOL_VAL_1", "true");
 
     let environment = Environment::new().try_parsing(false);
-    let mut config = Config::default();
-
-    config.set("tag", "Bool").unwrap();
 
-    config.merge(environment).unwrap();
+    let config = Config::builder()
+        .set_default("tag", "Bool")
+        .unwrap()
+        .add_source(environment)
+        .build()
+        .unwrap();
 
     env::remove_var("BOOL_VAL_1");
 
@@ -289,11 +301,13 @@ fn test_parse_int_fail() {
     env::set_var("INT_VAL_2", "not an int");
 
     let environment = Environment::new().try_parsing(true);
-    let mut config = Config::default();
 
-    config.set("tag", "Int").unwrap();
-
-    config.merge(environment).unwrap();
+    let config = Config::builder()
+        .set_default("tag", "Int")
+        .unwrap()
+        .add_source(environment)
+        .build()
+        .unwrap();
 
     env::remove_var("INT_VAL_2");
 
@@ -318,11 +332,13 @@ fn test_parse_float_fail() {
     env::set_var("FLOAT_VAL_2", "not a float");
 
     let environment = Environment::new().try_parsing(true);
-    let mut config = Config::default();
-
-    config.set("tag", "Float").unwrap();
 
-    config.merge(environment).unwrap();
+    let config = Config::builder()
+        .set_default("tag", "Float")
+        .unwrap()
+        .add_source(environment)
+        .build()
+        .unwrap();
 
     env::remove_var("FLOAT_VAL_2");
 
@@ -347,11 +363,13 @@ fn test_parse_bool_fail() {
     env::set_var("BOOL_VAL_2", "not a bool");
 
     let environment = Environment::new().try_parsing(true);
-    let mut config = Config::default();
-
-    config.set("tag", "Bool").unwrap();
 
-    config.merge(environment).unwrap();
+    let config = Config::builder()
+        .set_default("tag", "Bool")
+        .unwrap()
+        .add_source(environment)
+        .build()
+        .unwrap();
 
     env::remove_var("BOOL_VAL_2");
 
@@ -375,11 +393,13 @@ fn test_parse_string() {
     env::set_var("STRING_VAL", "test string");
 
     let environment = Environment::new().try_parsing(true);
-    let mut config = Config::default();
 
-    config.set("tag", "String").unwrap();
-
-    config.merge(environment).unwrap();
+    let config = Config::builder()
+        .set_default("tag", "String")
+        .unwrap()
+        .add_source(environment)
+        .build()
+        .unwrap();
 
     let config: TestStringEnum = config.try_into().unwrap();
 
@@ -409,11 +429,13 @@ fn test_parse_off_string() {
     env::set_var("STRING_VAL_1", "test string");
 
     let environment = Environment::new().try_parsing(false);
-    let mut config = Config::default();
-
-    config.set("tag", "String").unwrap();
 
-    config.merge(environment).unwrap();
+    let config = Config::builder()
+        .set_default("tag", "String")
+        .unwrap()
+        .add_source(environment)
+        .build()
+        .unwrap();
 
     let config: TestStringEnum = config.try_into().unwrap();