excel: better cell error handling

before, we were returning the debug value, which didn't have the '#' prefix to clearly indicate its a cell error - i.e. `#DIV/0!` instead of `Div0`

This improves Excel error handling while I research how to get the "original value" of a cell that's causing the error as discussed in #1682

src/cmd/excel.rs

@@ -671,7 +671,7 @@ pub fn run(argv: &[&str]) -> CliResult<()> {
 
                             record.push_field(&work_date);
                         },
-                        Data::Error(ref e) => record.push_field(&format!("{e:?}")),
+                        Data::Error(ref e) => record.push_field(&format!("{e}")),
                         Data::Bool(ref b) => {
                             record.push_field(if *b { "true" } else { "false" });
                         },

tests/test_excel.rs

@@ -19,6 +19,27 @@ fn excel_open_xls() {
     assert_eq!(got, expected);
 }
 
+#[test]
+fn excel_cellerrors() {
+    let wrk = Workdir::new("excel_cellerrors");
+
+    let xls_file = wrk.load_test_file("excel-xlsx.xlsx");
+
+    let mut cmd = wrk.command("excel");
+    cmd.args(["--sheet", "cellerrors"]).arg(xls_file);
+
+    let got: Vec<Vec<String>> = wrk.read_stdout(&mut cmd);
+    let expected = vec![
+        svec!["col1", "col 2", "column-3"],
+        svec!["1", "-50", "15"],
+        svec!["2", "#DIV/0!", "#NAME?"],
+        svec!["3", "50", "20"],
+        svec!["4", "33.333333333333336", "3"],
+        svec!["5", "25", "4"],
+    ];
+    assert_eq!(got, expected);
+}
+
 #[test]
 fn excel_open_xls_delimiter() {
     let wrk = Workdir::new("excel_open_xls_delimiter");