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 2022/02/04 12:21:49 UTC

[GitHub] [airflow] malthe commented on pull request #21323: Use mocking for Docker decorator tests

malthe commented on pull request #21323:
URL: https://github.com/apache/airflow/pull/21323#issuecomment-1029938645


   > a) What's wrong with using an actual docker container for this test? b) We are now testing the `docker_execute` function, rather than the code that the docker decorator injects.
   > 
   > So _If_ we don't want to us a container (which I'm not sure of) we at least need to test the real code.
   
   This is just the decorator code. The assumption is that it uses the Docker operator which is tested elsewhere.


-- 
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-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org