chore: remove code formatting for samples dir

Author: parthea

gapic/samplegen/samplegen.py

@@ -416,7 +416,7 @@ def _normal_request_setup(self, base_param_to_attrs, val, request, field):
                 )
             del request["field"]
 
-        if isinstance(request["value"], str):
+        if isinstance(request["value"], str) or isinstance(request["value"], list):
             # Passing value through json is a safe and simple way of
             # making sure strings are properly wrapped and quotes escaped.
             # This statement both wraps enums in quotes and escapes quotes
@@ -426,6 +426,9 @@ def _normal_request_setup(self, base_param_to_attrs, val, request, field):
             # This is preferable to adding the necessary import statement
             # and requires less munging of the assigned value
             request["value"] = json.dumps(request["value"])
+        elif isinstance(request["value"], bytes):
+            # Use double quotes for bytes literals for consistency.
+            request["value"] = f'b"{request["value"].decode("utf-8")}"'
 
         # Mypy isn't smart enough to handle dictionary unpacking,
         # so disable it for the AttributeRequestSetup ctor call.

gapic/templates/examples/feature_fragments.j2

@@ -174,11 +174,15 @@ client = {{ module_name }}.{{ client_name }}()
 {% endif %}
 {% endfor %}
 {% if not full_request.flattenable %}
+{% if full_request.request_list|length > 0 %}
 request = {{ module_name }}.{{ request_type.ident.name }}(
 {% for parameter in full_request.request_list %}
     {{ parameter.base }}={{ parameter.base if parameter.body else parameter.single.value }},
 {% endfor %}
 )
+{% else %}
+request = {{ module_name }}.{{ request_type.ident.name }}()
+{% endif %}
 {# Note: This template assumes only one request needs to be sent. When samples accept
 configs the client streaming logic should be modified to allow 2+ request objects. #}
 {# If client streaming, wrap the single request in a generator that produces 'requests' #}
@@ -256,8 +260,8 @@ operation
 # Make the request
 {% if calling_form in [calling_form_enum.Request, calling_form_enum.RequestStreamingClient] %}
 {% if response_statements %}response = {% endif %}{{ method_invocation_text|trim }}
-
 {% if response_statements %}
+
 # Handle the response
 {% for statement in response_statements %}
 {{ dispatch_statement(statement)|trim }}

gapic/templates/examples/sample.py.j2

@@ -39,6 +39,7 @@
     {{ frags.render_calling_form(method_call, calling_form, calling_form_enum, sample.transport, sample.response)|indent -}}
     {% endwith %}
 
+
 # [END {{ sample.id }}]
 {# TODO: Enable main block (or decide to remove main block from python sample) #}
 {# {{ frags.render_main_block(sample, sample.request) }} #}

gapic/templates/noxfile.py.j2

@@ -27,10 +27,9 @@ LINT_PATHS = ["docs", "{{ api.naming.versioned_module_name }}", "tests", "noxfil
 # and less concerned about the line length.
 LINT_LINE_LENGTH = 150
 
-# Add samples to the list of directories to format if the directory exists.
+# Add samples to the list of directories to lint if the directory exists.
 if os.path.isdir("samples"):
     LINT_PATHS.append("samples")
-    FORMAT_PATHS.append("samples")
 
 ALL_PYTHON = [
     "3.7",

tests/integration/goldens/asset/google/cloud/asset_v1/services/asset_service/async_client.py

@@ -2439,8 +2439,7 @@ async def sample_update_saved_query():
                 client = asset_v1.AssetServiceAsyncClient()
 
                 # Initialize request argument(s)
-                request = asset_v1.UpdateSavedQueryRequest(
-                )
+                request = asset_v1.UpdateSavedQueryRequest()
 
                 # Make the request
                 response = await client.update_saved_query(request=request)
@@ -2660,7 +2659,7 @@ async def sample_batch_get_effective_iam_policies():
                 # Initialize request argument(s)
                 request = asset_v1.BatchGetEffectiveIamPoliciesRequest(
                     scope="scope_value",
-                    names=['names_value1', 'names_value2'],
+                    names=["names_value1", "names_value2"],
                 )
 
                 # Make the request

tests/integration/goldens/asset/google/cloud/asset_v1/services/asset_service/client.py

@@ -2939,8 +2939,7 @@ def sample_update_saved_query():
                 client = asset_v1.AssetServiceClient()
 
                 # Initialize request argument(s)
-                request = asset_v1.UpdateSavedQueryRequest(
-                )
+                request = asset_v1.UpdateSavedQueryRequest()
 
                 # Make the request
                 response = client.update_saved_query(request=request)
@@ -3158,7 +3157,7 @@ def sample_batch_get_effective_iam_policies():
                 # Initialize request argument(s)
                 request = asset_v1.BatchGetEffectiveIamPoliciesRequest(
                     scope="scope_value",
-                    names=['names_value1', 'names_value2'],
+                    names=["names_value1", "names_value2"],
                 )
 
                 # Make the request

tests/integration/goldens/asset/noxfile.py

@@ -33,10 +33,9 @@
 # and less concerned about the line length.
 LINT_LINE_LENGTH = 150
 
-# Add samples to the list of directories to format if the directory exists.
+# Add samples to the list of directories to lint if the directory exists.
 if os.path.isdir("samples"):
     LINT_PATHS.append("samples")
-    FORMAT_PATHS.append("samples")
 
 ALL_PYTHON = [
     "3.7",

tests/integration/goldens/asset/samples/generated_samples/cloudasset_v1_generated_asset_service_analyze_iam_policy_async.py

@@ -52,4 +52,5 @@ async def sample_analyze_iam_policy():
     # Handle the response
     print(response)
 
+
 # [END cloudasset_v1_generated_AssetService_AnalyzeIamPolicy_async]

tests/integration/goldens/asset/samples/generated_samples/cloudasset_v1_generated_asset_service_analyze_iam_policy_longrunning_async.py

@@ -60,4 +60,5 @@ async def sample_analyze_iam_policy_longrunning():
     # Handle the response
     print(response)
 
+
 # [END cloudasset_v1_generated_AssetService_AnalyzeIamPolicyLongrunning_async]

tests/integration/goldens/asset/samples/generated_samples/cloudasset_v1_generated_asset_service_analyze_iam_policy_longrunning_sync.py

@@ -60,4 +60,5 @@ def sample_analyze_iam_policy_longrunning():
     # Handle the response
     print(response)
 
+
 # [END cloudasset_v1_generated_AssetService_AnalyzeIamPolicyLongrunning_sync]