add test cases for empty file.

Author: up9cloud

Cargo.lock

@@ -1,17 +1,18 @@
 use std::error::Error;
 
-use crate::error::{ConfigError, Unexpected};
+use crate::format;
 use crate::map::Map;
 use crate::value::{Value, ValueKind};
-
 use jsonc_parser::JsonValue;
 
 pub(crate) fn parse(
     uri: Option<&String>,
     text: &str,
 ) -> Result<Map<String, Value>, Box<dyn Error + Send + Sync>> {
-    let parsed = jsonc_parser::parse_to_value(text, &Default::default())?;
-    let value = from_jsonc_value(uri, parsed.unwrap_or(ValueKind::Nil));
+    let value = match jsonc_parser::parse_to_value(text, &Default::default())? {
+        Some(json_value) => from_jsonc_value(uri, json_value),
+        None => Value::new(uri, ValueKind::Nil),
+    };
     format::extract_root_table(uri, value)
 }
 

src/file/format/jsonc.rs

@@ -252,3 +252,14 @@ fn ini() {
     let date: DateTime<Utc> = s.get("ini_datetime").unwrap();
     assert_eq!(date, Utc.with_ymd_and_hms(2017, 5, 10, 2, 14, 53).unwrap());
 }
+
+#[test]
+fn test_nothing() {
+    let res = Config::builder()
+        .add_source(File::from_str("", FileFormat::Ini))
+        .build();
+    assert!(res.is_ok());
+    let c = res.unwrap();
+    let cc = format!("{:?}", c);
+    assert_data_eq!(cc, str!("Config { defaults: {}, overrides: {}, sources: [], cache: Value { origin: None, kind: Table({}) } }"));
+}

tests/testsuite/file_ini.rs

@@ -316,3 +316,15 @@ fn json() {
     let date: DateTime<Utc> = s.get("json_datetime").unwrap();
     assert_eq!(date, Utc.with_ymd_and_hms(2017, 5, 10, 2, 14, 53).unwrap());
 }
+
+#[test]
+fn test_nothing() {
+    let res = Config::builder()
+        .add_source(File::from_str("", FileFormat::Json))
+        .build();
+    assert!(res.is_err());
+    assert_data_eq!(
+        res.unwrap_err().to_string(),
+        format!("EOF while parsing a value at line 1 column 0")
+    );
+}

tests/testsuite/file_json.rs

@@ -325,3 +325,15 @@ fn json() {
     let date: DateTime<Utc> = s.get("json_datetime").unwrap();
     assert_eq!(date, Utc.with_ymd_and_hms(2017, 5, 10, 2, 14, 53).unwrap());
 }
+
+#[test]
+fn test_nothing() {
+    let res = Config::builder()
+        .add_source(File::from_str("", FileFormat::Json5))
+        .build();
+    assert!(res.is_err());
+    assert_data_eq!(
+        res.unwrap_err().to_string(),
+        format!(" --> 1:1\n  |\n1 | \n  | ^---\n  |\n  = expected array, boolean, null, number, object, or string")
+    );
+}

tests/testsuite/file_json5.rs

@@ -101,7 +101,7 @@ fn test_error_parse() {
     assert_eq!(
         res.unwrap_err().to_string(),
         format!(
-            "Expected a colon after the string or word in an object property on line 4 column 1. in {}",
+            "Expected colon after the string or word in object property on line 4 column 1 in {}",
             std::path::Path::new(&f)
                 .strip_prefix(std::env::current_dir().unwrap())
                 .unwrap_or(std::path::Path::new(&f))
@@ -187,3 +187,15 @@ fn test_override_lowercase_value_for_enums() {
         str!["enum EnumSettings does not have variant constructor bar"]
     );
 }
+
+#[test]
+fn test_nothing() {
+    let res = Config::builder()
+        .add_source(File::from_str("", FileFormat::Jsonc))
+        .build();
+    assert!(res.is_err());
+    assert_data_eq!(
+        res.unwrap_err().to_string(),
+        format!("invalid type: unit value, expected a map")
+    );
+}

tests/testsuite/file_jsonc.rs

@@ -317,3 +317,15 @@ fn ron() {
     let date: DateTime<Utc> = s.get("ron_datetime").unwrap();
     assert_eq!(date, Utc.with_ymd_and_hms(2021, 4, 19, 11, 33, 2).unwrap());
 }
+
+#[test]
+fn test_nothing() {
+    let res = Config::builder()
+        .add_source(File::from_str("", FileFormat::Ron))
+        .build();
+    assert!(res.is_err());
+    assert_data_eq!(
+        res.unwrap_err().to_string(),
+        format!("1:1: Unexpected end of RON")
+    );
+}

tests/testsuite/file_ron.rs

@@ -438,3 +438,14 @@ fn toml() {
     let date: DateTime<Utc> = s.get("toml_datetime").unwrap();
     assert_eq!(date, Utc.with_ymd_and_hms(2017, 5, 11, 14, 55, 15).unwrap());
 }
+
+#[test]
+fn test_nothing() {
+    let res = Config::builder()
+        .add_source(File::from_str("", FileFormat::Toml))
+        .build();
+    assert!(res.is_ok());
+    let c = res.unwrap();
+    let cc = format!("{:?}", c);
+    assert_data_eq!(cc, str!("Config { defaults: {}, overrides: {}, sources: [], cache: Value { origin: None, kind: Table({}) } }"));
+}

tests/testsuite/file_toml.rs

@@ -406,3 +406,14 @@ inner_float:
     assert_eq!(config.inner_float.get("0.1").unwrap(), "float 0.1");
     assert_eq!(config.inner_float.get("0.2").unwrap(), "float 0.2");
 }
+
+#[test]
+fn test_nothing() {
+    let res = Config::builder()
+        .add_source(File::from_str("", FileFormat::Yaml))
+        .build();
+    assert!(res.is_ok());
+    let c = res.unwrap();
+    let cc = format!("{:?}", c);
+    assert_data_eq!(cc, str!("Config { defaults: {}, overrides: {}, sources: [], cache: Value { origin: None, kind: Table({}) } }"));
+}