chore(formatter): prettier test diff output via similar-asserts (#3046

) Co-authored-by: Tom French <15848336+TomAFrench@users.noreply.github.com>
noir-lang · Oct 9, 2023 · 0bc2e4d · 0bc2e4d
1 parent 7e68976
commit 0bc2e4d
Show file tree

Hide file tree

Showing 4 changed files with 138 additions and 14 deletions.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/tooling/nargo_fmt/Cargo.toml b/tooling/nargo_fmt/Cargo.toml
@@ -11,3 +11,6 @@ noirc_frontend.workspace = true
 serde.workspace = true
 toml.workspace = true
 thiserror.workspace = true
+
+[dev-dependencies]
+similar-asserts = "1.5.0"
diff --git a/tooling/nargo_fmt/build.rs b/tooling/nargo_fmt/build.rs
@@ -59,7 +59,7 @@ fn format_{test_name}() {{
     let fmt_text = nargo_fmt::format(&input, parsed_module, &config);
 
 
-    assert_eq!(fmt_text, expected_output);
+    similar_asserts::assert_eq!(fmt_text, expected_output);
 }}
             "##
         )

diff --git a/tooling/nargo_fmt/src/lib.rs b/tooling/nargo_fmt/src/lib.rs
@@ -1,8 +1,7 @@
 #![forbid(unsafe_code)]
-#![warn(unused_crate_dependencies, unused_extern_crates)]
 #![warn(unreachable_pub)]
 #![warn(clippy::semicolon_if_nothing_returned)]
-#![warn(unused_qualifications, clippy::use_self)]
+#![cfg_attr(not(test), warn(unused_crate_dependencies, unused_extern_crates))]
 
 /// A Rust code formatting utility designed to manage and format untouched fragments of source code,
 /// including comments, whitespace, and other characters. While the module doesn't directly address comments,
-Original file line number
+Diff line change
@@ Expand Up / @@ -59,7 +59,7 @@ fn format_{test_name}() {{ @@
         let fmt_text = nargo_fmt::format(&input, parsed_module, &config);
-        assert_eq!(fmt_text, expected_output);
+        similar_asserts::assert_eq!(fmt_text, expected_output);
     }}
                 "##
             )
@@ Expand Down @@