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


Reply via email to