tests: Declare manifests and extra files in test code (#44)

* tests/autolib: Declare manifests and extra files in test code

This makes the tests a little easier to comprehend, since we don't have to crawl through the filesystem structure to understand the test setup.

* tests: Move `prepare()` fn to `utils` module

This will allow us to share this function with other tests

* tests/parse: Declare manifests and extra files in test code

This makes the tests a little easier to comprehend, since we don't have to crawl through the filesystem structure to understand the test setup.

Cargo.toml

@@ -22,3 +22,4 @@ toml = { version = "0.8", features = ["preserve_order"] }
 
 [dev-dependencies]
 insta = "1.39.0"
+tempfile = "3.10.1"

tests/autolib.rs

@@ -1,14 +1,24 @@
 use cargo_manifest::Manifest;
 
+mod utils;
+
+const BASIC_MANIFEST: &str = r#"
+[package]
+name = "auto-lib"
+version = "0.1.0"
+"#;
+
 #[test]
 fn test_bin() {
-    let m = Manifest::from_path("tests/autolib/bin/Cargo.toml").unwrap();
+    let tempdir = utils::prepare(BASIC_MANIFEST, vec!["src/main.rs"]);
+    let m = Manifest::from_path(tempdir.path().join("Cargo.toml")).unwrap();
     assert!(m.lib.is_none());
 }
 
 #[test]
 fn test_lib_rs() {
-    let m = Manifest::from_path("tests/autolib/lib_rs/Cargo.toml").unwrap();
+    let tempdir = utils::prepare(BASIC_MANIFEST, vec!["src/lib.rs"]);
+    let m = Manifest::from_path(tempdir.path().join("Cargo.toml")).unwrap();
 
     let lib = m.lib.unwrap();
     assert_eq!(lib.path.as_deref(), Some("src/lib.rs"));
@@ -19,7 +29,16 @@ fn test_lib_rs() {
 
 #[test]
 fn test_name_override() {
-    let m = Manifest::from_path("tests/autolib/name_override/Cargo.toml").unwrap();
+    let manifest = r#"
+    [package]
+    name = "auto-lib"
+    version = "0.1.0"
+
+    [lib]
+    name = "foo"
+    "#;
+    let tempdir = utils::prepare(manifest, vec!["src/lib.rs"]);
+    let m = Manifest::from_path(tempdir.path().join("Cargo.toml")).unwrap();
 
     let lib = m.lib.unwrap();
     assert_eq!(lib.path.as_deref(), Some("src/lib.rs"));
@@ -30,7 +49,16 @@ fn test_name_override() {
 
 #[test]
 fn test_path_override() {
-    let m = Manifest::from_path("tests/autolib/path_override/Cargo.toml").unwrap();
+    let manifest = r#"
+    [package]
+    name = "auto-lib"
+    version = "0.1.0"
+
+    [lib]
+    path = "src/foo.rs"
+    "#;
+    let tempdir = utils::prepare(manifest, vec!["src/foo.rs", "src/lib.rs"]);
+    let m = Manifest::from_path(tempdir.path().join("Cargo.toml")).unwrap();
 
     let lib = m.lib.unwrap();
     assert_eq!(lib.path.as_deref(), Some("src/foo.rs"));
@@ -41,7 +69,18 @@ fn test_path_override() {
 
 #[test]
 fn test_other_override() {
-    let m = Manifest::from_path("tests/autolib/other_override/Cargo.toml").unwrap();
+    let manifest = r#"
+    [package]
+    name = "auto-lib"
+    version = "0.1.0"
+    edition = "2018"
+
+    [lib]
+    proc-macro = true
+    test = false
+    "#;
+    let tempdir = utils::prepare(manifest, vec!["src/lib.rs"]);
+    let m = Manifest::from_path(tempdir.path().join("Cargo.toml")).unwrap();
 
     let lib = m.lib.unwrap();
     assert!(!lib.test);

tests/parse.rs

@@ -3,6 +3,8 @@ use cargo_manifest::{Manifest, MaybeInherited};
 use std::fs::read;
 use std::str::FromStr;
 
+mod utils;
+
 #[test]
 fn own() {
     let m = Manifest::from_slice(&read("Cargo.toml").unwrap()).unwrap();
@@ -32,19 +34,47 @@ fn opt_version() {
 
 #[test]
 fn autobin() {
-    let m = Manifest::from_path("tests/autobin/Cargo.toml").expect("load autobin");
+    let manifest = r#"
+    [package]
+    name = "auto-bin"
+    version = "0.1.0"
+    publish = false
+    edition = "2018"
+
+    [badges]
+    travis-ci = { repository = "…" }
+    "#;
+    let tempdir = utils::prepare(manifest, vec!["src/main.rs"]);
+    let m = Manifest::from_path(tempdir.path().join("Cargo.toml")).unwrap();
     insta::assert_debug_snapshot!(m);
 }
 
 #[test]
 fn autobuild() {
-    let m = Manifest::from_path("tests/autobuild/Cargo.toml").expect("load autobuild");
+    let manifest = r#"
+    [package]
+    name = "buildrstest"
+    version = "0.2.0"
+    "#;
+    let tempdir = utils::prepare(manifest, vec!["build.rs"]);
+    let m = Manifest::from_path(tempdir.path().join("Cargo.toml")).unwrap();
     insta::assert_debug_snapshot!(m);
 }
 
+/// Checks that explicit `build` key in Cargo.toml has precedence over auto-detected build.rs file.
 #[test]
 fn metadata() {
-    let m = Manifest::from_path("tests/metadata/Cargo.toml").expect("load metadata");
+    let manifest = r#"
+    [package]
+    name = "metadata"
+    version = "0.1.0"
+    build = "foobar.rs"
+
+    [lib]
+    path = "lib.rs"
+    "#;
+    let tempdir = utils::prepare(manifest, vec!["build.rs", "lib.rs"]);
+    let m = Manifest::from_path(tempdir.path().join("Cargo.toml")).unwrap();
     insta::assert_debug_snapshot!(m);
 }
 

tests/utils.rs

@@ -0,0 +1,17 @@
+use tempfile::TempDir;
+
+pub fn prepare(manifest: &str, extra_files: Vec<&str>) -> TempDir {
+    let tempdir = tempfile::tempdir().unwrap();
+
+    // Create `Cargo.toml` manifest file
+    std::fs::write(tempdir.path().join("Cargo.toml"), manifest).unwrap();
+
+    // Create extra files
+    for file in extra_files {
+        let path = tempdir.path().join(file);
+        std::fs::create_dir_all(path.parent().unwrap()).unwrap();
+        std::fs::write(path, "").unwrap();
+    }
+
+    tempdir
+}