[GitHub] feluelle commented on issue #4111: [AIRFLOW-3266] Add AWS Athena Operator and hook

[GitBox <gi...@apache.org>]

feluelle commented on issue #4111: [AIRFLOW-3266] Add AWS Athena Operator and hook
URL: https://github.com/apache/incubator-airflow/pull/4111#issuecomment-433896338
 
 
   > Doc part is still missing. I will be adding it next commit. I am more concerned about unit testing part. As current library does not support athena yet. I am hoping there is some way to test or some one can validate correctness and usability of above code.
   
   What you can always do in **UNIT** Tests is that you can mock the deps that means everything that does not belong to the responsibility of this Unit/Class you want to test.
   That would mean you mock the functions of `AwsHook` you want to use.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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


With regards,
Apache Git Services