chore: use alternative terms (googleapis#1157)

Author: parthea

gapic/ads-templates/%namespace/%name/%version/%sub/services/%service/client.py.j2

@@ -388,7 +388,7 @@ class {{ service.client_name }}(metaclass={{ service.client_name }}Meta):
         {% if not method.client_streaming %}
         # Create or coerce a protobuf request object.
         {% if method.flattened_fields %}
-        # Sanity check: If we got a request object, we should *not* have
+        # Quick check: If we got a request object, we should *not* have
         # gotten any keyword arguments that map to the request.
         has_flattened_params = any([{{ method.flattened_fields.values()|join(", ", attribute="name") }}])
         if request is not None and has_flattened_params:

gapic/ads-templates/%namespace/%name/%version/%sub/services/%service/transports/grpc.py.j2

@@ -234,7 +234,7 @@ class {{ service.name }}GrpcTransport({{ service.name }}Transport):
         This property caches on the instance; repeated calls return the same
         client.
         """
-        # Sanity check: Only create a new client if we do not already have one.
+        # Quick check: Only create a new client if we do not already have one.
         if self._operations_client is None:
             self._operations_client = operations_v1.OperationsClient(
                 self.grpc_channel

gapic/generator/generator.py

@@ -112,7 +112,7 @@ def get_response(
         # and instead of iterating over it/them, we iterate over samples
         # and plug those into the template.
         for template_name in client_templates:
-            # Sanity check: Skip "private" templates.
+            # Quick check: Skip "private" templates.
             filename = template_name.split("/")[-1]
             if filename.startswith("_") and filename != "__init__.py.j2":
                 continue
@@ -242,7 +242,7 @@ def _render_template(
         if not opts.metadata and template_name.endswith("gapic_metadata.json.j2"):
             return answer
 
-        # Sanity check: Rendering per service and per proto would be a
+        # Quick check: Rendering per service and per proto would be a
         # combinatorial explosion and is almost certainly not what anyone
         # ever wants. Error colorfully on it.
         if "%service" in template_name and "%proto" in template_name:
@@ -343,7 +343,7 @@ def _get_file(
             name=fn,
         )
 
-        # Sanity check: Do not render empty files.
+        # Quick check: Do not render empty files.
         if utils.empty(cgr_file.content) and not fn.endswith(
             ("py.typed", "__init__.py")
         ):

gapic/schema/api.py

@@ -195,7 +195,7 @@ def python_modules(self) -> Sequence[Tuple[str, str]]:
         answer = {
             t.ident.python_import
             for m in self.all_messages.values()
-            # Sanity check: We do make sure that we are not trying to have
+            # Quick check: We do make sure that we are not trying to have
             # a module import itself.
             for t in m.field_types if t.ident.python_import != self_reference
         }

gapic/schema/naming.py

@@ -77,7 +77,7 @@ def build(
         proto_packages = {fd.package for fd in file_descriptors}
         root_package = os.path.commonprefix(tuple(proto_packages)).rstrip('.')
 
-        # Sanity check: If there is no common ground in the package,
+        # Quick check: If there is no common ground in the package,
         # we are obviously in trouble.
         if not root_package:
             raise ValueError(
@@ -119,7 +119,7 @@ def build(
             version=match.get('version', ''),
         )
 
-        # Sanity check: Ensure that the package directives all inferred
+        # Quick check: Ensure that the package directives all inferred
         # the same information.
         if not package_info.version and len(proto_packages) > 1:
             raise ValueError('All protos must have the same proto package '

gapic/schema/wrappers.py

@@ -541,7 +541,7 @@ def get_field(self, *field_path: str,
                 visited_messages=frozenset({self}),
             )
 
-        # Sanity check: If cursor is a repeated field, then raise an exception.
+        # Quick check: If cursor is a repeated field, then raise an exception.
         # Repeated fields are only permitted in the terminal position.
         if cursor.repeated:
             raise KeyError(
@@ -552,7 +552,7 @@ def get_field(self, *field_path: str,
                 'in the fields list in a position other than the end.',
             )
 
-        # Sanity check: If this cursor has no message, there is a problem.
+        # Quick check: If this cursor has no message, there is a problem.
         if not cursor.message:
             raise KeyError(
                 f'Field {".".join(field_path)} could not be resolved from '

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

@@ -256,7 +256,7 @@ class {{ service.async_client_name }}:
         {% if not method.client_streaming %}
         # Create or coerce a protobuf request object.
         {% if method.flattened_fields %}
-        # Sanity check: If we got a request object, we should *not* have
+        # Quick check: If we got a request object, we should *not* have
         # gotten any keyword arguments that map to the request.
         has_flattened_params = any([{{ method.flattened_fields.values()|join(", ", attribute="name") }}])
         if request is not None and has_flattened_params:

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

@@ -422,7 +422,7 @@ class {{ service.client_name }}(metaclass={{ service.client_name }}Meta):
         {% if not method.client_streaming %}
         # Create or coerce a protobuf request object.
         {% if method.flattened_fields %}
-        # Sanity check: If we got a request object, we should *not* have
+        # Quick check: If we got a request object, we should *not* have
         # gotten any keyword arguments that map to the request.
         has_flattened_params = any([{{ method.flattened_fields.values()|join(", ", attribute="name") }}])
         if request is not None and has_flattened_params:

gapic/templates/%namespace/%name_%version/%sub/services/%service/transports/grpc.py.j2

@@ -234,7 +234,7 @@ class {{ service.name }}GrpcTransport({{ service.name }}Transport):
         This property caches on the instance; repeated calls return the same
         client.
         """
-        # Sanity check: Only create a new client if we do not already have one.
+        # Quick check: Only create a new client if we do not already have one.
         if self._operations_client is None:
             self._operations_client = operations_v1.OperationsClient(
                 self.grpc_channel

gapic/templates/%namespace/%name_%version/%sub/services/%service/transports/grpc_asyncio.py.j2

@@ -235,7 +235,7 @@ class {{ service.grpc_asyncio_transport_name }}({{ service.name }}Transport):
         This property caches on the instance; repeated calls return the same
         client.
         """
-        # Sanity check: Only create a new client if we do not already have one.
+        # Quick check: Only create a new client if we do not already have one.
         if self._operations_client is None:
             self._operations_client = operations_v1.OperationsAsyncClient(
                 self.grpc_channel