Tests + Changelog for 0.3.4 release

Author: IvanIsCoding

CHANGELOG.md

@@ -1,6 +1,19 @@
 Changelog
 =========
 
+[v0.3.4](https://github.com/IvanIsCoding/celq/releases/tag/v0.3.4) - 2026-03-07
+------------------------------------------------------------------------
+
+### Fixed
+
+* Fixed `--greppable` so NDJSON input prints all matching results instead of only the last one.
+* Fixed conflicting input-format flags so combinations such as `--from-xml --from-yaml` are rejected by the CLI.
+
+### Miscellaneous
+
+* Bumped the TOML parser (`toml` -> 1.0.6).
+* Bumped the YAML parser (`serde-saphyr` -> 0.0.21).
+
 [v0.3.3](https://github.com/IvanIsCoding/celq/releases/tag/v0.3.1) - 2026-02-05
 ------------------------------------------------------------------------
 
@@ -78,4 +91,4 @@ Changelog
 [v0.1.0](https://github.com/IvanIsCoding/celq/releases/tag/v0.1.0) - 2026-01-04
 ------------------------------------------------------------------------
 
-Initial release with support for JSON and JSON5!
+Initial release with support for JSON and JSON5!

src/cli.rs

@@ -82,7 +82,14 @@ impl std::str::FromStr for Argument {
     ),
     group(
         ArgGroup::new("input_format")
-            .args(&["slurp", "from_json5", "from_toml", "from_yaml", "from_gron"])
+            .args(&[
+                "slurp",
+                "from_json5",
+                "from_toml",
+                "from_yaml",
+                "from_xml",
+                "from_gron",
+            ])
     ),
     group(
         ArgGroup::new("output_style")

src/main.rs

@@ -95,16 +95,12 @@ fn main() -> io::Result<()> {
         Ok(results) => {
             // Print all outputs, unless void mode is enabled
             if !cli.void {
-                if !cli.greppable {
-                    for (output, _) in &results {
+                for (output, _) in &results {
+                    if cli.greppable {
+                        print!("{}", output);
+                    } else {
                         println!("{}", output);
                     }
-                } else {
-                    let last_gron = results
-                        .last()
-                        .map(|(output, _)| output.as_str())
-                        .unwrap_or("");
-                    println!("{}", last_gron);
                 }
             }
 

tests/golden.rs

@@ -63,6 +63,42 @@ fn golden_test(args: &[&str], input: &str, out_ex: &str) -> io::Result<()> {
     Ok(())
 }
 
+#[cfg(all(feature = "from-xml", feature = "from-yaml"))]
+fn golden_test_failure(args: &[&str], input: &str, err_ex: &str) -> io::Result<()> {
+    let mut child = process::Command::new(env!("CARGO_BIN_EXE_celq"))
+        .args(args)
+        .stdin(process::Stdio::piped())
+        .stdout(process::Stdio::piped())
+        .stderr(process::Stdio::piped())
+        .spawn()?;
+
+    use io::Write;
+    {
+        let mut stdin = child.stdin.take().unwrap();
+        stdin.write_all(input.as_bytes())?;
+        drop(stdin);
+    }
+
+    let output = child.wait_with_output()?;
+
+    assert!(
+        !output.status.success(),
+        "Expected failure, but process succeeded with stdout: {}",
+        String::from_utf8_lossy(&output.stdout)
+    );
+
+    let err_act = str::from_utf8(&output.stderr).expect("invalid UTF-8 in stderr");
+    let err_act = err_act.replace('\r', "");
+    assert!(
+        err_act.contains(err_ex),
+        "Expected stderr to contain:\n{}\n---\nActual stderr:\n{}\n---",
+        err_ex,
+        err_act
+    );
+
+    Ok(())
+}
+
 macro_rules! test {
     ($name:ident, $args:expr, $input:expr, $output:expr) => {
         #[test]
@@ -771,6 +807,19 @@ json.person.name = "Bob";
 "#
 );
 
+#[cfg(feature = "greppable")]
+test!(
+    greppable_ndjson_prints_all_results,
+    &["--greppable", "-S", "this"],
+    r#"{"a":1}
+{"b":2}"#,
+    r#"json = {};
+json.a = 1;
+json = {};
+json.b = 2;
+"#
+);
+
 #[cfg(feature = "greppable")]
 test!(
     greppable_json5,
@@ -992,6 +1041,16 @@ fn test_slice_fails_without_extensions() -> io::Result<()> {
     Ok(())
 }
 
+#[cfg(all(feature = "from-xml", feature = "from-yaml"))]
+#[test]
+fn test_conflicting_input_format_flags_fail() -> io::Result<()> {
+    golden_test_failure(
+        &["--from-xml", "--from-yaml", "this"],
+        "<a>1</a>",
+        "the argument '--from-xml' cannot be used with '--from-yaml'",
+    )
+}
+
 // XML tests
 
 // XML with nested elements