Skip to content

Commit

Permalink
fix: mark test case as xfail where Acero currently crashes
Browse files Browse the repository at this point in the history
Signed-off-by: Ingo Müller <[email protected]>
  • Loading branch information
ingomueller-net committed Nov 28, 2024
1 parent 6bff9e9 commit 193d70c
Showing 1 changed file with 4 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -17,12 +17,16 @@ def mark_consumer_tests_as_xfail(request):
"""Marks a subset of tests as expected to be fail."""
producer = request.getfixturevalue('producer')
consumer = request.getfixturevalue('consumer')
test_name = request.getfixturevalue('test_name')
if consumer.__class__.__name__ == 'DuckDBConsumer':
if producer.__class__.__name__ != 'DuckDBProducer':
pytest.skip(reason=f'Unsupported Integration: DuckDBConsumer with non {producer.__class__.__name__}')
elif consumer.__class__.__name__ == 'DataFusionConsumer':
if producer.__class__.__name__ != 'DataFusionProducer':
pytest.skip(reason=f'Unsupported Integration: DataFusionConsumer with non {producer.__class__.__name__}')
elif consumer.__class__.__name__ == 'AceroConsumer':
if producer.__class__.__name__ == 'IsthmusProducer' and test_name == 'compute_within_aggregate':
pytest.skip(reason="'isthmus-acero-compute_within_aggregate' currently crashes")


@pytest.mark.usefixtures("prepare_small_tpch_parquet_data")
Expand Down

0 comments on commit 193d70c

Please sign in to comment.