ashb commented on a change in pull request #7472: [AIRFLOW-6850] Handle no
access to code in UI
URL: https://github.com/apache/airflow/pull/7472#discussion_r381931498
##########
File path: tests/www/test_views.py
##########
@@ -569,6 +569,11 @@ def test_code(self):
resp = self.client.get(url, follow_redirects=True)
self.check_content_in_response('example_bash_operator', resp)
+ def test_code_no_file(self):
+ url = 'code?dag_id=example_bash_operator_xxx'
+ resp = self.client.get(url, follow_redirects=True)
Review comment:
This error is dag not found (i.e. it's not even in the database) which is
very different to dag _source_ code could not be read by webserver. i.e. this
test isn't checking what you describe in the PR.
(This request should result in a 302 back to home and show a flash of "DAG
example_bash_operator_xxx not found" or similar)
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
