Migrate Google example gcp_transfer to new design AIP-47

docs/apache-airflow-providers-google/operators/cloud/cloud_storage_transfer_service.rst

@@ -58,7 +58,7 @@ For parameter definition, take a look at
 Using the operator
 """"""""""""""""""
 
-.. exampleinclude:: /../../airflow/providers/google/cloud/example_dags/example_cloud_storage_transfer_service_gcp.py
+.. exampleinclude:: /../../tests/system/providers/google/cloud/transfers/example_cloud_storage_transfer_service_gcp.py
       :language: python
       :start-after: [START howto_operator_gcp_transfer_create_job_body_gcp]
       :end-before: [END howto_operator_gcp_transfer_create_job_body_gcp]
@@ -138,12 +138,12 @@ For parameter definition, take a look at
 Using the operator
 """"""""""""""""""
 
-.. exampleinclude:: /../../airflow/providers/google/cloud/example_dags/example_cloud_storage_transfer_service_gcp.py
+.. exampleinclude:: /../../tests/system/providers/google/cloud/transfers/example_cloud_storage_transfer_service_gcp.py
       :language: python
       :start-after: [START howto_operator_gcp_transfer_update_job_body]
       :end-before: [END howto_operator_gcp_transfer_update_job_body]
 
-.. exampleinclude:: /../../airflow/providers/google/cloud/example_dags/example_cloud_storage_transfer_service_gcp.py
+.. exampleinclude:: /../../tests/system/providers/google/cloud/transfers/example_cloud_storage_transfer_service_gcp.py
       :language: python
       :dedent: 4
       :start-after: [START howto_operator_gcp_transfer_update_job]

airflow/providers/google/cloud/example_dags/example_cloud_storage_transfer_service_gcp.py → tests/system/providers/google/cloud/transfers/example_cloud_storage_transfer_service_gcp.py

@@ -32,7 +32,6 @@
 from datetime import datetime, timedelta
 
 from airflow import models
-from airflow.models.baseoperator import chain
 from airflow.providers.google.cloud.hooks.cloud_storage_transfer_service import (
     ALREADY_EXISTING_IN_SINK,
     BUCKET_NAME,
@@ -64,6 +63,10 @@
 from airflow.providers.google.cloud.sensors.cloud_storage_transfer_service import (
     CloudDataTransferServiceJobStatusSensor,
 )
+from airflow.utils.trigger_rule import TriggerRule
+
+ENV_ID = os.environ.get("SYSTEM_TESTS_ENV_ID")
+DAG_ID = "example_gcp_transfer"
 
 GCP_PROJECT_ID = os.environ.get("GCP_PROJECT_ID", "example-project")
 GCP_TRANSFER_FIRST_TARGET_BUCKET = os.environ.get(
@@ -100,12 +103,12 @@
 # [END howto_operator_gcp_transfer_update_job_body]
 
 with models.DAG(
-    "example_gcp_transfer",
+    DAG_ID,
+    schedule='@once',
     start_date=datetime(2021, 1, 1),
     catchup=False,
-    tags=["example"],
+    tags=["example", "gcp"],
 ) as dag:
-
     create_transfer = CloudDataTransferServiceCreateJobOperator(
         task_id="create_transfer", body=gcs_to_gcs_transfer_body
     )
@@ -142,13 +145,28 @@
         task_id="delete_transfer_from_gcp_job",
         job_name="{{task_instance.xcom_pull('create_transfer')['name']}}",
         project_id=GCP_PROJECT_ID,
+        trigger_rule=TriggerRule.ALL_DONE,
     )
 
-    chain(
-        create_transfer,
-        wait_for_transfer,
-        update_transfer,
-        list_operations,
-        get_operation,
-        delete_transfer,
+    (
+        # TEST SETUP
+        create_transfer
+        # TEST BODY
+        >> wait_for_transfer
+        >> update_transfer
+        >> list_operations
+        >> get_operation
+        # TEST TEARDOWN
+        >> delete_transfer
     )
+
+    from tests.system.utils.watcher import watcher
+
+    # This test needs watcher in order to properly mark success/failure
+    # when "tearDown" task with trigger rule is part of the DAG
+    list(dag.tasks) >> watcher()
+
+from tests.system.utils import get_test_run  # noqa: E402
+
+# Needed to run the example DAG with pytest (see: tests/system/README.md#run_via_pytest)
+test_run = get_test_run(dag)