fix(templates): resolve UndefinedError by moving auto_populated_fields check to Jinja template namespace using api.all_method_settings

Author: hebaalazzeh

packages/gapic-generator/gapic/schema/wrappers.py

@@ -2413,17 +2413,6 @@ def operation_polling_method(self) -> Optional[Method]:
     def is_internal(self) -> bool:
         return any(m.is_internal for m in self.methods.values())
 
-    @utils.cached_property
-    def has_auto_populated_fields(self) -> bool:
-        """Returns True if any method in this service has auto-populated fields (e.g. AIP-4235 request_id)."""
-        return any(
-            bool(
-                self.api.all_method_settings.get(m.meta.address.proto)
-                and self.api.all_method_settings[m.meta.address.proto].auto_populated_fields
-            )
-            for m in self.methods.values()
-        )
-
     @cached_proto_context
     def with_context(
         self,

packages/gapic-generator/gapic/templates/%namespace/%name_%version/%sub/services/%service/async_client.py.j2

@@ -18,7 +18,14 @@ from {{package_path}} import gapic_version as package_version
 from google.api_core.client_options import ClientOptions
 from google.api_core import exceptions as core_exceptions
 from google.api_core import gapic_v1
-{% if service.has_auto_populated_fields %}
+{% set has_auto_populated_fields = namespace(value=false) %}
+{% for method in service.methods.values() %}
+  {% set method_settings = api.all_method_settings.get(method.meta.address.proto) %}
+  {% if method_settings and method_settings.auto_populated_fields %}
+    {% set has_auto_populated_fields.value = true %}
+  {% endif %}
+{% endfor %}
+{% if has_auto_populated_fields.value %}
 from {{package_path}}._compat import setup_request_id
 {% endif %}
 from google.api_core import retry_async as retries

packages/gapic-generator/gapic/templates/%namespace/%name_%version/%sub/services/%service/client.py.j2

@@ -27,7 +27,14 @@ from google.api_core import exceptions as core_exceptions
 from google.api_core import extended_operation
 {% endif %}
 from google.api_core import gapic_v1
-{% if service.has_auto_populated_fields %}
+{% set has_auto_populated_fields = namespace(value=false) %}
+{% for method in service.methods.values() %}
+  {% set method_settings = api.all_method_settings.get(method.meta.address.proto) %}
+  {% if method_settings and method_settings.auto_populated_fields %}
+    {% set has_auto_populated_fields.value = true %}
+  {% endif %}
+{% endfor %}
+{% if has_auto_populated_fields.value %}
 from {{package_path}}._compat import setup_request_id
 {% endif %}
 from google.api_core import retry as retries

packages/gapic-generator/tests/unit/schema/wrappers/test_service.py

@@ -751,7 +751,5 @@ def test_resource_messages_raises_on_malformed_typeless_resource():
         _ = service.resource_messages
 
 
-def test_service_has_auto_populated_fields():
-    service = make_service(name="ThingDoer")
-    assert not service.has_auto_populated_fields
+