dabla commented on PR #44809:
URL: https://github.com/apache/airflow/pull/44809#issuecomment-2586456702
@potiuk @eladkal I still have breeze test failing for selected tests on main
(why do we only test on main branch?) since I moved the generic transfer to
common sql, to me the expected values seems correct, but apparently I still get
common.sql as additional provider for the bask operator, which is weird as
standard provider doesn't need that dependency anymore since I moved the
generic transfer, or is it because we run it against main branch, but then why?
```
=========================== short test summary info
============================
FAILED
tests/test_selective_checks.py::test_expected_output_pull_request_main[Providers
standard tests and Serialization tests to run when airflow bash.py changed] -
AssertionError: Correct value for 'selected-providers-list-as-string'
assert 'common.compat common.sql standard' == 'common.compat standard'
- common.compat standard
+ common.compat common.sql standard
? +++++++++++
FAILED
tests/test_selective_checks.py::test_expected_output_pull_request_main[Force
Core and Serialization tests to run when tests bash changed] - AssertionError:
Correct value for 'providers-test-types-list-as-string'
assert 'Providers[common.compat,common.sql] Providers[standard]' ==
'Providers[common.compat] Providers[standard]'
- Providers[common.compat] Providers[standard]
+ Providers[common.compat,common.sql] Providers[standard]
? +++++++++++
FAILED
tests/test_selective_checks.py::test_expected_output_pull_request_main[Only
Python tests] - AssertionError: Correct value for
'providers-test-types-list-as-string'
assert 'Providers[common.compat,common.sql] Providers[standard]' ==
'Providers[common.compat] Providers[standard]'
- Providers[common.compat] Providers[standard]
+ Providers[common.compat,common.sql] Providers[standard]
? +++++++++++
```
--
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]