refactor: Move eBPF loader tests to the end of the file and clean up test module

Author: hamzzy

agent/src/collector/lock.rs

@@ -167,7 +167,7 @@ impl LockCollector {
 #[cfg(test)]
 mod tests {
     use super::*;
-    use aperture_shared::types::profile::{Frame, Stack};
+    use aperture_shared::types::profile::Stack;
 
     #[test]
     fn test_lock_collector() {

agent/src/ebpf/loader.rs

@@ -217,19 +217,6 @@ pub fn cleanup(links: PerfEventLinks) {
     drop(links);
 }
 
-#[cfg(test)]
-mod tests {
-    use super::*;
-
-    #[test]
-    #[ignore] // Requires eBPF build artifacts
-    fn test_load_cpu_profiler() {
-        // This test requires the eBPF program to be built
-        let result = load_cpu_profiler();
-        assert!(result.is_err()); // Expected to fail until implemented
-    }
-}
-
 use aya::programs::raw_trace_point::RawTracePointLinkId;
 use aya::programs::trace_point::TracePointLinkId;
 use aya::programs::{RawTracePoint, TracePoint};
@@ -449,3 +436,16 @@ pub fn attach_syscall_tracer(
 
     Ok(links)
 }
+
+#[cfg(test)]
+mod tests {
+    use super::*;
+
+    #[test]
+    #[ignore] // Requires eBPF build artifacts
+    fn test_load_cpu_profiler() {
+        // This test requires the eBPF program to be built
+        let result = load_cpu_profiler();
+        assert!(result.is_err()); // Expected to fail until implemented
+    }
+}