Merge pull request #568 from Enselic/insta

tests: Use `insta` instead of `expect-test` for public API test
trishume · Jan 15, 2025 · 55c8be8 · 55c8be8
2 parents f4bbad0 + c060ca2
commit 55c8be8
Show file tree

Hide file tree

Showing 4 changed files with 43 additions and 19 deletions.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/Cargo.toml b/Cargo.toml
@@ -43,7 +43,7 @@ pretty_assertions = "0.6"
 rustup-toolchain = "0.1.5"
 rustdoc-json = "0.8.8"
 public-api = "0.33.1"
-expect-test = "1.4.1"
+insta = "1.42.0"
 
 [features]
 

diff --git a/tests/public_api.rs b/tests/public_api.rs
@@ -24,5 +24,5 @@ fn public_api() {
         .unwrap();
 
     // Assert that the public API looks correct
-    expect_test::expect_file!["public-api.txt"].assert_eq(&public_api.to_string());
+    insta::assert_snapshot!(public_api);
 }
diff --git a/tests/public-api.txt → tests/snapshots/public_api__public_api.snap b/tests/public-api.txt → tests/snapshots/public_api__public_api.snap
@@ -1,3 +1,7 @@
+---
+source: tests/public_api.rs
+expression: public_api
+---
 pub mod syntect
 pub mod syntect::dumps
 pub fn syntect::dumps::dump_binary<T: serde::ser::Serialize>(o: &T) -> alloc::vec::Vec<u8>