Merge pull request #18 from alexwlchan/fix-bug

Fix a bug when finding the dominant colour of some animated GIFs

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## v1.1.3 - 2022-10-17
+
+Fix a bug when finding the dominant colour of some animated GIFs.
+
 ## v1.1.2 - 2022-10-16
 
 Add support for TIFF images.

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "dominant_colours"
-version = "1.1.2"
+version = "1.1.3"
 edition = "2018"
 
 [dependencies]

src/get_bytes.rs

@@ -101,3 +101,18 @@ pub fn get_bytes_for_gif(path: &str) -> Vec<u8> {
         .flatten()
         .collect()
 }
+
+#[cfg(test)]
+mod test {
+    use crate::get_bytes;
+
+    // This image comes from https://stacks.wellcomecollection.org/peering-through-mri-scans-of-fruit-and-veg-part-1-a2e8b07bde6f
+    //
+    // I don't remember how I got these images, but for some reason they
+    // caused v1.1.2 to fall over.  This is a test that they can still be
+    // processed correctly.
+    #[test]
+    fn it_gets_bytes_for_mri_fruit() {
+        get_bytes::get_bytes_for_gif("./src/tests/garlic.gif");
+    }
+}