You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/03/23 10:53:03 UTC

[GitHub] [airflow] eladkal opened a new pull request #14953: remove duplicate assert in TestS3KeysUnchangedSensor

eladkal opened a new pull request #14953:
URL: https://github.com/apache/airflow/pull/14953


   `test_poke_succeeds_on_upload_complete` in `TestS3KeysUnchangedSensor` has the same assert twice.
   The duplication exist since the sensor was added https://github.com/apache/airflow/pull/9817. This PR removes the duplication.


-- 
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:
users@infra.apache.org

[GitHub] [airflow] eladkal closed pull request #14953: remove duplicate assert in TestS3KeysUnchangedSensor

Posted by GitBox <gi...@apache.org>.

eladkal closed pull request #14953:
URL: https://github.com/apache/airflow/pull/14953


   


-- 
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:
users@infra.apache.org

[GitHub] [airflow] eladkal commented on pull request #14953: remove duplicate assert in TestS3KeysUnchangedSensor

Posted by GitBox <gi...@apache.org>.

eladkal commented on pull request #14953:
URL: https://github.com/apache/airflow/pull/14953#issuecomment-804840061


   worked locally but seems not on CI.
   I'll investigate


-- 
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:
users@infra.apache.org