config: Make doctest options optional when disabled (#3)

Author: rdmsr

example/cppdoc.toml

@@ -17,5 +17,3 @@ base_url = "/cppdoc"
 
 [doctests]
 enable = false 
-run = true
-compiler_invocation = ["clang++", "{file}", "-o", "{out}", "-Iinclude", "-std=c++20"]

src/config.rs

@@ -39,8 +39,8 @@ pub struct Output {
 #[derive(Deserialize, Serialize, Clone, Debug)]
 pub struct Doctest {
     pub enable: bool,
-    pub run: bool,
-    pub compiler_invocation: Vec<String>,
+    pub run: Option<bool>,
+    pub compiler_invocation: Option<Vec<String>>,
 }
 
 impl Config {

src/doctest.rs

@@ -61,7 +61,9 @@ impl Doctest {
     }
 
     pub fn compile(&self, config: &config::Doctest) -> std::path::PathBuf {
-        let compiler_invocation = config.compiler_invocation.join(" ");
+        // This is fine, this should never panic
+        let compiler_invocation = config.compiler_invocation.clone().unwrap();
+        let compiler_invocation = compiler_invocation.join(" ");
 
         let mut in_file = tempfile::Builder::new().suffix(".cpp").tempfile().unwrap();
 

src/main.rs

@@ -159,10 +159,20 @@ fn main() {
                         ProgressStyle::with_template("Running doctest {pos}/{len}").unwrap(),
                     );
 
+                    if let None = doctest_conf.run {
+                        report_error("Doctests enabled but no run option specified");
+                        std::process::exit(1);
+                    }
+
+                    if let None = doctest_conf.compiler_invocation {
+                        report_error("Doctests enabled but no compiler invocation specified");
+                        std::process::exit(1);
+                    }
+
                     for doc in doctests {
                         let out = doc.compile(doctest_conf);
 
-                        if doctest_conf.run {
+                        if doctest_conf.run.unwrap() {
                             doc.run(out);
                         }