forked from elastic/elasticsearch
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
ESQL: Reduce number of ignored tests (elastic#108471)
This reduces the number of skipped tests for functions, dropping the number from 130754 to 3226. This doesn't buy us much more coverage, but it doesn't really take any more time so it's probably ok. It'd be nice to have some understanding of each of the skipped tests, 130k is way too many to put in your head. The actual test change is: when you are need to build an evaluator but can't because you'll get a type error, just assert that you get a type error and let the test finish. This is nearly as fast as just bailing, and it gets us to the point where we can start reasoning about the skipped tests.
- Loading branch information
Showing
1 changed file
with
40 additions
and
21 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters