thesuperzapper commented on a change in pull request #9133: URL: https://github.com/apache/airflow/pull/9133#discussion_r437078140
########## File path: tests/www/test_views.py ########## @@ -1983,7 +1983,7 @@ def test_failed_success(self): past="false", ) resp = self.client.post('failed', data=form) - self.check_content_in_response('Redirecting', resp, 302) + self.check_content_in_response('Redirecting', resp) Review comment: @ashb That test has been broken since it was created (mostly because `can_failure` was missing). You're probably right, about 'Redirecting' what content does that POST return? What text should we check for, other tests in that file check for `example_bash_operator` in the response, should this do the same? ---------------------------------------------------------------- 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: us...@infra.apache.org