mik-laj commented on issue #5770: [AIRFLOW-5162] GCS Hook Upload Method 
Improvement
URL: https://github.com/apache/airflow/pull/5770#issuecomment-526879426
 
 
   Unit tests for GCS Hook is available here:
   
https://github.com/apache/airflow/blob/master/tests/contrib/hooks/test_gcs_hook.py
   
   When it comes to writing tests for GCP integration, we distinguish two types:
   * Unit tests -  It uses very actively mocking. You should never use 
authorization data in these tests. Method calls from the Google library should 
be mocked.
   * System tests - It uses real DAG examples and uses authorization data. 
Unfortunately, they are not yet automatically launched on CI.
   
   Hooks and operators are tested using unit tests, but whole integration is 
tested using system tests. I hope you will write unit tests.
   System tests for integration with GCS have not yet been approved, but there 
is open PR: 
   https://github.com/apache/airflow/pull/5951
   My team has been actively working over the past two weeks to increase system 
test coverage.

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

Reply via email to