cfg(test) does not apply to integration tests, we specify a separate …

…feature to make it work
metno · Jun 24, 2024 · 628c795 · 628c795
1 parent 3ce2acc
commit 628c795
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 5 deletions.
diff --git a/ingestion/Cargo.toml b/ingestion/Cargo.toml
@@ -3,6 +3,9 @@ name = "lard_ingestion"
 version = "0.1.0"
 edition.workspace = true
 
+[features]
+integration_tests = []
+
 [dependencies]
 axum.workspace = true
 bb8.workspace = true

diff --git a/ingestion/src/kldata.rs b/ingestion/src/kldata.rs
@@ -213,9 +213,8 @@ pub async fn filter_and_label_kldata(
             param_id.to_owned(),
         )? {
             // TODO: log that the timeseries is closed? Mostly useful for tests
-            #[cfg(test)]
+            #[cfg(feature = "integration_tests")]
             eprintln!("station {}: timeseries is closed", chunk.station_id);
-
             continue;
         }
 

diff --git a/ingestion/src/permissions.rs b/ingestion/src/permissions.rs
@@ -12,8 +12,7 @@ pub struct ParamPermit {
     permit_id: i32,
 }
 
-// Only used in tests
-#[doc(hidden)]
+#[cfg(feature = "integration_tests")]
 impl ParamPermit {
     pub fn new(type_id: i32, param_id: i32, permit_id: i32) -> ParamPermit {
         ParamPermit {

diff --git a/integration_tests/Cargo.toml b/integration_tests/Cargo.toml
@@ -5,7 +5,7 @@ edition.workspace = true
 
 [dependencies]
 lard_api = { path = "../api" }
-lard_ingestion = { path = "../ingestion" }
+lard_ingestion = { path = "../ingestion", features = ["integration_tests"] }
 chrono.workspace = true
 tokio.workspace = true
 tokio-postgres.workspace = true