[GitHub] [airflow] potiuk commented on issue #30407: merge breeze's --test-type and --test-types options

[via GitHub]

potiuk commented on issue #30407:
URL: https://github.com/apache/airflow/issues/30407#issuecomment-1492700622

   * None test type means to run "pytest tests" (everything you find there)
   
   * default `test-types`  list means run whatever tests types you should have  
in parallell:
   
   <img width="1180" alt="Screenshot 2023-04-01 at 00 50 25" 
src="https://user-images.githubusercontent.com/595491/229246228-14755b51-8bf4-4780-88d1-f8f3c2ec806a.png";>
   
   Also `test-type` allows you to select a subset of tests that are NOT 
included in `--test-types` as default list (because the selection is 
"cross-cutting" - i.e. subset of tests that are partialy also present in other 
tests :
   
   <img width="2027" alt="Screenshot 2023-04-01 at 00 53 19" 
src="https://user-images.githubusercontent.com/595491/229246853-6f56c8a8-3c14-40e0-b7aa-b2f66cf07ff5.png";>
   
   * Postgres or MySQL -> runs all test that are tagged with 
(@backend("postgres') - but they might be in CLI, or Core, or Other
   * Quarantine -> they are separate "quarantine" tests (tagged with 
(@quarantined) that have a completely separate job in CI because failure of 
Quarantine tests is not failing for the build (so this job is marked with     
`continue-on-error: true`. Those tests are skipped by default when you run the 
default "All" tests BTW via "pytest tests" 
   
   So those two parameters are rather different, and if they are confusing, 
they should rather be renamed.
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@airflow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org