airflow icon indicating copy to clipboard operation
airflow copied to clipboard

Move core system tests outside of providers system tests

Open Taragolis opened this issue 1 year ago • 3 comments

This tests for Core, not providers, so better store it separatly. I'm not sure but maybe we should move it into the airflow/example_dags


^ Add meaningful description above Read the Pull Request Guidelines for more information. In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed. In case of a new dependency, check compliance with the ASF 3rd Party License Policy. In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

Taragolis avatar Apr 30 '24 20:04 Taragolis

Will this also fix the associated "[source]" link that's currently broken?

image

It seems like the existing link should work even if it doesn't belong under providers.

RNHTTR avatar Apr 30 '24 21:04 RNHTTR

I'm not sure this one fix link to the sources. Because these directory not included into the build this kind of sources in apache-airflow documentation.

I could have a look tomorrow morning, as other solution move it into the example_dags

Taragolis avatar Apr 30 '24 22:04 Taragolis

@RNHTTR After minor changes in doc configurations tests/system/coreavailable into the apache-airflow documentation

https://github.com/apache/airflow/assets/3998685/446a03a6-3f39-4d72-a423-b078998cc0f3

Taragolis avatar May 01 '24 08:05 Taragolis

Awesome work, congrats on your first merged pull request! You are invited to check our Issue Tracker for additional contributions.

boring-cyborg[bot] avatar May 07 '24 14:05 boring-cyborg[bot]