feat(windows): add basic unit tests

Author: paologallinaharbur

.github/workflows/push_pr_checks_tests.yml

@@ -89,16 +89,6 @@ jobs:
         with:
           toolchain: ${{ env.RUST_VERSION }}
 
-      - uses: actions/cache/restore@v4
-        with:
-          path: |
-            ~/.cargo/bin/
-            ~/.cargo/registry/index/
-            ~/.cargo/registry/cache/
-            ~/.cargo/git/db/
-            target/
-          key: ${{ runner.os }}-cargo-${{ hashFiles('**/Cargo.lock') }}-checks
-
       - name: Create documentation
         run: cargo doc --no-deps --package newrelic_agent_control
         env:
@@ -409,7 +399,7 @@ jobs:
       E2E_LICENSE_KEY: ${{ secrets.AC_PROD_E2E_LICENSE_KEY }}
 
   build-windows:
-    name: build-widows-binary
+    name: build-windows-binary
     runs-on: windows-2025
     steps:
       - uses: actions/checkout@v4
@@ -420,6 +410,19 @@ jobs:
           RUST_VERSION=$(grep "rust-version" Cargo.toml | cut -d "=" -f2 | tr -d "[:space:]")
           echo "RUST_VERSION=${RUST_VERSION}" >> $GITHUB_ENV
 
+      - name: cargo generate-lockfile
+        if: hashFiles('Cargo.lock') == ''
+        run: cargo generate-lockfile
+      - uses: actions/cache@v4
+        with:
+          path: |
+            ~/.cargo/bin/
+            ~/.cargo/registry/index/
+            ~/.cargo/registry/cache/
+            ~/.cargo/git/db/
+            target/
+          key: ${{ runner.os }}-cargo-${{ hashFiles('**/Cargo.lock') }}-windows-release
+
       - name: Install Rust ${{ env.RUST_VERSION }}
         uses: dtolnay/rust-toolchain@master
         with:

agent-control/src/agent_control/pid_cache.rs

@@ -111,6 +111,7 @@ pub mod tests {
     use fs::mock::MockLocalFile;
     use std::path::PathBuf;
 
+    #[cfg(target_family = "unix")] //TODO This should be removed when Windows support is added
     #[test]
     fn test_new_pid_when_already_running_pid_fails_storing() {
         let already_running_pid: u32 = 123;

agent-control/src/sub_agent/on_host/command/shutdown.rs

@@ -90,6 +90,7 @@ mod tests {
         thread::{self, sleep},
     };
 
+    #[cfg(target_family = "unix")] //TODO This should be removed when Windows support is added
     #[test]
     fn shutdown_custom_timeout() {
         let mut trap_cmd = Command::new("sh")
@@ -125,6 +126,7 @@ mod tests {
         assert_eq!("signal: 9 (SIGKILL)", result.unwrap().to_string());
     }
 
+    #[cfg(target_family = "unix")] //TODO This should be removed when Windows support is added
     #[test]
     fn shutdown_on_time() {
         let mut trap_cmd = Command::new("sh")

agent-control/src/sub_agent/on_host/supervisor.rs

@@ -485,6 +485,7 @@ pub mod tests {
     use tracing_test::internal::logs_with_scope_contain;
     use tracing_test::traced_test;
 
+    #[cfg(target_family = "unix")] //TODO This should be removed when Windows support is added
     #[traced_test]
     #[rstest]
     #[case::long_running_process_shutdown_after_start(
@@ -502,11 +503,6 @@ pub mod tests {
         ExecutableData::new("sleep".to_owned(), "sleep".to_owned()).with_args(vec!["10".to_owned()]),
         None,
         vec!["supervisor stopped before starting the process", "stopped supervisor without process running"])]
-    #[case::long_running_process_shutdown_after_start(
-        "long-running",
-        ExecutableData::new("sleep".to_owned(), "sleep".to_owned()).with_args(vec!["10".to_owned()]),
-        Some(Duration::from_secs(1)),
-        vec!["stopping supervisor process", "supervisor has been stopped and process terminated"])]
     fn test_supervisor_gracefully_shutdown(
         #[case] agent_id: &str,
         #[case] executable: ExecutableData,