Tests: Use test-log in TagLib tests

Author: Serial-ATA

lofty/tests/taglib/test_aiff.rs

@@ -10,7 +10,7 @@ use lofty::iff::aiff::{AiffCompressionType, AiffFile};
 use lofty::probe::Probe;
 use lofty::tag::{Accessor, TagType};
 
-#[test]
+#[test_log::test]
 fn test_aiff_properties() {
 	let file = get_file::<AiffFile>("tests/taglib/data/empty.aiff");
 
@@ -26,7 +26,7 @@ fn test_aiff_properties() {
 	assert!(properties.compression_type().is_none());
 }
 
-#[test]
+#[test_log::test]
 fn test_aifc_properties() {
 	let file = get_file::<AiffFile>("tests/taglib/data/alaw.aifc");
 
@@ -55,7 +55,7 @@ fn test_aifc_properties() {
 	);
 }
 
-#[test]
+#[test_log::test]
 fn test_save_id3v2() {
 	let mut file = temp_file!("tests/taglib/data/empty.aiff");
 
@@ -95,19 +95,19 @@ fn test_save_id3v2() {
 	}
 }
 
-#[test]
+#[test_log::test]
 #[ignore]
 fn test_save_id3v23() {
 	todo!("Support writing ID3v2.3 tags")
 }
 
-#[test]
+#[test_log::test]
 #[ignore]
 fn test_duplicate_id3v2() {
 	// Marker test, Lofty will overwrite values in the original tag with any new values it finds in the next tag.
 }
 
-#[test]
+#[test_log::test]
 fn test_fuzzed_file1() {
 	assert_eq!(
 		Probe::open("tests/taglib/data/segfault.aif")
@@ -119,7 +119,7 @@ fn test_fuzzed_file1() {
 	);
 }
 
-#[test]
+#[test_log::test]
 #[ignore]
 fn test_fuzzed_file2() {
 	// Marker test, this file doesn't even have a valid signature. No idea how TagLib manages to read it.

lofty/tests/taglib/test_ape.rs

@@ -25,22 +25,22 @@ fn test_399(path: &str) {
 	assert_eq!(properties.version(), 3990)
 }
 
-#[test]
+#[test_log::test]
 fn test_properties_399() {
 	test_399("tests/taglib/data/mac-399.ape")
 }
 
-#[test]
+#[test_log::test]
 fn test_properties_399_tagged() {
 	test_399("tests/taglib/data/mac-399-tagged.ape")
 }
 
-#[test]
+#[test_log::test]
 fn test_properties_399_id3v2() {
 	test_399("tests/taglib/data/mac-399-id3v2.ape")
 }
 
-#[test]
+#[test_log::test]
 fn test_properties_396() {
 	let f = get_file::<ApeFile>("tests/taglib/data/mac-396.ape");
 	let properties = f.properties();
@@ -55,7 +55,7 @@ fn test_properties_396() {
 	assert_eq!(properties.version(), 3960)
 }
 
-#[test]
+#[test_log::test]
 fn test_properties_390() {
 	let f = get_file::<ApeFile>("tests/taglib/data/mac-390-hdr.ape");
 	let properties = f.properties();
@@ -70,7 +70,7 @@ fn test_properties_390() {
 	assert_eq!(properties.version(), 3900)
 }
 
-#[test]
+#[test_log::test]
 fn test_fuzzed_file_1() {
 	assert_eq!(
 		Probe::open("tests/taglib/data/longloop.ape")
@@ -82,7 +82,7 @@ fn test_fuzzed_file_1() {
 	);
 }
 
-#[test]
+#[test_log::test]
 fn test_fuzzed_file_2() {
 	assert_eq!(
 		Probe::open("tests/taglib/data/zerodiv.ape")
@@ -94,7 +94,7 @@ fn test_fuzzed_file_2() {
 	);
 }
 
-#[test]
+#[test_log::test]
 fn test_strip_and_properties() {
 	let mut file = temp_file!("tests/taglib/data/mac-399.ape");
 	{
@@ -127,7 +127,7 @@ fn test_strip_and_properties() {
 	}
 }
 
-#[test]
+#[test_log::test]
 fn test_properties() {
 	let mut tag = ApeTag::default();
 	tag.insert(
@@ -356,7 +356,7 @@ fn test_properties() {
 	}
 }
 
-#[test]
+#[test_log::test]
 fn test_repeated_save() {
 	let mut file = temp_file!("tests/taglib/data/mac-399.ape");
 	{

lofty/tests/taglib/test_apetag.rs

@@ -8,7 +8,7 @@ use lofty::file::AudioFile;
 use lofty::musepack::MpcFile;
 use lofty::tag::{Accessor, ItemValue, TagExt};
 
-#[test]
+#[test_log::test]
 fn test_is_empty() {
 	let mut tag = ApeTag::default();
 	assert!(tag.is_empty());
@@ -22,49 +22,55 @@ fn test_is_empty() {
 	assert!(!tag.is_empty());
 }
 
-#[test]
+#[test_log::test]
 fn test_is_empty_2() {
 	let mut tag = ApeTag::default();
 	assert!(tag.is_empty());
 	tag.set_artist(String::from("Mike Oldfield"));
 	assert!(!tag.is_empty());
 }
 
-#[test]
+#[test_log::test]
 #[ignore]
 fn test_property_interface_1() {
 	// Marker test, Lofty does not replicate the TagLib property API
 }
 
-#[test]
+#[test_log::test]
 #[ignore]
 fn test_property_interface_2() {
 	// Marker test, Lofty does not replicate the TagLib property API
 }
 
-#[test]
+#[test_log::test]
 fn test_invalid_keys() {
 	static INVALID_KEY_ONE_CHARACTER: &str = "A";
 	static INVALID_KEY_FORBIDDEN_STRING: &str = "MP+";
 	static INVALID_KEY_UNICODE: &str = "\x1234\x3456";
 	static VALID_KEY_SPACE_AND_TILDE: &str = "A B~C";
 	static VALID_KEY_NORMAL_ONE: &str = "ARTIST";
 
-	assert!(ApeItem::new(
-		String::from(INVALID_KEY_ONE_CHARACTER),
-		ItemValue::Text(String::from("invalid key: one character"))
-	)
-	.is_err());
-	assert!(ApeItem::new(
-		String::from(INVALID_KEY_FORBIDDEN_STRING),
-		ItemValue::Text(String::from("invalid key: forbidden string"))
-	)
-	.is_err());
-	assert!(ApeItem::new(
-		String::from(INVALID_KEY_UNICODE),
-		ItemValue::Text(String::from("invalid key: Unicode"))
-	)
-	.is_err());
+	assert!(
+		ApeItem::new(
+			String::from(INVALID_KEY_ONE_CHARACTER),
+			ItemValue::Text(String::from("invalid key: one character"))
+		)
+		.is_err()
+	);
+	assert!(
+		ApeItem::new(
+			String::from(INVALID_KEY_FORBIDDEN_STRING),
+			ItemValue::Text(String::from("invalid key: forbidden string"))
+		)
+		.is_err()
+	);
+	assert!(
+		ApeItem::new(
+			String::from(INVALID_KEY_UNICODE),
+			ItemValue::Text(String::from("invalid key: Unicode"))
+		)
+		.is_err()
+	);
 
 	let valid_space_and_tilde = ApeItem::new(
 		String::from(VALID_KEY_SPACE_AND_TILDE),
@@ -84,15 +90,15 @@ fn test_invalid_keys() {
 	assert_eq!(tag.len(), 2);
 }
 
-#[test]
+#[test_log::test]
 #[ignore]
 fn test_text_binary() {
 	// Marker test, this is useless as Lofty does not have a similar API that the test is based upon:
 	// https://github.com/taglib/taglib/blob/a31356e330674640a07bef7d71d08242cae8e9bf/tests/test_apetag.cpp#L153
 }
 
 // TODO: Does not work! We fall for this collision.
-#[test]
+#[test_log::test]
 #[ignore]
 fn test_id3v1_collision() {
 	let mut file = temp_file!("tests/taglib/data/no-tags.mpc");