vandonr-amz commented on PR #31018: URL: https://github.com/apache/airflow/pull/31018#issuecomment-1568878647
> I have run the system tests using Breeze and it ran fine locally for me It works fine because you poke once before deferring to see if the key is there https://github.com/apache/airflow/pull/31018/files#diff-67c5417e9404f79455ed588c00813384f728f92efc802dd9c7af1a06b0573a97R146-R147 and it's already there in the context of the system test, so the task is never deferred it's not a great situation, because if they key's not here, the test would timeout instead of failing (because deferring is currently not supported when running system tests with breeze, deferred tasks are never picked up). Also, this makes it look like we're testing the deferrable capabilities, when actually `defer` is never called. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
