Skip to content

Conversation

@kaxil
Copy link
Member

@kaxil kaxil commented Sep 15, 2020

I encountered this while working on #10949 (Test Run: https://github.com/apache/airflow/pull/10949/checks?check_run_id=1117381842)

TestApiKerberos::test_trigger_dag previously was dependent that the example_bash_operator exist in the Database.

If one of the other tests didn't write it to the DB or if one of the other tests cleared it from the DB, this test failed.


^ Add meaningful description above

Read the Pull Request Guidelines for more information.
In case of fundamental code change, 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 UPDATING.md.

TestApiKerberos::test_trigger_dag previously was dependent that the `example_bash_operator` exist in the Database.

If one of the other tests didn't write it to the DB or if one of the other tests cleared it from the DB, this test failed.
@kaxil kaxil requested review from ashb and turbaszek September 15, 2020 12:18
@boring-cyborg boring-cyborg bot added area:API Airflow's REST/HTTP API area:webserver Webserver related Issues labels Sep 15, 2020
@kaxil kaxil changed the title Remove test dependency from TestApiKerberos Remove test dependency in TestApiKerberos Sep 15, 2020
@kaxil kaxil removed the area:webserver Webserver related Issues label Sep 15, 2020
@kaxil kaxil merged commit d43bb75 into apache:master Sep 15, 2020
@kaxil kaxil deleted the remove-test-side-effects-api branch September 15, 2020 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:API Airflow's REST/HTTP API

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants