move creating test project to lib

kondo-lib/src/lib.rs

@@ -3,7 +3,9 @@ mod lib_test;
 use std::{
     borrow::Cow,
     error::{self, Error},
-    fs, path,
+    fs,
+    io::Write,
+    path,
     time::SystemTime,
 };
 
@@ -481,6 +483,33 @@ pub fn path_canonicalise(
         Ok(base.join(tail).canonicalize()?)
     }
 }
+// Given a name, create a new simulated python project in a safe to delete directry
+pub fn create_fake_python_project(name: String) -> tempfile::TempDir {
+    // Make a new project in a temporary directory
+    let tmp_dir = tempfile::tempdir().unwrap();
+
+    // make a new root in the tmp dir
+    let project_dir = tmp_dir.path().join(name);
+    std::fs::create_dir(&project_dir).unwrap();
+
+    // Must have a directory to hold the project.
+    let cache_dir = project_dir.join("__pycache__");
+    std::fs::create_dir(&cache_dir).unwrap();
+
+    // Must have data in the cache to delete
+    let mut data_file = std::fs::File::create(cache_dir.join("cache.data")).unwrap();
+    data_file.write_all(b"#oodles of cache')\n").unwrap();
+    let mut data_file_b = std::fs::File::create(cache_dir.join("other.cache")).unwrap();
+    data_file_b.write_all(b"#oodles of cache')\n").unwrap();
+
+    // and a file of type .py to signal we're a python project
+    let mut python_file = std::fs::File::create(project_dir.join("main.py")).unwrap();
+    python_file
+        .write_all(b"#!/bin/python\n\nprint('Hello, world!')\n")
+        .unwrap();
+
+    tmp_dir
+}
 
 #[cfg(test)]
 mod tests {

kondo-lib/src/lib_test.rs

@@ -4,7 +4,7 @@ mod test {
     use std::path::PathBuf;
     use tempfile;
 
-    use crate::{ScanOptions, Project, ProjectType};
+    use crate::{Project, ProjectType, ScanOptions};
 
     #[test]
     fn test_clean() {
@@ -49,7 +49,6 @@ mod test {
         tempdir.close().unwrap();
     }
 
-
     // copy the test_data dir to a ephemeral temp dir for destructive testing
     fn get_copy_of_test_data_as_temp_dir() -> tempfile::TempDir {
         extern crate fs_extra;
@@ -68,4 +67,5 @@ mod test {
 
         return tmp_dir;
     }
+
 }

kondo/tests/common.rs

@@ -11,4 +11,4 @@ pub fn bin() -> PathBuf {
         panic!("kondo binary not found at {:?}", path);
     }
     return path;
-}
+}

kondo/tests/test.rs

@@ -1,7 +1,6 @@
 mod common;
-use std::{io::Write, process::Command};
-
 use kondo;
+use std::process::Command;
 
 // parse age filter is marked public. Can we use it?
 #[test]
@@ -27,7 +26,7 @@ fn test_version() {
 #[ignore = "in progress"]
 #[test]
 fn test_can_run_cargo() {
-    let tmpdir = create_fake_python_project("testing".to_string());
+    let tmpdir = kondo_lib::create_fake_python_project("testing".to_string());
     let bin = common::bin();
 
     println!("tmpdr: {:?}", tmpdir.path());
@@ -59,31 +58,3 @@ fn test_can_run_cargo() {
     // clean up
     tmpdir.close().unwrap();
 }
-
-// Given a name, create a new simulated python project in a safe to delete directry
-fn create_fake_python_project(name: String) -> tempfile::TempDir {
-    // Make a new project in a temporary directory
-    let tmp_dir = tempfile::tempdir().unwrap();
-
-    // make a new root in the tmp dir
-    let project_dir = tmp_dir.path().join(&name);
-    std::fs::create_dir(&project_dir).unwrap();
-
-    // Must have a directory to hold the project.
-    let cache_dir = project_dir.join("__pycache__");
-    std::fs::create_dir(&cache_dir).unwrap();
-
-    // Must have data in the cache to delete
-    let mut data_file = std::fs::File::create(cache_dir.join("cache.data")).unwrap();
-    data_file.write_all(b"#oodles of cache')\n").unwrap();
-    let mut data_file_b = std::fs::File::create(cache_dir.join("other.cache")).unwrap();
-    data_file_b.write_all(b"#oodles of cache')\n").unwrap();
-
-    // and a file of type .py to signal we're a python project
-    let mut python_file = std::fs::File::create(project_dir.join("main.py")).unwrap();
-    python_file
-        .write_all(b"#!/bin/python\n\nprint('Hello, world!')\n")
-        .unwrap();
-
-    return tmp_dir;
-}