test: use pretty_assertions (#260)

* test: use assert_matches in pretty_assertions

* style: apply automated fixes

---------

Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>

Author: risu729

Cargo.lock

@@ -36,11 +36,10 @@ tracing-subscriber = "0.3.22"
 usage-lib = { version = "2.18.2", features = ["clap", "docs"] }
 
 [dev-dependencies]
-assert_matches = "1.5.0"
 ctor = "0.2.8"
 duct = "1.1.1"
 insta = { version = "1.46.3", features = ["json"] }
-pretty_assertions = "1.4.1"
+pretty_assertions = { version = "1.4.1", features = ["unstable"] }
 rstest = "0.26.1"
 serial_test = "3.4.0"
 tempfile = "3.26.0"

Cargo.toml

@@ -1,3 +1,7 @@
+disallowed-macros = [
+	{ path = "std::assert_eq", reason = "Please use pretty_assertions::assert_eq! for better diff output in tests." },
+	{ path = "std::assert_ne", reason = "Please use pretty_assertions::assert_ne! for better diff output in tests." },
+]
 disallowed-types = [
 	{ path = "eyre::Result", reason = "Please use `crate::Result` instead to ensure consistent error handling across the crate." },
 	{ path = "color_eyre::Result", reason = "Please use `crate::Result` instead to ensure consistent error handling across the crate." },

clippy.toml

@@ -128,6 +128,7 @@ fn quote_keys_for_jsonc(body: &str) -> String {
 mod tests {
 	use super::*;
 	use insta::assert_snapshot;
+	use pretty_assertions::assert_eq;
 	use rstest::rstest;
 
 	#[rstest]

src/cli/init.rs

@@ -121,6 +121,7 @@ pub async fn run() -> Result<()> {
 #[cfg(test)]
 mod tests {
 	use super::*;
+	use pretty_assertions::assert_eq;
 
 	#[test]
 	fn cli_run_subcommand() {

src/cli/mod.rs

@@ -26,8 +26,8 @@ impl Run {
 #[cfg(test)]
 mod tests {
 	use crate::cli::{Cli, Commands};
-	use assert_matches::assert_matches;
 	use clap::Parser as _;
+	use pretty_assertions::{assert_eq, assert_matches};
 
 	#[test]
 	fn run_command() {

src/cli/run.rs

@@ -41,6 +41,7 @@ impl ConfigFormat {
 #[cfg(test)]
 mod tests {
 	use super::*;
+	use pretty_assertions::assert_eq;
 
 	#[test]
 	fn all_formats() {

src/config/format.rs

@@ -201,8 +201,7 @@ fn find_single_config(base_paths_no_ext: &[PathBuf]) -> Result<Option<(PathBuf,
 mod tests {
 	use super::*;
 	use crate::testing::EnvCleanup;
-	use assert_matches::assert_matches;
-	use pretty_assertions::assert_eq;
+	use pretty_assertions::{assert_eq, assert_matches, assert_ne};
 	use rstest::rstest;
 	use serial_test::serial;
 	use std::fs;

src/config/load.rs

@@ -126,7 +126,7 @@ fn resolve_default_key_path() -> Option<PathBuf> {
 #[cfg(test)]
 mod tests {
 	use super::*;
-	use assert_matches::assert_matches;
+	use pretty_assertions::assert_matches;
 	use serial_test::serial;
 	use std::fs;
 

src/ssh/auth.rs

@@ -162,6 +162,7 @@ pub async fn execute_command(
 #[cfg(test)]
 mod tests {
 	use super::*;
+	use pretty_assertions::assert_eq;
 
 	#[test]
 	fn build_command_no_args() {