fix: Lazy load git repo and only run module if not disabled (#306)

A couple of optimizations are done in this PR. One, we now will check config ahead of time to see if a module is disabled before running any module code. Also, we won't try to discover a git repository unless the module requests access to it.

1 files changed, 1 insertions, 22 deletions

diff --git a/tests/testsuite/nodejs.rs b/tests/testsuite/nodejs.rs
index bea6248a1..33b0df2af 100644
--- a/tests/testsuite/nodejs.rs
+++ b/tests/testsuite/nodejs.rs
@@ -2,7 +2,7 @@ use ansi_term::Color;
 use std::fs::{self, File};
 use std::io;
 
-use crate::common::{self, TestCommand};
+use crate::common;
 
 #[test]
 fn folder_without_node_files() -> io::Result<()> {
@@ -70,24 +70,3 @@ fn folder_with_node_modules() -> io::Result<()> {
     assert_eq!(expected, actual);
     Ok(())
 }
-
-#[test]
-#[ignore]
-fn config_disabled() -> io::Result<()> {
-    let dir = common::new_tempdir()?;
-    File::create(dir.path().join("package.json"))?;
-
-    let output = common::render_module("nodejs")
-        .use_config(toml::toml! {
-            [nodejs]
-            disabled = true
-        })
-        .arg("--path")
-        .arg(dir.path())
-        .output()?;
-    let actual = String::from_utf8(output.stdout).unwrap();
-
-    let expected = "";
-    assert_eq!(expected, actual);
-    Ok(())
-}