VladaZakharova commented on code in PR #56324: URL: https://github.com/apache/airflow/pull/56324#discussion_r2559487316
########## tests/system/providers/google/common/example_quota_project_system.py: ########## @@ -0,0 +1,112 @@ +# Licensed to the Apache Software Foundation (ASF) under one Review Comment: There is no need to skip the test, I think if you are introducing changes for project_id and quota project, I suppose you have tested that the quota is actually used from the correct project. So you ran some tests to check that. I understand that adding a new system tests for this field can be tricky, okay, then please show how you were testing it yourself on your system. Or you were using only unit tests? -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
