[GitHub] [airflow]: Workflow run "Build images for Fix discoverability of tests for ARM in Breeze https://github.com/apache/airflow/pull/28432 " is working again!

[GitBox <gi...@apache.org>]

The GitHub Actions job "Build images for Fix discoverability of tests for ARM in Breeze https://github.com/apache/airflow/pull/28432
" on airflow.git has succeeded.
Run started by GitHub user potiuk (triggered by potiuk).

Head commit for run:
dd6a357b5a34dafc6457b482fd6eb2a26e1ef60b / Jarek Potiuk <ja...@potiuk.com>
Fix discoverability of tests for ARM in Breeze

Breeze in case of ARM processor lacks support for several components
(because they do not have supported ARM binaries available):

* MySQL
* MSSQL
* LevelDB
* Azure Service Bus

When you try to attempt to run pytest on a group of tests that import
one of those, the collection failed and none of the tests could run
even if some of them could.

This change uses pytest's skip on a module level and local imports
in case the tests are inter-mixed with other tests in the same module
to avoid import errors during collection.

The try/except pattern over pytest.importorskip is preferred because
we are using try/except in a number of other cases and we are pretty
familiar with similar pattern and importorskipi has a bit unexpected
behaviour (it returns imported module and you do not see the usual
`import nnnn`. Also in our case we often wrap more than one
import in one try/except (and it would lead to a duplicating messages
to print really.

We also add a separate command in ci to just perform a collection of
tests and see if all tests are collectable after uninstalling all
those libraries. This would prevent the problems from reapparing.

Report URL: https://github.com/apache/airflow/actions/runs/3721640255

With regards,
GitHub Actions via GitBox


---------------------------------------------------------------------
To unsubscribe, e-mail: jobs-unsubscribe@airflow.apache.org
For additional commands, e-mail: jobs-help@airflow.apache.org