more output_path() test in capnpc-test, and make it actually test something interesting

Author: dwrensha

capnpc/src/lib.rs

@@ -324,17 +324,7 @@ impl CompilerCommand {
 #[test]
 #[cfg_attr(miri, ignore)]
 fn compiler_command_new_no_out_dir() {
+    std::env::remove_var("OUT_DIR");
     let error = CompilerCommand::new().run().unwrap_err().description;
     assert!(error.starts_with("Could not access `OUT_DIR` environment variable"));
 }
-
-#[test]
-#[cfg_attr(miri, ignore)]
-fn compiler_command_with_output_path_no_out_dir() {
-    let error = CompilerCommand::new()
-        .output_path("foo")
-        .run()
-        .unwrap_err()
-        .description;
-    assert!(error.starts_with("Error while trying to execute `capnp compile`"));
-}

capnpc/test/build.rs

@@ -22,4 +22,15 @@ fn main() {
         ])
         .run()
         .expect("compiling schema");
+
+    let mut output_path =
+        std::path::PathBuf::from(std::env::var("OUT_DIR").expect("OUT_DIR env var is not set"));
+
+    // `capnp compile` will create this directory
+    output_path.push("inner-output-path");
+    capnpc::CompilerCommand::new()
+        .file("test-output-path.capnp")
+        .output_path(output_path)
+        .run()
+        .expect("compiling schema")
 }

capnpc/test/test-output-path.capnp

@@ -0,0 +1,9 @@
+# Schema to use to test `CompilerCommand::output_path()`.
+
+@0xbc9d501a1b13e0e9;
+
+struct Foo {}
+
+struct Bar {
+    foo @0 : Foo;
+}

capnpc/test/test.rs

@@ -71,6 +71,13 @@ pub mod test_in_src_prefix_dir_capnp {
     include!(concat!(env!("OUT_DIR"), "/test_in_src_prefix_dir_capnp.rs"));
 }
 
+pub mod test_output_path_capnp {
+    include!(concat!(
+        env!("OUT_DIR"),
+        "/inner-output-path/test_output_path_capnp.rs"
+    ));
+}
+
 #[cfg(test)]
 mod test_util;