Make code-coverage an executable

Author: ales-otf

Diff for: Cargo.lock

@@ -46,7 +46,6 @@ subtensor-linting = { workspace = true }
 
 [features]
 default = ["std"]
-code-coverage = []
 std = [
 	"codec/std",
 	"frame-benchmarking?/std",

Diff for: pallets/admin-utils/Cargo.toml

@@ -1,15 +1,4 @@
 fn main() {
     println!("cargo:rerun-if-changed=src");
     subtensor_linting::walk_src();
-
-    #[cfg(feature = "code-coverage")]
-    {
-        use std::env;
-        use std::path::Path;
-        let source_dir = env::var("CARGO_MANIFEST_DIR").unwrap();
-        let source_dir = Path::new(&source_dir);
-        let rust_files = subtensor_code_coverage::collect_rust_files(source_dir);
-        // Generate code coverage report
-        subtensor_code_coverage::analyze_files(&rust_files, source_dir);
-    }
 }

Diff for: pallets/admin-utils/build.rs

@@ -36,7 +36,6 @@ subtensor-linting = { workspace = true }
 
 [features]
 default = ["std"]
-code-coverage = []
 std = [
 	"codec/std",
 	"frame-benchmarking?/std",

Diff for: pallets/collective/Cargo.toml

@@ -1,15 +1,4 @@
 fn main() {
     println!("cargo:rerun-if-changed=src");
     subtensor_linting::walk_src();
-
-    #[cfg(feature = "code-coverage")]
-    {
-        use std::env;
-        use std::path::Path;
-        let source_dir = env::var("CARGO_MANIFEST_DIR").unwrap();
-        let source_dir = Path::new(&source_dir);
-        let rust_files = subtensor_code_coverage::collect_rust_files(source_dir);
-        // Generate code coverage report
-        subtensor_code_coverage::analyze_files(&rust_files, source_dir);
-    }
 }

Diff for: pallets/collective/build.rs

@@ -40,7 +40,6 @@ subtensor-linting = { workspace = true }
 
 [features]
 default = ["std"]
-code-coverage = []
 std = [
 	"codec/std",
 	"frame-benchmarking?/std",

Diff for: pallets/commitments/Cargo.toml

@@ -1,15 +1,4 @@
 fn main() {
     println!("cargo:rerun-if-changed=src");
     subtensor_linting::walk_src();
-
-    #[cfg(feature = "code-coverage")]
-    {
-        use std::env;
-        use std::path::Path;
-        let source_dir = env::var("CARGO_MANIFEST_DIR").unwrap();
-        let source_dir = Path::new(&source_dir);
-        let rust_files = subtensor_code_coverage::collect_rust_files(source_dir);
-        // Generate code coverage report
-        subtensor_code_coverage::analyze_files(&rust_files, source_dir);
-    }
 }

Diff for: pallets/commitments/build.rs

@@ -39,7 +39,6 @@ subtensor-linting = { workspace = true }
 
 [features]
 default = ["std"]
-code-coverage = []
 std = [
 	"codec/std",
 	"frame-benchmarking?/std",

Diff for: pallets/registry/Cargo.toml

@@ -1,15 +1,4 @@
 fn main() {
     println!("cargo:rerun-if-changed=src");
     subtensor_linting::walk_src();
-
-    #[cfg(feature = "code-coverage")]
-    {
-        use std::env;
-        use std::path::Path;
-        let source_dir = env::var("CARGO_MANIFEST_DIR").unwrap();
-        let source_dir = Path::new(&source_dir);
-        let rust_files = subtensor_code_coverage::collect_rust_files(source_dir);
-        // Generate code coverage report
-        subtensor_code_coverage::analyze_files(&rust_files, source_dir);
-    }
 }

Diff for: pallets/registry/build.rs

@@ -67,7 +67,6 @@ subtensor-linting = { workspace = true }
 
 [features]
 default = ["std"]
-code-coverage = []
 std = [
 	"codec/std",
 	"frame-benchmarking/std",

Diff for: pallets/subtensor/Cargo.toml

@@ -1,15 +1,4 @@
 fn main() {
     println!("cargo:rerun-if-changed=src");
     subtensor_linting::walk_src();
-
-    #[cfg(feature = "code-coverage")]
-    {
-        use std::env;
-        use std::path::Path;
-        let source_dir = env::var("CARGO_MANIFEST_DIR").unwrap();
-        let source_dir = Path::new(&source_dir);
-        let rust_files = subtensor_code_coverage::collect_rust_files(source_dir);
-        // Generate code coverage report
-        subtensor_code_coverage::analyze_files(&rust_files, source_dir);
-    }
 }

Diff for: pallets/subtensor/build.rs

@@ -12,5 +12,9 @@ procedural-fork.workspace = true
 walkdir.workspace = true
 rayon.workspace = true
 
+[dev-dependencies]
+frame-system.workspace = true
+frame-support.workspace = true
+
 [lints]
 workspace = true

Diff for: support/code-coverage/Cargo.toml

@@ -9,7 +9,7 @@ use rayon::{
 use std::{
     collections::{HashMap, HashSet},
     ffi::OsStr,
-    fs::{self},
+    fs,
     path::{Path, PathBuf},
     str::FromStr,
 };

Diff for: support/code-coverage/src/lib.rs

@@ -0,0 +1,14 @@
+use std::env;
+use std::path::PathBuf;
+
+fn main() {
+	// should we instead provide an argument to set the directory?
+    let source_dir = env::var("CARGO_MANIFEST_DIR").unwrap();
+    let mut source_dir = PathBuf::from(&source_dir);
+    source_dir.pop();
+    source_dir.pop();
+    let rust_files = subtensor_code_coverage::collect_rust_files(source_dir.as_path());
+
+    // Generate code coverage report
+    subtensor_code_coverage::analyze_files(&rust_files, source_dir.as_path());
+}