Skip to content

Commit

Permalink
test: Remove the lsm_supported integration test
Browse files Browse the repository at this point in the history
Having this as a test is misleading, because it's always going to fail
on kernels or configurations which don't support eBPF LSM. Such case
should result in graceful rollback to kprobes, not in failign tests.
  • Loading branch information
vadorovsky committed May 10, 2024
1 parent 8a082d9 commit 1b21aba
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 24 deletions.
23 changes: 0 additions & 23 deletions crates/bpf-common/src/feature_autodetect/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -80,26 +80,3 @@ pub(crate) fn load_program(
))
}
}

#[cfg(feature = "test-suite")]
pub mod test_suite {
use crate::test_runner::{TestCase, TestReport, TestSuite};

use super::lsm::lsm_supported;

pub fn tests() -> TestSuite {
TestSuite {
name: "feature_autodetect",
tests: vec![lsm()],
}
}

fn lsm() -> TestCase {
TestCase::new("lsm", async {
TestReport {
success: tokio::task::spawn_blocking(lsm_supported).await.unwrap(),
lines: vec![],
}
})
}
}
1 change: 0 additions & 1 deletion test-suite/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@ pub use test_suite_runner::TestSuiteRunner;
/// Returns the testable modules of Pulsar
pub fn modules() -> Vec<bpf_common::test_runner::TestSuite> {
vec![
bpf_common::feature_autodetect::test_suite::tests(),
bpf_filtering::test_suite::tests(),
file_system_monitor::test_suite::tests(),
network_monitor::test_suite::tests(),
Expand Down

0 comments on commit 1b21aba

Please sign in to comment.