fix: hash should different when map are different (#136)

Author: SyMind

src/cached_source.rs

@@ -174,7 +174,7 @@ impl<T: Source + Hash + PartialEq + Eq + 'static> Hash for CachedSource<T> {
   fn hash<H: std::hash::Hasher>(&self, state: &mut H) {
     (self.cached_hash.get_or_init(|| {
       let mut hasher = FxHasher::default();
-      hasher.write(self.source().as_bytes());
+      self.inner.hash(&mut hasher);
       hasher.finish()
     }))
     .hash(state);
@@ -208,8 +208,8 @@ mod tests {
   use std::borrow::Borrow;
 
   use crate::{
-    ConcatSource, OriginalSource, RawSource, SourceExt, SourceMapSource,
-    WithoutOriginalOptions,
+    ConcatSource, OriginalSource, RawSource, ReplaceSource, SourceExt,
+    SourceMapSource, WithoutOriginalOptions,
   };
 
   use super::*;
@@ -364,4 +364,27 @@ mod tests {
     source.source();
     assert_eq!(source.buffer(), buf.as_slice());
   }
+
+  #[test]
+  fn hash_should_different_when_map_are_different() {
+    let hash1 = {
+      let mut source =
+        ReplaceSource::new(OriginalSource::new("Hello", "hello.txt").boxed());
+      source.insert(5, " world", None);
+      let cache = CachedSource::new(source);
+      let mut hasher = FxHasher::default();
+      cache.hash(&mut hasher);
+      hasher.finish()
+    };
+
+    let hash2 = {
+      let source = OriginalSource::new("Hello world", "hello.txt").boxed();
+      let cache = CachedSource::new(source);
+      let mut hasher = FxHasher::default();
+      cache.hash(&mut hasher);
+      hasher.finish()
+    };
+
+    assert!(hash1 != hash2);
+  }
 }

src/source.rs

@@ -524,7 +524,7 @@ mod tests {
     RawBufferSource::from("a".as_bytes()).hash(&mut state);
     (&RawSource::from("h") as &dyn Source).hash(&mut state);
     ReplaceSource::new(RawSource::from("i").boxed()).hash(&mut state);
-    assert_eq!(format!("{:x}", state.finish()), "709931db47fa47dc");
+    assert_eq!(format!("{:x}", state.finish()), "f4b280bd9a8d4d3b");
   }
 
   #[test]