Bowrna commented on code in PR #38525:
URL: https://github.com/apache/airflow/pull/38525#discussion_r1542338886
##########
tests/providers/sftp/triggers/test_sftp.py:
##########
@@ -176,12 +176,11 @@ async def
test_sftp_trigger_run_trigger_failure_state(self, mock_get_files_by_pa
trigger = SFTPTrigger(path="test/path/", sftp_conn_id="sftp_default",
file_pattern="my_test_file")
expected_event = {"status": "failure", "message": "An unexpected
exception"}
-
- with pytest.raises(Exception):
Review Comment:
<img width="1356" alt="Screenshot 2024-03-28 at 11 03 11 AM"
src="https://github.com/apache/airflow/assets/10162465/7755d85a-c527-4bfa-a470-f0cf54a5f4f1";>
When wrapper trigger.run() part and await part in code inside the with
statement, it didn't raise any exception as the code is handled in such way
that we don't throw exception but return the error message
--
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.
To unsubscribe, e-mail: commits-unsubscr...@airflow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
