feat: `CommitRef::message_trailers()` as shortcut… (#198)

…which also saves parsing time.

Sebastian Thiel committed 5 years ago

Commit 53243911 parent c3b01614 files · +36 −12Browse files

Showing the changes to git-object/tests/immutable/commit (2 of 4 files). Show all

Unified · Split

2 files

 use git_actor::{Sign, SignatureRef, Time};
-use git_object::{bstr::ByteSlice, CommitRef};
+use git_object::{bstr::ByteSlice, commit::message::body::TrailerRef, CommitRef};
 use smallvec::SmallVec;
 use crate::immutable::{
         )
     );
     assert_eq!(
-        commit.summary(),
+        commit.message_summary(),
         message.summary(),
         "both summaries are the same, but the commit one does less parsing"
     );
             .as_bstr(),
         "body doesn't contain footer"
     );
+    assert_eq!(
+        body.trailers().collect::<Vec<_>>(),
+        vec![
+            TrailerRef {
+                token: "Signed-off-by".into(),
+                value: "Sebastian Thiel <sebastian.thiel@icloud.com>".into()
+            },
+            TrailerRef {
+                token: "Signed-off-by".into(),
+                value: "Kim Altintop <kim@eagain.st>".into()
+            }
+        ]
+    );
+    assert_eq!(
+        body.trailers().collect::<Vec<_>>(),
+        commit.message_trailers().collect::<Vec<_>>(),
+        "messages trailers are accessible on commit level and yield the same result"
+    );
     Ok(())
 }

@@ -250,7 +250,7 @@ mod summary { · 249 unmodified lines
250	message: input.as_bstr(),	250	message: input.as_bstr(),
251	extra_headers: vec![]	251	extra_headers: vec![]
252	}	252	}
253	.summary(),	253	.message_summary(),
254	summary,	254	summary,
255	"both versions create the same result"	255	"both versions create the same result"
256	);	256	);