Re: [PR] Change google sheets operator and connected system tests [airflow]

["lubimow-xwf (via GitHub)" <gi...@apache.org>]

lubimow-xwf commented on code in PR #66929:
URL: https://github.com/apache/airflow/pull/66929#discussion_r3765025500


##########
providers/google/tests/system/google/cloud/gcs/example_gcs_to_sheets.py:
##########
@@ -88,7 +99,10 @@ def create_connection(connection_id: str):
     create_connection_task = create_connection(connection_id=CONNECTION_ID)
 
     create_spreadsheet = GoogleSheetsCreateSpreadsheetOperator(
-        task_id="create_spreadsheet", spreadsheet=SPREADSHEET, gcp_conn_id=CONNECTION_ID
+        task_id="create_spreadsheet",
+        spreadsheet=SPREADSHEET,
+        gcp_conn_id=CONNECTION_ID,
+        drive_id=GDRIVE_ID,

Review Comment:
   hey Shahar, I'm going to finish this PR
   
   question about this, wouldn't it be better to create a new test using gdrive and add a flag to enable or disable it?
   



-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org