Add more basic bibtex tests to help avoid regressions

Author: CraftSpider

tests/bibtex.rs

@@ -3,6 +3,7 @@
 
 use std::collections::HashSet;
 use std::default::Default;
+use std::path::PathBuf;
 
 use tectonic::io::{FilesystemIo, IoProvider, IoStack, MemoryIo};
 use tectonic::BibtexEngine;
@@ -16,29 +17,46 @@ use crate::util::{test_path, ExpectedInfo};
 
 struct TestCase {
     stem: String,
+    subdir: Option<String>,
+    test_bbl: bool,
 }
 
 impl TestCase {
-    fn new(stem: &str) -> Self {
+    fn new(
+        stem: &str,
+        subdir: Option<&str>,
+    ) -> Self {
         TestCase {
             stem: stem.to_owned(),
+            subdir: subdir.map(String::from),
+            test_bbl: true,
         }
     }
 
-    fn go(&mut self) {
-        util::set_test_root();
+    fn test_bbl(mut self, test: bool) -> Self {
+        self.test_bbl = test;
+        self
+    }
 
+    fn test_dir(&self) -> PathBuf {
         let mut p = test_path(&["bibtex"]);
+        if let Some(subdir) = &self.subdir {
+            p.push(subdir);
+        }
+        p
+    }
 
-        p.push(&self.stem);
+    fn go(&mut self) {
+        util::set_test_root();
+
+        let mut p = self.test_dir();
 
-        p.set_extension("aux");
-        let auxname = p.file_name().unwrap().to_str().unwrap().to_owned();
+        let auxname = format!("{}.aux", self.stem);
 
         // MemoryIo layer that will accept the outputs.
         let mut mem = MemoryIo::new(true);
 
-        let mut assets = FilesystemIo::new(&test_path(&["bibtex"]), false, false, HashSet::new());
+        let mut assets = FilesystemIo::new(&p, false, false, HashSet::new());
 
         let mut genio = GenuineStdoutIo::new();
 
@@ -55,17 +73,42 @@ impl TestCase {
 
         // Check that outputs match expectations.
 
-        let expected_bbl = ExpectedInfo::read_with_extension(&mut p, "bbl");
-        let expected_blg = ExpectedInfo::read_with_extension(&mut p, "blg");
+        p.push(&self.stem);
 
         let files = mem.files.borrow();
 
-        expected_bbl.test_from_collection(&files);
+        if self.test_bbl {
+            let expected_bbl = ExpectedInfo::read_with_extension(&mut p, "bbl");
+            expected_bbl.test_from_collection(&files);
+        }
+
+        let expected_blg = ExpectedInfo::read_with_extension(&mut p, "blg");
         expected_blg.test_from_collection(&files);
     }
 }
 
 #[test]
 fn single_entry() {
-    TestCase::new("single_entry").go()
+    TestCase::new("single_entry", None).go()
+}
+
+#[test]
+fn test_empty_files() {
+    TestCase::new("empty", Some("empty"))
+        .test_bbl(false)
+        .go()
+}
+
+#[test]
+fn test_mismatched_function() {
+    TestCase::new("function", Some("mismatched_braces"))
+        .test_bbl(false)
+        .go();
+}
+
+#[test]
+fn test_mismatched_expr() {
+    TestCase::new("expr", Some("mismatched_braces"))
+        .test_bbl(false)
+        .go();
 }

tests/bibtex/empty/empty.aux

@@ -0,0 +1,7 @@
+This is BibTeX, Version 0.99d
+Capacity: max_strings=35307, hash_size=35307, hash_prime=30011
+The top-level auxiliary file: empty.aux
+I found no \citation commands---while reading file empty.aux
+I found no \bibdata command---while reading file empty.aux
+I found no \bibstyle command---while reading file empty.aux
+(There were 3 error messages)

tests/bibtex/empty/empty.bbl

@@ -0,0 +1,5 @@
+\relax
+\citation{Mismatched01}
+\bibdata{expr}
+\bibcite{Mismatched01}{1}
+\bibstyle{expr}

tests/bibtex/empty/empty.bib

@@ -0,0 +1,9 @@
+This is BibTeX, Version 0.99d
+Capacity: max_strings=35307, hash_size=35307, hash_prime=30011
+The top-level auxiliary file: expr.aux
+The style file: expr.bst
+"}" can't start a style-file command---line 5 of file expr.bst
+ : 
+ : }
+(Error may have been on previous line)
+(There was 1 error message)