Extract HEADER_FIELDS

Author: cruessler

gix-blame/tests/blame.rs

@@ -16,6 +16,24 @@ mod baseline {
     use super::Baseline;
     use gix_blame::BlameEntry;
 
+    // These fields are used by `git` in its porcelain output.
+    const HEADER_FIELDS: [&str; 12] = [
+        // https://github.com/git/git/blob/6258f68c3c1092c901337895c864073dcdea9213/builtin/blame.c#L256-L280
+        "author",
+        "author-mail",
+        "author-time",
+        "author-tz",
+        "committer",
+        "committer-mail",
+        "committer-time",
+        "committer-tz",
+        "summary",
+        "boundary",
+        // https://github.com/git/git/blob/6258f68c3c1092c901337895c864073dcdea9213/builtin/blame.c#L239-L248
+        "previous",
+        "filename",
+    ];
+
     impl<'a> Baseline<'a> {
         pub fn collect(baseline_path: impl AsRef<Path>) -> std::io::Result<Vec<BlameEntry>> {
             let content = std::fs::read(baseline_path)?;
@@ -71,23 +89,7 @@ mod baseline {
                         (line_number_in_final_file - 1)..(line_number_in_final_file + number_of_lines_in_group - 1);
                     assert!(ranges.is_none(), "should not overwrite existing ranges");
                     ranges = Some((blame_range, original_range));
-                } else if ![
-                    "author",
-                    "author-mail",
-                    "author-time",
-                    "author-tz",
-                    "committer",
-                    "committer-mail",
-                    "committer-time",
-                    "committer-tz",
-                    "summary",
-                    "previous",
-                    "filename",
-                    "boundary",
-                ]
-                .contains(&fields[0])
-                    && ObjectId::from_hex(fields[0].as_bytes()).is_err()
-                {
+                } else if !HEADER_FIELDS.contains(&fields[0]) && ObjectId::from_hex(fields[0].as_bytes()).is_err() {
                     panic!("unexpected line: '{:?}'", line.as_bstr());
                 }
             }