CI: convert tests to use Breeze tests command
See original GitHub issueThe tests that are run in CI should use breeze
and parallellism implemented in Python.
NOTE: We should pay an attention to provide better error feedback when docker containers could not be started - see https://github.com/apache/airflow/issues/23523
Issue Analytics
- State:
- Created a year ago
- Comments:12 (12 by maintainers)
Top Results From Across the Web
Convert Helm tests to use the new Python Breeze #25678
This PR converts the Helm tests to use the new Python Breeze. It has all the features of previous Kind breeze command and...
Read more >End-to-end testing with GitLab CI/CD and WebdriverIO
The function describe allows you to group related tests. This can be useful if, for example, you want to run the same initialization...
Read more >Testing - Breeze | JS
Double-click the test or click the faint 'Rerun' link on the right to run just that one test. Tests are grouped in modules....
Read more >Migrating from Jest to Vitest - Tech with Cathal
coverage : This is the configuration I use for the c8 reporter. I also specify the folders that I exclude from the report....
Read more >Laravel Dusk - Laravel - The PHP Framework For Web Artisans
Getting Started. Generating Tests. To generate a Dusk test, use the dusk:make Artisan command. The generated test will be placed in ...
Read more >Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start FreeTop Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Top GitHub Comments
Sure. Any test that is currently run by “ci_run_airflow_testing.sh”:
https://github.com/apache/airflow/blob/main/.github/workflows/ci.yml#L1061
Yeah. It’s an ambitious task though 😃.
Just pay attention to the --parallell flag I’ve implemented in other commands (pull-images for one). What we’ve been doing so far - we run the tests in parallel for different test types when we had enough CPUs. But this was overly complex with gnu-parallel and we should do it as single breeze command with
--parallel
flag as parallelism is much better handled in Python and easier to reason about.One of the goals of the task is not only to duplicate the current behaviour, but also improve it - using python we might want to do a bit better when it comes to grabbing the output of such parallel tests and displaying them to the user. The current solution “looks” good, but there were a few things that were just too complex to implement in Bash easily, but are entirely possible if we do it in Python.
There are a few properties for the target solution:
The difficult thing is that it has to be “all in one go” - so it will be quite a big PR overall.
Also pay attention to a change I am about to submit - I am doing a refactor of packages/modules in Breeze - so you might want to wait with some heavy implementation once it is merged.