test(ini): Reduce the scope of fixtures

Author: epage

tests/testsuite/file_ini.rs

@@ -1,7 +1,5 @@
 #![cfg(feature = "ini")]
 
-use std::path::PathBuf;
-
 use chrono::{DateTime, TimeZone, Utc};
 use serde_derive::Deserialize;
 
@@ -26,7 +24,22 @@ fn test_file() {
     }
 
     let c = Config::builder()
-        .add_source(File::new("tests/Settings", FileFormat::Ini))
+        .add_source(File::from_str(
+            r#"
+debug = true
+production = false
+FOO = FOO should be overridden
+bar = I am bar
+[place]
+name = Torre di Pisa
+longitude = 43.7224985
+latitude = 10.3970522
+favorite = false
+reviews = 3866
+rating = 4.5
+"#,
+            FileFormat::Ini,
+        ))
         .build()
         .unwrap();
     let s: Settings = c.try_deserialize().unwrap();
@@ -49,18 +62,19 @@ fn test_file() {
 #[test]
 fn test_error_parse() {
     let res = Config::builder()
-        .add_source(File::new("tests/Settings-invalid", FileFormat::Ini))
+        .add_source(File::from_str(
+            r#"
+ok : true,
+error
+"#,
+            FileFormat::Ini,
+        ))
         .build();
 
-    let path: PathBuf = ["tests", "Settings-invalid.ini"].iter().collect();
-
     assert!(res.is_err());
     assert_eq!(
         res.unwrap_err().to_string(),
-        format!(
-            r#"3:1 expecting "[Some('='), Some(':')]" but found EOF. in {}"#,
-            path.display()
-        )
+        format!(r#"4:1 expecting "[Some('='), Some(':')]" but found EOF."#,)
     );
 }
 
@@ -81,7 +95,22 @@ fn test_override_uppercase_value_for_struct() {
     std::env::set_var("APP_FOO", "I HAVE BEEN OVERRIDDEN_WITH_UPPER_CASE");
 
     let cfg = Config::builder()
-        .add_source(File::new("tests/Settings", FileFormat::Ini))
+        .add_source(File::from_str(
+            r#"
+debug = true
+production = false
+FOO = FOO should be overridden
+bar = I am bar
+[place]
+name = Torre di Pisa
+longitude = 43.7224985
+latitude = 10.3970522
+favorite = false
+reviews = 3866
+rating = 4.5
+"#,
+            FileFormat::Ini,
+        ))
         .add_source(config::Environment::with_prefix("APP").separator("_"))
         .build()
         .unwrap();
@@ -121,7 +150,22 @@ fn test_override_lowercase_value_for_struct() {
     std::env::set_var("config_foo", "I have been overridden_with_lower_case");
 
     let cfg = Config::builder()
-        .add_source(File::new("tests/Settings", FileFormat::Ini))
+        .add_source(File::from_str(
+            r#"
+debug = true
+production = false
+FOO = FOO should be overridden
+bar = I am bar
+[place]
+name = Torre di Pisa
+longitude = 43.7224985
+latitude = 10.3970522
+favorite = false
+reviews = 3866
+rating = 4.5
+"#,
+            FileFormat::Ini,
+        ))
         .add_source(config::Environment::with_prefix("config").separator("_"))
         .build()
         .unwrap();
@@ -144,7 +188,12 @@ fn test_override_uppercase_value_for_enums() {
     std::env::set_var("APPS_BAR", "I HAVE BEEN OVERRIDDEN_WITH_UPPER_CASE");
 
     let cfg = Config::builder()
-        .add_source(File::new("tests/Settings-enum-test", FileFormat::Ini))
+        .add_source(File::from_str(
+            r#"
+bar = "bar is a lowercase param"
+"#,
+            FileFormat::Ini,
+        ))
         .add_source(config::Environment::with_prefix("APPS").separator("_"))
         .build()
         .unwrap();
@@ -166,7 +215,12 @@ fn test_override_lowercase_value_for_enums() {
     std::env::set_var("test_bar", "I have been overridden_with_lower_case");
 
     let cfg = Config::builder()
-        .add_source(File::new("tests/Settings-enum-test", FileFormat::Ini))
+        .add_source(File::from_str(
+            r#"
+bar = "bar is a lowercase param"
+"#,
+            FileFormat::Ini,
+        ))
         .add_source(config::Environment::with_prefix("test").separator("_"))
         .build()
         .unwrap();