feat: `CommitRefIter::(author|committer)()`, better usability (#364)

The methods returning an iterator are now consuming, which allows them to be nested by callers.

Sebastian Thiel committed 4 years ago

Commit c51a9251 parent 91065cd2 files · +41 −20Browse files

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

Unified · Split

git-object/tests/immutable/commit/iter.rs+23−20

 #[test]
 fn signed_with_encoding() -> crate::Result {
+    let input = fixture_bytes("commit", "signed-with-encoding.txt");
+    let mut iter = CommitRefIter::from_bytes(&input);
     assert_eq!(
-        CommitRefIter::from_bytes(&fixture_bytes("commit", "signed-with-encoding.txt"))
+        iter.collect::<Result<Vec<_>, _>>()?,
-            .collect::<Result<Vec<_>, _>>()?,
         vec![
             Token::Tree {
                 id: hex_to_id("1973afa74d87b2bb73fa884aaaa8752aec43ea88")
             Token::Message(b"encoding & sig".as_bstr()),
         ]
     );
+    assert_eq!(iter.author(), Some(signature(1592448995)));
+    assert_eq!(iter.committer(), Some(signature(1592449083)));
     Ok(())
 }
 #[test]
 fn mergetag() -> crate::Result {
+    let input = fixture_bytes("commit", "mergetag.txt");
+    let mut iter = CommitRefIter::from_bytes(&input);
     assert_eq!(
-        CommitRefIter::from_bytes(&fixture_bytes("commit", "mergetag.txt")).collect::<Result<Vec<_>, _>>()?,
+        iter.collect::<Result<Vec<_>, _>>()?,
         vec![
             Token::Tree {
                 id: hex_to_id("1c61918031bf2c7fab9e17dde3c52a6a9884fcb5")
                 signature: linus_signature(1591996221)
             },
             Token::ExtraHeader((b"mergetag".as_bstr(), MERGE_TAG.as_bytes().as_bstr().into())),
-            Token::Message(LONG_MESSAGE.as_bytes().as_bstr()),
+            Token::Message(LONG_MESSAGE.into()),
         ]
     );
     assert_eq!(
-        CommitRefIter::from_bytes(&fixture_bytes("commit", "mergetag.txt"))
+        iter.parent_ids().collect::<Vec<_>>(),
-            .parent_ids()
-            .collect::<Vec<_>>(),
         vec![
             hex_to_id("44ebe016df3aad96e3be8f95ec52397728dd7701"),
             hex_to_id("8d485da0ddee79d0e6713405694253d401e41b93")
         ]
     );
+    assert_eq!(iter.message(), Some(LONG_MESSAGE.into()));
     Ok(())
 }
     #[test]
     fn tree_id() -> crate::Result {
+        let input = fixture_bytes("commit", "unsigned.txt");
+        let iter = CommitRefIter::from_bytes(&input);
         assert_eq!(
-            CommitRefIter::from_bytes(&fixture_bytes("commit", "unsigned.txt")).tree_id(),
+            iter.clone().tree_id(),
             Some(hex_to_id("1b2dfb4ac5e42080b682fc676e9738c94ce6d54d"))
         );
         assert_eq!(
-            CommitRefIter::from_bytes(&fixture_bytes("commit", "unsigned.txt"))
+            iter.signatures().collect::<Vec<_>>(),
-                .signatures()
-                .collect::<Vec<_>>(),
             vec![signature(1592437401), signature(1592437401)]
         );
-        assert_eq!(
+        assert_eq!(iter.parent_ids().count(), 0);
-            CommitRefIter::from_bytes(&fixture_bytes("commit", "unsigned.txt"))
-                .parent_ids()
-                .count(),
-            0
-        );
         Ok(())
     }
     #[test]
     fn signatures() -> crate::Result {
+        let input = fixture_bytes("commit", "unsigned.txt");
+        let mut iter = CommitRefIter::from_bytes(&input);
         assert_eq!(
-            CommitRefIter::from_bytes(&fixture_bytes("commit", "unsigned.txt"))
+            iter.signatures().collect::<Vec<_>>(),
-                .signatures()
-                .collect::<Vec<_>>(),
             vec![signature(1592437401), signature(1592437401)]
         );
+        assert_eq!(iter.author(), Some(signature(1592437401)));
+        assert_eq!(iter.committer(), Some(signature(1592437401)));
+        assert_eq!(iter.committer(), None, "it's consuming");
         Ok(())
     }
 }