Commit 89a9c9d
authored
Feat: Add an option for fast tests by gating slow tests to extended_tests feature (#19237)
## Which issue does this PR close?
Closes #19228
## Rationale for this change
This PR gates all fuzz tests behind the ```extended_tests``` feature
flag, allowing developers to run a fast test suite (ideally under 1
minute) during development while still being able to run the full test
suite in CI or when needed.
## What changes are included in this PR?
Added #[cfg(feature = "extended_tests")] to all fuzz test modules in
mod.rs. This includes the following modules:
- aggregate_fuzz
- distinct_count_string_fuzz
- join_fuzz
- merge_fuzz
- sort_fuzz
- sort_query_fuzz
- topk_filter_pushdown
- aggregation_fuzzer
- equivalence
- pruning
- limit_fuzz
- sort_preserving_repartition_fuzz
- window_fuzz
- Utility modules
## Are these changes tested?
Without the extended_tests feature:
```cargo test --package datafusion --test fuzz -- --list```
This shows 0 tests.
With the extended_tests feature:
```cargo test --package datafusion --test fuzz --features extended_tests
-- --list```
This shows all ~100+ fuzz tests.
## Are there any user-facing changes?
Yes. Users can now run:
Fast test suite (default):
```cargo test```
or
```cargo nextest run```
Full test suite with fuzz tests:
```cargo test --features extended_tests```
or
```cargo nextest run --features extended_tests```1 parent 39a1f75 commit 89a9c9d
1 file changed
+4
-1
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
15 | 15 | | |
16 | 16 | | |
17 | 17 | | |
18 | | - | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
19 | 22 | | |
20 | 23 | | |
21 | 24 | | |
| |||
0 commit comments