fix: Fixup minor test compile / fmt issues.

This commit fixes some minor issues with compiling default-ignored
tests, and also updates the CI config to make sure those cases are
caught in the future.

Signed-off-by: Andrew Lilley Brinker <[email protected]>

.github/workflows/test.yml

@@ -25,9 +25,9 @@ jobs:
       - uses: actions/checkout@v4
       - uses: dtolnay/rust-toolchain@stable
       - name: Build
-        run: cargo build --verbose
+        run: cargo build --verbose --workspace
       - name: Test
-        run: cargo test --verbose
+        run: cargo test --verbose --workspace
       - name: Lint
-        run: cargo clippy --verbose
+        run: cargo clippy --verbose --workspace
 

libs/hc_git/src/lib.rs

@@ -106,7 +106,7 @@ pub fn get_commits_for_file(repo_path: &Path, file: &str) -> Result<String> {
 #[cfg(test)]
 mod test {
 	use super::*;
-	use hc_command_util::DependentProgram;
+	use hc_common::command_util::DependentProgram;
 
 	#[test]
 	#[ignore = "can't guarantee availability of Git"]

libs/hc_npm/src/lib.rs

@@ -255,8 +255,7 @@ impl NpmCommand {
 #[cfg(test)]
 mod test {
 	use super::*;
-
-	use hc_command_util::DependentProgram;
+	use hc_common::command_util::DependentProgram;
 
 	#[test]
 	#[ignore = "can't guarantee availability of NPM"]

libs/hc_session/src/lib.rs

@@ -494,7 +494,7 @@ pub enum CheckType {
 #[cfg(test)]
 mod tests {
 	use super::*;
-	use hc_common::with_env_vars;
+	use hc_common::test_util::with_env_vars;
 	use tempfile::TempDir;
 
 	const TEMPDIR_PREFIX: &str = "hc_test";

tests/can_run.rs

@@ -1,4 +1,4 @@
-use hc_common::{CheckKind, Result};
+use hc_common::{error::Result, CheckKind};
 use hc_core::*;
 use std::ffi::OsString;
 use tempfile::tempdir;