From fd45d1454711317d881e542f233ca2ad7522109e Mon Sep 17 00:00:00 2001
From: Serial <69764315+Serial-ATA@users.noreply.github.com>
Date: Sat, 14 Jan 2023 20:55:03 -0500
Subject: [PATCH] Tests: Add TagLib Speex tests

---
 tests/taglib/main.rs       |  1 +
 tests/taglib/test_speex.rs | 75 ++++++++++++++++++++++++++++++++++++++
 2 files changed, 76 insertions(+)
 create mode 100644 tests/taglib/test_speex.rs

diff --git a/tests/taglib/main.rs b/tests/taglib/main.rs
index 6b5eb5821..dd456d239 100644
--- a/tests/taglib/main.rs
+++ b/tests/taglib/main.rs
@@ -8,3 +8,4 @@ mod test_flac;
 mod test_flacpicture;
 mod test_id3v1;
 mod test_info;
+mod test_speex;
diff --git a/tests/taglib/test_speex.rs b/tests/taglib/test_speex.rs
new file mode 100644
index 000000000..dc81bc369
--- /dev/null
+++ b/tests/taglib/test_speex.rs
@@ -0,0 +1,75 @@
+use crate::temp_file;
+
+use std::io::Seek;
+
+use lofty::ogg::{SpeexFile, VorbisComments};
+use lofty::{Accessor, AudioFile, ParseOptions};
+
+#[test]
+#[ignore]
+fn test_audio_properties() {
+	let mut file = temp_file!("tests/taglib/data/empty.spx");
+	let f = SpeexFile::read_from(&mut file, ParseOptions::new()).unwrap();
+
+	assert_eq!(f.properties().duration().as_secs(), 3);
+	// TODO: We report 3684, we're off by one
+	assert_eq!(f.properties().duration().as_millis(), 3685);
+	// TODO: We report zero, we aren't properly calculating bitrates for Speex
+	assert_eq!(f.properties().audio_bitrate(), 53);
+	assert_eq!(f.properties().nominal_bitrate(), -1);
+	assert_eq!(f.properties().channels(), 2);
+	assert_eq!(f.properties().sample_rate(), 44100);
+}
+
+#[test]
+#[ignore] // TODO: This test doesn't work, it's very specific with file/packet sizes. Have to determine whether or not to even keep this one.
+fn test_split_packets() {
+	let mut file = temp_file!("tests/taglib/data/empty.spx");
+
+	let text = String::from_utf8(vec![b'X'; 128 * 1024]).unwrap();
+
+	{
+		let f = SpeexFile::read_from(&mut file, ParseOptions::new()).unwrap();
+		file.rewind().unwrap();
+
+		let mut tag = VorbisComments::default();
+		tag.set_title(text.clone());
+		f.save_to(&mut file).unwrap();
+	}
+	file.rewind().unwrap();
+	{
+		let mut f = SpeexFile::read_from(&mut file, ParseOptions::new()).unwrap();
+		file.rewind().unwrap();
+
+		assert_eq!(file.metadata().unwrap().len(), 156330);
+		assert_eq!(f.vorbis_comments().title().as_deref(), Some(text.as_str()));
+
+		// NOTE: TagLib exposes the packets and page headers through `Speex::File`.
+		//       Lofty does not keep this information around, so we just double check with `ogg_pager`.
+		let packets = ogg_pager::Packets::read(&mut file).unwrap();
+		assert_eq!(packets.get(0).unwrap().len(), 80);
+		assert_eq!(packets.get(1).unwrap().len(), 131116);
+		assert_eq!(packets.get(2).unwrap().len(), 93);
+		assert_eq!(packets.get(3).unwrap().len(), 93);
+
+		assert_eq!(f.properties().duration().as_millis(), 3685);
+
+		f.vorbis_comments_mut().set_title(String::from("ABCDE"));
+		file.rewind().unwrap();
+		f.save_to(&mut file).unwrap();
+	}
+	file.rewind().unwrap();
+	{
+		let f = SpeexFile::read_from(&mut file, ParseOptions::new()).unwrap();
+		assert_eq!(file.metadata().unwrap().len(), 24317);
+		assert_eq!(f.vorbis_comments().title().as_deref(), Some("ABCDE"));
+
+		let packets = ogg_pager::Packets::read(&mut file).unwrap();
+		assert_eq!(packets.get(0).unwrap().len(), 80);
+		assert_eq!(packets.get(1).unwrap().len(), 49);
+		assert_eq!(packets.get(2).unwrap().len(), 93);
+		assert_eq!(packets.get(3).unwrap().len(), 93);
+
+		assert_eq!(f.properties().duration().as_millis(), 3685);
+	}
+}