Increase test coverage pre-release

Author: IvanIsCoding

src/args2cel_test.rs

@@ -43,6 +43,35 @@ fn test_bool() {
     assert!(matches!(vars.get("x").unwrap(), CelValue::Bool(true)));
 }
 
+#[test]
+fn test_alias_types() {
+    let args = vec![
+        ("x".to_string(), "i64".to_string(), "42".to_string()),
+        ("y".to_string(), "u64".to_string(), "7".to_string()),
+        ("z".to_string(), "double".to_string(), "2.5".to_string()),
+        ("w".to_string(), "boolean".to_string(), "false".to_string()),
+        ("s".to_string(), "str".to_string(), "hello".to_string()),
+    ];
+    let vars = args_to_cel_variables(&args).unwrap();
+
+    assert!(matches!(vars.get("x").unwrap(), CelValue::Int(42)));
+    assert!(matches!(vars.get("y").unwrap(), CelValue::UInt(7)));
+    assert!(matches!(vars.get("z").unwrap(), CelValue::Float(f) if (*f - 2.5).abs() < 0.001));
+    assert!(matches!(vars.get("w").unwrap(), CelValue::Bool(false)));
+    assert!(matches!(vars.get("s").unwrap(), CelValue::String(s) if s.as_str() == "hello"));
+}
+
+#[test]
+fn test_duplicate_args_last_wins() {
+    let args = vec![
+        ("x".to_string(), "int".to_string(), "1".to_string()),
+        ("x".to_string(), "int".to_string(), "2".to_string()),
+    ];
+    let vars = args_to_cel_variables(&args).unwrap();
+    assert_eq!(vars.len(), 1);
+    assert!(matches!(vars.get("x").unwrap(), CelValue::Int(2)));
+}
+
 #[test]
 fn test_multiple_args() {
     let args = vec![

src/input_handler_test.rs

@@ -54,6 +54,20 @@ fn test_handle_json_with_args() {
     assert!(is_truthy);
 }
 
+#[test]
+fn test_handle_json_input_overrides_arg_with_same_name() {
+    let program = Program::compile("this.value").unwrap();
+    let mut args = BTreeMap::new();
+    args.insert("this".to_string(), CelValue::Int(999));
+    let json = r#"{"value": 50}"#;
+    let params = default_params();
+
+    let (output, is_truthy) = handle_json(&program, &args, &params, Some(json)).unwrap();
+
+    assert!(output.contains("50"));
+    assert!(is_truthy);
+}
+
 #[test]
 fn test_handle_json_args_and_json() {
     let program = Program::compile("x + this.value").unwrap();
@@ -114,6 +128,36 @@ fn test_handle_json_truthiness_empty_string() {
     assert!(!is_truthy);
 }
 
+#[test]
+fn test_handle_json_raw_output_for_string() {
+    let program = Program::compile(r#""hello""#).unwrap();
+    let args = BTreeMap::new();
+    let mut params = default_params();
+    params.raw_output = true;
+
+    let (output, is_truthy) = handle_json(&program, &args, &params, None).unwrap();
+
+    assert_eq!(output, "hello");
+    assert!(is_truthy);
+}
+
+#[test]
+fn test_handle_json_sorted_pretty_output() {
+    let program = Program::compile(r#"{"z": {"b": 2, "a": 1}, "a": 0}"#).unwrap();
+    let args = BTreeMap::new();
+    let mut params = default_params();
+    params.sort_keys = true;
+    params.pretty_print = true;
+
+    let (output, is_truthy) = handle_json(&program, &args, &params, None).unwrap();
+
+    assert_eq!(
+        output,
+        "{\n  \"a\": 0,\n  \"z\": {\n    \"a\": 1,\n    \"b\": 2\n  }\n}"
+    );
+    assert!(is_truthy);
+}
+
 #[test]
 fn test_handle_json_invalid_json() {
     let program = Program::compile("this.x").unwrap();
@@ -239,3 +283,20 @@ fn test_handle_buffer_skip_empty_lines() {
     assert!(results[1].0.contains("2"));
     assert!(results[2].0.contains("3"));
 }
+
+#[test]
+fn test_handle_buffer_custom_root_var() {
+    let program = Program::compile("request.x").unwrap();
+    let args = BTreeMap::new();
+    let input = r#"{"x": 42}"#;
+    let cursor = Cursor::new(input.as_bytes());
+    let reader = BufReader::new(cursor);
+    let mut params = default_params();
+    params.root_var = "request".to_string();
+
+    let results = handle_buffer(&program, &args, &params, reader).unwrap();
+
+    assert_eq!(results.len(), 1);
+    assert_eq!(results[0].0, "42");
+    assert!(results[0].1);
+}

src/json2cel_test.rs

@@ -169,6 +169,18 @@ fn test_slurp_parallel() {
     }
 }
 
+#[test]
+fn test_slurp_empty_input() {
+    let vars =
+        json_to_cel_variables("", ROOT_VAR, InputFormat::SlurpJson, DEFAULT_PARALLELISM).unwrap();
+
+    if let CelValue::List(list) = vars.get("this").unwrap() {
+        assert!(list.is_empty());
+    } else {
+        panic!("Expected list");
+    }
+}
+
 #[test]
 #[cfg(feature = "from-toml")]
 fn test_toml_format() {
@@ -245,6 +257,24 @@ fn test_yaml_format_disabled() {
     assert!(vars.is_err());
 }
 
+#[test]
+#[cfg(feature = "from-yaml")]
+fn test_yaml_multi_document_format() {
+    let yaml_input = r#"name: first
+---
+name: second
+"#;
+
+    let vars = json_to_cel_variables(yaml_input, ROOT_VAR, InputFormat::Yaml, DEFAULT_PARALLELISM)
+        .unwrap();
+
+    if let CelValue::List(list) = vars.get("this").unwrap() {
+        assert_eq!(list.len(), 2);
+    } else {
+        panic!("Expected list");
+    }
+}
+
 #[test]
 #[cfg(feature = "greppable")]
 fn test_gron_format() {

src/ungron_test.rs

@@ -80,3 +80,24 @@ json.nothing = null;"#;
 
     assert_eq!(result, expected);
 }
+
+#[test]
+fn test_empty_container_assignments_do_not_overwrite_existing_values() {
+    let input = r#"json = {};
+json.user = {};
+json.user.name = "John";
+json.user = {};
+json.items = [];
+json.items[0] = "first";
+json.items = [];"#;
+
+    let result = gron_to_json(input).unwrap();
+    let expected: JsonValue = serde_json::json!({
+        "user": {
+            "name": "John"
+        },
+        "items": ["first"]
+    });
+
+    assert_eq!(result, expected);
+}

tests/golden.rs

@@ -472,6 +472,18 @@ test!(
     "20"
 );
 
+test!(
+    rename_root_variable_overrides_same_named_arg,
+    &[
+        "--root-var=request",
+        "--arg",
+        "request:int=99",
+        "request.value"
+    ],
+    r#"{"value":42}"#,
+    "42"
+);
+
 // Multi-line JSON5 (with trailing comma and comment)
 test!(
     multiline_json5_object,
@@ -692,6 +704,16 @@ value: third
     "[\"first\",\"second\",\"third\"]"
 );
 
+#[cfg(feature = "from-yaml")]
+test!(
+    yaml_multi_document_single_doc_stays_scalar,
+    &["--from-yaml", "this.name"],
+    r#"name: Alice
+age: 30
+"#,
+    "\"Alice\""
+);
+
 // Void test
 test!(void_mode, &["--void", "-n", "2 + 2"], "", "");
 
@@ -857,6 +879,17 @@ json.a.b.c.d[0].e = "deep";
     r#"{"a":{"b":{"c":{"d":[{"e":"deep"}]}}}}"#
 );
 
+#[cfg(feature = "greppable")]
+test!(
+    greppable_round_trip_special_keys,
+    &["--from-gron", "-S", "this"],
+    r#"json = {};
+json["key with spaces"] = "value";
+json["key-with-dashes"] = "other";
+"#,
+    r#"{"key with spaces":"value","key-with-dashes":"other"}"#
+);
+
 // Slice extension tests
 test!(
     slice_basic,
@@ -886,6 +919,13 @@ test!(
     "[3]"
 );
 
+test!(
+    slice_empty_result,
+    &["this.items.slice(3, 3)"],
+    r#"{"items":[1,2,3,4]}"#,
+    "[]"
+);
+
 test!(
     slice_negative_to_positive,
     &["this.items.slice(-5, 4)"],