chore(python): add nox session to sort python imports (#182)

gcf-owl-bot[bot] · web-flow · commit 6122ac06bf5e · 2022-04-20T20:19:24.000-04:00
Source-Link: googleapis/synthtool@1b71c10 Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-python:latest@sha256:00c9d764fd1cd56265f12a5ef4b99a0c9e87cf261018099141e2ca5158890416 Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
diff --git a/packages/google-cloud-iam/.github/.OwlBot.lock.yaml b/packages/google-cloud-iam/.github/.OwlBot.lock.yaml
@@ -13,5 +13,5 @@
 # limitations under the License.
 docker:
   image: gcr.io/cloud-devrel-public-resources/owlbot-python:latest
-  digest: sha256:8a5d3f6a2e43ed8293f34e06a2f56931d1e88a2694c3bb11b15df4eb256ad163
-# created: 2022-04-06T10:30:21.687684602Z
+  digest: sha256:00c9d764fd1cd56265f12a5ef4b99a0c9e87cf261018099141e2ca5158890416
+# created: 2022-04-20T23:42:53.970438194Z
diff --git a/packages/google-cloud-iam/docs/conf.py b/packages/google-cloud-iam/docs/conf.py
@@ -24,9 +24,9 @@
 # All configuration values have a default; values that are commented out
 # serve to show the default.
 
-import sys
 import os
 import shlex
+import sys
 
 # If extensions (or modules to document with autodoc) are in another directory,
 # add these directories to sys.path here. If the directory is relative to the
diff --git a/packages/google-cloud-iam/google/cloud/iam_credentials/__init__.py b/packages/google-cloud-iam/google/cloud/iam_credentials/__init__.py
@@ -14,22 +14,23 @@
 # limitations under the License.
 #
 
+from google.cloud.iam_credentials_v1.services.iam_credentials.async_client import (
+    IAMCredentialsAsyncClient,
+)
 from google.cloud.iam_credentials_v1.services.iam_credentials.client import (
     IAMCredentialsClient,
 )
-from google.cloud.iam_credentials_v1.services.iam_credentials.async_client import (
-    IAMCredentialsAsyncClient,
+from google.cloud.iam_credentials_v1.types.common import (
+    GenerateAccessTokenRequest,
+    GenerateAccessTokenResponse,
+    GenerateIdTokenRequest,
+    GenerateIdTokenResponse,
+    SignBlobRequest,
+    SignBlobResponse,
+    SignJwtRequest,
+    SignJwtResponse,
 )
 
-from google.cloud.iam_credentials_v1.types.common import GenerateAccessTokenRequest
-from google.cloud.iam_credentials_v1.types.common import GenerateAccessTokenResponse
-from google.cloud.iam_credentials_v1.types.common import GenerateIdTokenRequest
-from google.cloud.iam_credentials_v1.types.common import GenerateIdTokenResponse
-from google.cloud.iam_credentials_v1.types.common import SignBlobRequest
-from google.cloud.iam_credentials_v1.types.common import SignBlobResponse
-from google.cloud.iam_credentials_v1.types.common import SignJwtRequest
-from google.cloud.iam_credentials_v1.types.common import SignJwtResponse
-
 __all__ = (
     "IAMCredentialsClient",
     "IAMCredentialsAsyncClient",
diff --git a/packages/google-cloud-iam/google/cloud/iam_credentials_v1/__init__.py b/packages/google-cloud-iam/google/cloud/iam_credentials_v1/__init__.py
@@ -14,17 +14,17 @@
 # limitations under the License.
 #
 
-from .services.iam_credentials import IAMCredentialsClient
-from .services.iam_credentials import IAMCredentialsAsyncClient
-
-from .types.common import GenerateAccessTokenRequest
-from .types.common import GenerateAccessTokenResponse
-from .types.common import GenerateIdTokenRequest
-from .types.common import GenerateIdTokenResponse
-from .types.common import SignBlobRequest
-from .types.common import SignBlobResponse
-from .types.common import SignJwtRequest
-from .types.common import SignJwtResponse
+from .services.iam_credentials import IAMCredentialsAsyncClient, IAMCredentialsClient
+from .types.common import (
+    GenerateAccessTokenRequest,
+    GenerateAccessTokenResponse,
+    GenerateIdTokenRequest,
+    GenerateIdTokenResponse,
+    SignBlobRequest,
+    SignBlobResponse,
+    SignJwtRequest,
+    SignJwtResponse,
+)
 
 __all__ = (
     "IAMCredentialsAsyncClient",
diff --git a/packages/google-cloud-iam/google/cloud/iam_credentials_v1/services/iam_credentials/__init__.py b/packages/google-cloud-iam/google/cloud/iam_credentials_v1/services/iam_credentials/__init__.py
@@ -13,8 +13,8 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 #
-from .client import IAMCredentialsClient
 from .async_client import IAMCredentialsAsyncClient
+from .client import IAMCredentialsClient
 
 __all__ = (
     "IAMCredentialsClient",
diff --git a/packages/google-cloud-iam/google/cloud/iam_credentials_v1/services/iam_credentials/async_client.py b/packages/google-cloud-iam/google/cloud/iam_credentials_v1/services/iam_credentials/async_client.py
@@ -17,26 +17,28 @@
 import functools
 import re
 from typing import Dict, Mapping, Optional, Sequence, Tuple, Type, Union
-import pkg_resources
 
-from google.api_core.client_options import ClientOptions
 from google.api_core import exceptions as core_exceptions
 from google.api_core import gapic_v1
 from google.api_core import retry as retries
+from google.api_core.client_options import ClientOptions
 from google.auth import credentials as ga_credentials  # type: ignore
 from google.oauth2 import service_account  # type: ignore
+import pkg_resources
 
 try:
     OptionalRetry = Union[retries.Retry, gapic_v1.method._MethodDefault]
 except AttributeError:  # pragma: NO COVER
     OptionalRetry = Union[retries.Retry, object]  # type: ignore
 
-from google.cloud.iam_credentials_v1.types import common
 from google.protobuf import duration_pb2  # type: ignore
 from google.protobuf import timestamp_pb2  # type: ignore
-from .transports.base import IAMCredentialsTransport, DEFAULT_CLIENT_INFO
-from .transports.grpc_asyncio import IAMCredentialsGrpcAsyncIOTransport
+
+from google.cloud.iam_credentials_v1.types import common
+
 from .client import IAMCredentialsClient
+from .transports.base import DEFAULT_CLIENT_INFO, IAMCredentialsTransport
+from .transports.grpc_asyncio import IAMCredentialsGrpcAsyncIOTransport
 
 
 class IAMCredentialsAsyncClient:
diff --git a/packages/google-cloud-iam/google/cloud/iam_credentials_v1/services/iam_credentials/client.py b/packages/google-cloud-iam/google/cloud/iam_credentials_v1/services/iam_credentials/client.py
@@ -17,27 +17,29 @@
 import os
 import re
 from typing import Dict, Mapping, Optional, Sequence, Tuple, Type, Union
-import pkg_resources
 
 from google.api_core import client_options as client_options_lib
 from google.api_core import exceptions as core_exceptions
 from google.api_core import gapic_v1
 from google.api_core import retry as retries
 from google.auth import credentials as ga_credentials  # type: ignore
+from google.auth.exceptions import MutualTLSChannelError  # type: ignore
 from google.auth.transport import mtls  # type: ignore
 from google.auth.transport.grpc import SslCredentials  # type: ignore
-from google.auth.exceptions import MutualTLSChannelError  # type: ignore
 from google.oauth2 import service_account  # type: ignore
+import pkg_resources
 
 try:
     OptionalRetry = Union[retries.Retry, gapic_v1.method._MethodDefault]
 except AttributeError:  # pragma: NO COVER
     OptionalRetry = Union[retries.Retry, object]  # type: ignore
 
-from google.cloud.iam_credentials_v1.types import common
 from google.protobuf import duration_pb2  # type: ignore
 from google.protobuf import timestamp_pb2  # type: ignore
-from .transports.base import IAMCredentialsTransport, DEFAULT_CLIENT_INFO
+
+from google.cloud.iam_credentials_v1.types import common
+
+from .transports.base import DEFAULT_CLIENT_INFO, IAMCredentialsTransport
 from .transports.grpc import IAMCredentialsGrpcTransport
 from .transports.grpc_asyncio import IAMCredentialsGrpcAsyncIOTransport
 
diff --git a/packages/google-cloud-iam/google/cloud/iam_credentials_v1/services/iam_credentials/transports/__init__.py b/packages/google-cloud-iam/google/cloud/iam_credentials_v1/services/iam_credentials/transports/__init__.py
@@ -20,7 +20,6 @@
 from .grpc import IAMCredentialsGrpcTransport
 from .grpc_asyncio import IAMCredentialsGrpcAsyncIOTransport
 
-
 # Compile a registry of transports.
 _transport_registry = OrderedDict()  # type: Dict[str, Type[IAMCredentialsTransport]]
 _transport_registry["grpc"] = IAMCredentialsGrpcTransport
diff --git a/packages/google-cloud-iam/google/cloud/iam_credentials_v1/services/iam_credentials/transports/base.py b/packages/google-cloud-iam/google/cloud/iam_credentials_v1/services/iam_credentials/transports/base.py
@@ -15,15 +15,15 @@
 #
 import abc
 from typing import Awaitable, Callable, Dict, Optional, Sequence, Union
-import pkg_resources
 
-import google.auth  # type: ignore
 import google.api_core
 from google.api_core import exceptions as core_exceptions
 from google.api_core import gapic_v1
 from google.api_core import retry as retries
+import google.auth  # type: ignore
 from google.auth import credentials as ga_credentials  # type: ignore
 from google.oauth2 import service_account  # type: ignore
+import pkg_resources
 
 from google.cloud.iam_credentials_v1.types import common
 
diff --git a/packages/google-cloud-iam/google/cloud/iam_credentials_v1/services/iam_credentials/transports/grpc.py b/packages/google-cloud-iam/google/cloud/iam_credentials_v1/services/iam_credentials/transports/grpc.py
@@ -13,19 +13,18 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 #
-import warnings
 from typing import Callable, Dict, Optional, Sequence, Tuple, Union
+import warnings
 
-from google.api_core import grpc_helpers
-from google.api_core import gapic_v1
+from google.api_core import gapic_v1, grpc_helpers
 import google.auth  # type: ignore
 from google.auth import credentials as ga_credentials  # type: ignore
 from google.auth.transport.grpc import SslCredentials  # type: ignore
-
 import grpc  # type: ignore
 
 from google.cloud.iam_credentials_v1.types import common
-from .base import IAMCredentialsTransport, DEFAULT_CLIENT_INFO
+
+from .base import DEFAULT_CLIENT_INFO, IAMCredentialsTransport
 
 
 class IAMCredentialsGrpcTransport(IAMCredentialsTransport):
diff --git a/packages/google-cloud-iam/google/cloud/iam_credentials_v1/services/iam_credentials/transports/grpc_asyncio.py b/packages/google-cloud-iam/google/cloud/iam_credentials_v1/services/iam_credentials/transports/grpc_asyncio.py
@@ -13,19 +13,18 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 #
-import warnings
 from typing import Awaitable, Callable, Dict, Optional, Sequence, Tuple, Union
+import warnings
 
-from google.api_core import gapic_v1
-from google.api_core import grpc_helpers_async
+from google.api_core import gapic_v1, grpc_helpers_async
 from google.auth import credentials as ga_credentials  # type: ignore
 from google.auth.transport.grpc import SslCredentials  # type: ignore
-
 import grpc  # type: ignore
 from grpc.experimental import aio  # type: ignore
 
 from google.cloud.iam_credentials_v1.types import common
-from .base import IAMCredentialsTransport, DEFAULT_CLIENT_INFO
+
+from .base import DEFAULT_CLIENT_INFO, IAMCredentialsTransport
 from .grpc import IAMCredentialsGrpcTransport
 
 
diff --git a/packages/google-cloud-iam/google/cloud/iam_credentials_v1/types/common.py b/packages/google-cloud-iam/google/cloud/iam_credentials_v1/types/common.py
@@ -13,11 +13,9 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 #
-import proto  # type: ignore
-
 from google.protobuf import duration_pb2  # type: ignore
 from google.protobuf import timestamp_pb2  # type: ignore
-
+import proto  # type: ignore
 
 __protobuf__ = proto.module(
     package="google.iam.credentials.v1",
diff --git a/packages/google-cloud-iam/noxfile.py b/packages/google-cloud-iam/noxfile.py
@@ -17,6 +17,7 @@
 # Generated by synthtool. DO NOT EDIT!
 
 from __future__ import absolute_import
+
 import os
 import pathlib
 import shutil
@@ -25,7 +26,8 @@
 import nox
 
 BLACK_VERSION = "black==22.3.0"
-BLACK_PATHS = ["docs", "google", "tests", "noxfile.py", "setup.py"]
+ISORT_VERSION = "isort==5.10.1"
+LINT_PATHS = ["docs", "google", "tests", "noxfile.py", "setup.py"]
 
 DEFAULT_PYTHON_VERSION = "3.8"
 
@@ -83,7 +85,7 @@ def lint(session):
     session.run(
         "black",
         "--check",
-        *BLACK_PATHS,
+        *LINT_PATHS,
     )
     session.run("flake8", "google", "tests")
 
@@ -94,7 +96,27 @@ def blacken(session):
     session.install(BLACK_VERSION)
     session.run(
         "black",
-        *BLACK_PATHS,
+        *LINT_PATHS,
+    )
+
+
+@nox.session(python=DEFAULT_PYTHON_VERSION)
+def format(session):
+    """
+    Run isort to sort imports. Then run black
+    to format code to uniform standard.
+    """
+    session.install(BLACK_VERSION, ISORT_VERSION)
+    # Use the --fss option to sort imports using strict alphabetical order.
+    # See https://pycqa.github.io/isort/docs/configuration/options.html#force-sort-within-sections
+    session.run(
+        "isort",
+        "--fss",
+        *LINT_PATHS,
+    )
+    session.run(
+        "black",
+        *LINT_PATHS,
     )
 
 
diff --git a/packages/google-cloud-iam/tests/unit/gapic/credentials_v1/test_iam_credentials.py b/packages/google-cloud-iam/tests/unit/gapic/credentials_v1/test_iam_credentials.py
@@ -13,36 +13,30 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 #
-import os
-import mock
-
-import grpc
-from grpc.experimental import aio
 import math
-import pytest
-from proto.marshal.rules.dates import DurationRule, TimestampRule
-
+import os
 
+from google.api_core import gapic_v1, grpc_helpers, grpc_helpers_async, path_template
 from google.api_core import client_options
 from google.api_core import exceptions as core_exceptions
-from google.api_core import gapic_v1
-from google.api_core import grpc_helpers
-from google.api_core import grpc_helpers_async
-from google.api_core import path_template
+import google.auth
 from google.auth import credentials as ga_credentials
 from google.auth.exceptions import MutualTLSChannelError
+from google.oauth2 import service_account
+from google.protobuf import duration_pb2  # type: ignore
+from google.protobuf import timestamp_pb2  # type: ignore
+import grpc
+from grpc.experimental import aio
+import mock
+from proto.marshal.rules.dates import DurationRule, TimestampRule
+import pytest
+
 from google.cloud.iam_credentials_v1.services.iam_credentials import (
     IAMCredentialsAsyncClient,
-)
-from google.cloud.iam_credentials_v1.services.iam_credentials import (
     IAMCredentialsClient,
+    transports,
 )
-from google.cloud.iam_credentials_v1.services.iam_credentials import transports
 from google.cloud.iam_credentials_v1.types import common
-from google.oauth2 import service_account
-from google.protobuf import duration_pb2  # type: ignore
-from google.protobuf import timestamp_pb2  # type: ignore
-import google.auth
 
 
 def client_cert_source_callback():
diff --git a/packages/google-cloud-iam/tests/unit/gapic/iam_credentials_v1/test_iam_credentials.py b/packages/google-cloud-iam/tests/unit/gapic/iam_credentials_v1/test_iam_credentials.py

Original file line number	Diff line number	Diff line change
`@@ -13,8 +13,8 @@`
`13`	`13`	`# See the License for the specific language governing permissions and`
`14`	`14`	`# limitations under the License.`
`15`	`15`	`#`
`16`		`-from .client import IAMCredentialsClient`
`17`	`16`	`from .async_client import IAMCredentialsAsyncClient`
	`17`	`+from .client import IAMCredentialsClient`
`18`	`18`
`19`	`19`	`__all__ = (`
`20`	`20`	`"IAMCredentialsClient",`