malthe edited a comment 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 (I hope!) -- 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]
