style: gate logic on its used platforms

Author: DavSanchez

test/e2e-runner/src/common.rs

@@ -1,11 +1,17 @@
 use std::path::PathBuf;
 
+#[cfg(any(target_os = "linux", target_os = "windows"))]
 pub mod config;
+#[cfg(target_os = "windows")]
 pub mod exec;
+#[cfg(any(target_os = "linux", target_os = "windows"))]
 pub mod file;
 pub mod fleet_control_api;
+#[cfg(any(target_os = "linux", target_os = "windows"))]
 pub mod logs;
+#[cfg(any(target_os = "linux", target_os = "windows"))]
 pub mod nrql;
+#[cfg(any(target_os = "linux", target_os = "windows"))]
 pub mod on_drop;
 pub mod test;
 
@@ -93,6 +99,7 @@ pub struct FleetControlApiArgs {
 }
 
 /// Data to set up installation
+#[cfg(any(target_os = "linux", target_os = "windows"))]
 pub struct RecipeData {
     pub args: InstallationArgs,
     pub fleet_id: String,
@@ -102,6 +109,7 @@ pub struct RecipeData {
     pub monitoring_source: String,
 }
 
+#[cfg(any(target_os = "linux", target_os = "windows"))]
 impl Default for RecipeData {
     fn default() -> Self {
         Self {

test/e2e-runner/src/common/config.rs

@@ -1,8 +1,10 @@
-use crate::common::file::write;
-use serde_yaml::Value;
 use std::{fs, io::Write, path::PathBuf};
+
+use serde_yaml::Value;
 use tracing::info;
 
+use crate::common::file::write;
+
 /// Updates the agent control config in `config_path` to include the content specified in `new_content`
 pub fn update_config(config_path: impl AsRef<str>, new_content: impl AsRef<str>) {
     let config_path = config_path.as_ref();
@@ -35,6 +37,7 @@ pub fn update_config(config_path: impl AsRef<str>, new_content: impl AsRef<str>)
     write(config_path, updated_content);
 }
 
+#[cfg(target_os = "windows")]
 /// Updates the agent control agents config in `config_path` to the specified in `new_content`
 pub fn modify_agents_config(
     config_path: impl AsRef<str>,

test/e2e-runner/src/common/exec.rs

@@ -76,6 +76,7 @@ impl LongRunningProcess {
             .clone()
     }
 }
+
 impl Drop for LongRunningProcess {
     fn drop(&mut self) {
         #[cfg(target_family = "windows")]

test/e2e-runner/src/common/on_drop.rs

@@ -2,13 +2,15 @@
 pub struct CleanUp<F: FnOnce()> {
     action: Option<F>,
 }
+
 impl<F: FnOnce()> CleanUp<F> {
     pub fn new(action: F) -> Self {
         Self {
             action: Some(action),
         }
     }
 }
+
 impl<F: FnOnce()> Drop for CleanUp<F> {
     fn drop(&mut self) {
         if let Some(action) = self.action.take() {

test/e2e-runner/src/linux.rs

@@ -1,3 +1,4 @@
+use crate::common::fleet_control_api;
 use crate::{LinuxCli, LinuxScenarios, init_logging};
 use clap::Parser;
 
@@ -44,7 +45,7 @@ pub fn run_linux_e2e() {
             scenarios::fleet_control::test_fleet_control(args);
         }
         LinuxScenarios::FleetControlApi(args) => {
-            scenarios::fleet_control::run_fleet_control_api(args);
+            fleet_control_api::run_fleet_control_api(args);
         }
     };
 }

test/e2e-runner/src/linux/scenarios/fleet_control.rs

@@ -7,14 +7,6 @@ use crate::linux::install::{install_agent_control_from_recipe, tear_down_test};
 use std::time::Duration;
 use tracing::info;
 
-/// Runs Fleet Control API interaction (trigger tests and poll for completion).
-///
-/// This function only handles the Fleet Control API communication and does not
-/// install or configure Agent Control. Useful when AC is already deployed externally.
-pub fn run_fleet_control_api(args: FleetControlApiArgs) {
-    fleet_control_api::run_fleet_control_api(args);
-}
-
 pub fn test_fleet_control(args: InstallationArgs) {
     let fleet_id = args
         .fleet_id

test/e2e-runner/src/main.rs

@@ -1,6 +1,9 @@
 mod common;
+#[cfg(target_os = "linux")]
 mod linux;
+#[cfg(target_os = "macos")]
 mod macos;
+#[cfg(target_os = "windows")]
 mod windows;
 
 use crate::common::{FleetControlApiArgs, InstallationArgs};