You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by po...@apache.org on 2022/10/31 00:28:51 UTC
[airflow] branch main updated: Google Drive to local - system tests migrations (AIP-47) (#26798)
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new 896479d1a0 Google Drive to local - system tests migrations (AIP-47) (#26798)
896479d1a0 is described below
commit 896479d1a0624f10a5a9c462071b9c61502cda87
Author: Beata Kossakowska <10...@users.noreply.github.com>
AuthorDate: Mon Oct 31 01:28:40 2022 +0100
Google Drive to local - system tests migrations (AIP-47) (#26798)
---
.../cloud/example_dags/example_gdrive_to_local.py | 50 ---------
.../operators/transfer/gdrive_to_local.rst | 2 +-
.../cloud/transfers/example_gdrive_to_local.py | 118 +++++++++++++++++++++
3 files changed, 119 insertions(+), 51 deletions(-)
diff --git a/airflow/providers/google/cloud/example_dags/example_gdrive_to_local.py b/airflow/providers/google/cloud/example_dags/example_gdrive_to_local.py
deleted file mode 100644
index 1994b39bab..0000000000
--- a/airflow/providers/google/cloud/example_dags/example_gdrive_to_local.py
+++ /dev/null
@@ -1,50 +0,0 @@
-#
-# Licensed to the Apache Software Foundation (ASF) under one
-# or more contributor license agreements. See the NOTICE file
-# distributed with this work for additional information
-# regarding copyright ownership. The ASF licenses this file
-# to you under the Apache License, Version 2.0 (the
-# "License"); you may not use this file except in compliance
-# with the License. You may obtain a copy of the License at
-#
-# http://www.apache.org/licenses/LICENSE-2.0
-#
-# Unless required by applicable law or agreed to in writing,
-# software distributed under the License is distributed on an
-# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
-# KIND, either express or implied. See the License for the
-# specific language governing permissions and limitations
-# under the License.
-from __future__ import annotations
-
-import os
-from datetime import datetime
-
-from airflow import models
-from airflow.providers.google.cloud.transfers.gdrive_to_local import GoogleDriveToLocalOperator
-from airflow.providers.google.suite.sensors.drive import GoogleDriveFileExistenceSensor
-
-FOLDER_ID = os.environ.get("FILE_ID", "1234567890qwerty")
-FILE_NAME = os.environ.get("FILE_NAME", "file.pdf")
-OUTPUT_FILE = os.environ.get("OUTPUT_FILE", "out_file.pdf")
-
-with models.DAG(
- "example_gdrive_to_local_with_gdrive_sensor",
- start_date=datetime(2021, 1, 1),
- catchup=False,
- tags=["example"],
-) as dag:
- # [START detect_file]
- detect_file = GoogleDriveFileExistenceSensor(
- task_id="detect_file", folder_id=FOLDER_ID, file_name=FILE_NAME
- )
- # [END detect_file]
- # [START download_from_gdrive_to_local]
- download_from_gdrive_to_local = GoogleDriveToLocalOperator(
- task_id="download_from_gdrive_to_local",
- folder_id=FOLDER_ID,
- file_name=FILE_NAME,
- output_file=OUTPUT_FILE,
- )
- # [END download_from_gdrive_to_local]
- detect_file >> download_from_gdrive_to_local
diff --git a/docs/apache-airflow-providers-google/operators/transfer/gdrive_to_local.rst b/docs/apache-airflow-providers-google/operators/transfer/gdrive_to_local.rst
index 10fc61920d..8b3b7155e2 100644
--- a/docs/apache-airflow-providers-google/operators/transfer/gdrive_to_local.rst
+++ b/docs/apache-airflow-providers-google/operators/transfer/gdrive_to_local.rst
@@ -38,7 +38,7 @@ data from Google Drive to local filesystem.
Below is an example of using this operator to download file from Google Drive to Local Filesystem.
-.. exampleinclude:: /../../airflow/providers/google/cloud/example_dags/example_gdrive_to_local.py
+.. exampleinclude:: /../../tests/system/providers/google/cloud/transfers/example_gdrive_to_local.py
:language: python
:dedent: 4
:start-after: [START download_from_gdrive_to_local]
diff --git a/tests/system/providers/google/cloud/transfers/example_gdrive_to_local.py b/tests/system/providers/google/cloud/transfers/example_gdrive_to_local.py
new file mode 100644
index 0000000000..b31406fdba
--- /dev/null
+++ b/tests/system/providers/google/cloud/transfers/example_gdrive_to_local.py
@@ -0,0 +1,118 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements. See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership. The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License. You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied. See the License for the
+# specific language governing permissions and limitations
+# under the License.
+"""
+Example DAG using GoogleDriveToLocalOperator.
+
+Using this operator requires the following additional scopes:
+https://www.googleapis.com/auth/drive
+"""
+from __future__ import annotations
+
+import os
+from datetime import datetime
+from pathlib import Path
+
+from airflow import models
+from airflow.models.baseoperator import chain
+from airflow.providers.google.cloud.operators.gcs import GCSCreateBucketOperator, GCSDeleteBucketOperator
+from airflow.providers.google.cloud.transfers.gdrive_to_local import GoogleDriveToLocalOperator
+from airflow.providers.google.cloud.transfers.local_to_gcs import LocalFilesystemToGCSOperator
+from airflow.providers.google.suite.sensors.drive import GoogleDriveFileExistenceSensor
+from airflow.providers.google.suite.transfers.gcs_to_gdrive import GCSToGoogleDriveOperator
+from airflow.utils.trigger_rule import TriggerRule
+
+ENV_ID = os.environ.get("SYSTEM_TESTS_ENV_ID")
+PROJECT_ID = os.environ.get("SYSTEM_TESTS_GCP_PROJECT")
+
+FILE_NAME = "empty.txt"
+OUTPUT_FILE = "out_file.txt"
+DAG_ID = "example_gdrive_to_local"
+BUCKET_NAME = f"bucket_{DAG_ID}_{ENV_ID}"
+
+LOCAL_PATH = str(Path(__file__).parent / "resources" / FILE_NAME)
+
+
+with models.DAG(
+ DAG_ID,
+ start_date=datetime(2021, 1, 1),
+ schedule="@once",
+ catchup=False,
+ tags=["example"],
+) as dag:
+
+ create_bucket = GCSCreateBucketOperator(
+ task_id="create_bucket", bucket_name=BUCKET_NAME, project_id=PROJECT_ID
+ )
+
+ upload_file = LocalFilesystemToGCSOperator(
+ task_id="upload_file",
+ src=LOCAL_PATH,
+ dst=FILE_NAME,
+ bucket=BUCKET_NAME,
+ )
+
+ copy_single_file = GCSToGoogleDriveOperator(
+ task_id="copy_single_file",
+ source_bucket=BUCKET_NAME,
+ source_object=FILE_NAME,
+ destination_object=FILE_NAME,
+ )
+
+ # [START detect_file]
+ detect_file = GoogleDriveFileExistenceSensor(
+ task_id="detect_file",
+ folder_id="",
+ file_name=FILE_NAME,
+ )
+ # [END detect_file]
+
+ # [START download_from_gdrive_to_local]
+ download_from_gdrive_to_local = GoogleDriveToLocalOperator(
+ task_id="download_from_gdrive_to_local",
+ folder_id="",
+ file_name=FILE_NAME,
+ output_file=OUTPUT_FILE,
+ )
+ # [END download_from_gdrive_to_local]
+
+ delete_bucket = GCSDeleteBucketOperator(
+ task_id="delete_bucket", bucket_name=BUCKET_NAME, trigger_rule=TriggerRule.ALL_DONE
+ )
+
+ chain(
+ # TEST SETUP
+ create_bucket,
+ upload_file,
+ copy_single_file,
+ # TEST BODY
+ detect_file,
+ download_from_gdrive_to_local,
+ # TEST TEARDOWN
+ delete_bucket,
+ )
+
+ 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)