Coverage

components/datap_json/src/lib.rs

@@ -23,6 +23,7 @@ impl From<serde_json::error::Error> for Error {
 }
 
 /// A data provider reading from a JSON file.
+#[derive(Debug)]
 pub struct JsonDataProvider {
     data: schema::JsonSchema,
 }
@@ -58,3 +59,15 @@ impl<'a> datap::DataProvider<'a, 'a> for JsonDataProvider {
         Ok(response)
     }
 }
+
+#[test]
+fn test_empty_str() {
+    let result = JsonDataProvider::from_str("");
+    assert!(result.is_err());
+    let err = result.unwrap_err();
+    println!("{:?}", err);  // Coverage for Debug trait
+    // An unconditional let is possible here because it is a one-element enum.
+    // If more cases are needed, see https://github.com/rust-lang/rfcs/pull/1303
+    let Error::JsonError(json_err) = err;
+    assert_eq!(json_err.classify(), serde_json::error::Category::Eof);
+}

components/datap_json/src/schema.rs

@@ -4,12 +4,12 @@ use serde::{Deserialize, Serialize};
 #[allow(unused_imports)]
 use std::prelude::v1::*;
 
-#[derive(Serialize, Deserialize)]
+#[derive(Serialize, Deserialize, Debug)]
 pub(crate) struct DecimalJsonSchema {
     pub(crate) symbols_v1_a: datap::decimal::SymbolsV1,
 }
 
-#[derive(Serialize, Deserialize)]
+#[derive(Serialize, Deserialize, Debug)]
 pub(crate) struct JsonSchema {
     pub(crate) decimal: DecimalJsonSchema,
 }

components/datap_json/tests/test_file_io.rs

@@ -14,6 +14,7 @@ fn test_read_json() {
     let file = File::open("tests/testdata/all.json").unwrap();
     let reader = BufReader::new(file);
     let json_data_provider = JsonDataProvider::from_reader(reader).unwrap();
+    println!("{:?}", json_data_provider);  // Coverage for Debug trait
     let response = json_data_provider
         .load(&datap::Request {
             locale: "root".to_string(),

components/datap_json/tests/test_no_std.rs

@@ -51,6 +51,14 @@ fn test_read_string() {
     check_data(decimal_data);
 }
 
+#[test]
+fn test_read_utf8() {
+    let provider = JsonDataProvider::from_slice(DATA.as_bytes()).unwrap();
+    let response = get_response(&provider);
+    let decimal_data: &datap::decimal::SymbolsV1 = response.borrow_payload().unwrap();
+    check_data(decimal_data);
+}
+
 #[test]
 fn test_borrow_payload_mut() {
     let provider = get_provider();