shahar1 commented on PR #39768:
URL: https://github.com/apache/airflow/pull/39768#issuecomment-2129449686
> Some history: It moved into integration because it have unsupported marker
`@pytest.mark.backend("mssql")`: #38530, there was no check for unsupported DB
Backends at that time. That not a case anymore, because right now we check for
supported backends: `mysql`, `postgres`, `sqlite`
>
> All tests which marked as `@pytest.mark.backend` also automatically
(implicitly) marked as `@pytest.mark.db_test` so it never run at non-db
environment. Some modules or individual test might be required to be marked
with `pytest.mark.db_test`, you could check locally in breeze which moved tests
required this marker
https://github.com/apache/airflow/blob/main/contributing-docs/testing/unit_tests.rst#db-and-non-db-tests
>
> ```
> ❯ breeze shell --backend none
>
> root@a2b2d941fc72:/opt/airflow# pytest tests --skip-db-tests
tests/providers/google/cloud/transfers/test_mssql_to_gcs.py
tests/providers/google/cloud/transfers/test_mssql_to_gcs.py
tests/providers/apache/hive/transfers/test_mssql_to_hive.py
tests/providers/google/cloud/transfers/test_bigquery_to_mssql.py
> ```
I ran the tests with these commands and all ran smoothly.
As I previously stated, as of the present time, these are simple unit tests
- so there shouldn't be a reason for them not to work.
I'll start working on the mssql integration tests soon after that.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
