Skip to content

Commit 98c88df

Browse files
committed
chore: remove test for "missing" fields
1 parent a52d849 commit 98c88df

1 file changed

Lines changed: 0 additions & 16 deletions

File tree

  • rust/operator-binary/src/controller/build/properties/product_logging

rust/operator-binary/src/controller/build/properties/product_logging/mod.rs

Lines changed: 0 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -242,29 +242,13 @@ REMOTE_TASK_LOG = airflow_local_settings.REMOTE_TASK_LOG
242242
mod tests {
243243
use super::*;
244244

245-
/// The vendored `vector.yaml` keeps only the sources Airflow produces (Python JSON, stdout,
246-
/// stderr) and drops the ones it does not (log4j/log4j2/airlift/opa/tracing). Guards against
247-
/// accidental drift.
248245
#[test]
249246
fn test_vector_config_file_content() {
250247
let content = vector_config_file_content();
251248
assert!(!content.is_empty());
252249
// Airflow logs JSON to `airflow.py.json`, so the Python-JSON source must be present.
253250
assert!(content.contains("files_py"));
254251
assert!(content.contains("*.py.json"));
255-
// Sources Airflow does not emit must have been trimmed out.
256-
for dropped in [
257-
"files_log4j",
258-
"files_log4j2",
259-
"files_airlift",
260-
"files_opa_json",
261-
"files_tracing_rs",
262-
] {
263-
assert!(
264-
!content.contains(dropped),
265-
"vendored vector.yaml should not contain the dropped source {dropped}"
266-
);
267-
}
268252
// The config is env-var-parameterized (resolved at runtime by the Vector container), not
269253
// baked, so the role-group identity must appear as placeholders.
270254
assert!(content.contains("${ROLE_NAME}"));

0 commit comments

Comments
 (0)