Skip to content

Commit e0fa123

Browse files
committed
fix tests
1 parent bdc19b9 commit e0fa123

File tree

1 file changed

+12
-4
lines changed
  • metadata-ingestion-modules/airflow-plugin/tests/integration

1 file changed

+12
-4
lines changed

metadata-ingestion-modules/airflow-plugin/tests/integration/test_plugin.py

+12-4
Original file line numberDiff line numberDiff line change
@@ -393,10 +393,18 @@ class DagTestCase:
393393
if HAS_AIRFLOW_DAG_LISTENER_API
394394
else f"v2_{test_case.dag_id}_no_dag_listener"
395395
),
396-
marks=pytest.mark.skipif(
397-
not HAS_AIRFLOW_LISTENER_API,
398-
reason="Cannot test plugin v2 without the Airflow plugin listener API",
399-
),
396+
marks=[
397+
pytest.mark.skipif(
398+
not HAS_AIRFLOW_LISTENER_API,
399+
reason="Cannot test plugin v2 without the Airflow plugin listener API",
400+
),
401+
pytest.mark.skipif(
402+
AIRFLOW_VERSION < packaging.version.parse("2.4.0"),
403+
reason="We skip testing the v2 plugin on Airflow 2.3 because it causes flakiness in the custom properties. "
404+
"Ideally we'd just fix these, but given that Airflow 2.3 is EOL and likely going to be deprecated "
405+
"soon anyways, it's not worth the effort.",
406+
),
407+
],
400408
)
401409
for test_case in test_cases
402410
],

0 commit comments

Comments
 (0)