Anthonios Partheniou

Add `fail_n_times` field to add ability to configure a request to fail x number of times before receiving a valid response

2

Proposal: Add a `Showcase.Echo` RPC with a `int fail_n_times` field that initially fails n number of times before returning a valid response. This will allow us to test the following...

Unit tests fail with `cryptography >= 39.0.0` when using python 3.7

1

See build log [here](https://btx-internal.corp.google.com/invocations/9fa007b9-470b-41b4-8129-43aa2eb9922d/log) where python 3.7 unit tests fail when using `cryptography >= 39.0.0` ``` ============================= test session starts ============================== platform linux -- Python 3.7.17, pytest-7.4.3, pluggy-1.2.0 rootdir: /tmpfs/src/github/google-auth-library-python...

DeprecationWarning: Inheritance class AuthorizedSession from ClientSession is discouraged

1

See the build log [here](https://source.cloud.google.com/results/invocations/e335a328-5ba0-4532-8daa-7e15d2383de1/targets) which contains the warning below. ``` google/auth/transport/_aiohttp_requests.py:200 /tmpfs/src/github/google-auth-library-python/google/auth/transport/_aiohttp_requests.py:200: DeprecationWarning: Inheritance class AuthorizedSession from ClientSession is discouraged class AuthorizedSession(aiohttp.ClientSession): ``` This issue tracks fixing the underlying...

See the `Kokoro` build log which contains the following warning: ``` tests_async/oauth2/test__client_async.py::test_refresh_grant_retry_with_retry[False] /tmpfs/src/github/google-auth-library-python/.nox/unit-3-7/lib/python3.7/site-packages/mock/mock.py:1469: RuntimeWarning: coroutine 'AsyncMockMixin._execute_mock_call' was never awaited return await func(*newargs, **newkeywargs) ```

Add an environment that tests code against required dependencies only

1

There was a bug introduced in https://github.com/googleapis/google-auth-library-python/pull/948 where `urllib3` was introduced as a dependency without being declared in setup.py [here](https://github.com/googleapis/google-auth-library-python/blob/main/setup.py#L22). This results in `ModuleNotFoundError: No module named 'urllib3'` in environments...

Release notes should indicate when new API versions are added

1

`dataportability` `v1beta` was [added](https://github.com/googleapis/google-api-python-client/commits/main/googleapiclient/discovery_cache/documents/dataportability.v1beta.json) last week, and shipped in [v2.117.0](https://github.com/googleapis/google-api-python-client/releases/tag/v2.117.0). The release notes for v2.117.0 don't mention that a new API/version was added which led to issue https://github.com/googleapis/google-api-python-client/issues/2336.

Towards b/330986358

Fixes warning from `flake8 . --select E721` ``` (py39) partheniou@partheniou-vm-2:~/git/google-api-python-client$ flake8 . --select E721 ./googleapiclient/model.py:177:28: E721 do not compare types, use 'isinstance()' ./googleapiclient/model.py:396:37: E721 do not compare types, use 'isinstance()'...

Fixes warning from `flake8 . --select E712` ``` (py39) partheniou@partheniou-vm-2:~/git/google-api-python-client$ flake8 . --select E712 ./scripts/buildprbody.py:83:40: E712 comparison to False should be 'if cond is False:' or 'if not cond:' ./scripts/changesummary.py:211:46:...

Fixes warning from `flake8 . --select F841` ``` (py39) partheniou@partheniou-vm-2:~/git/google-api-python-client$ flake8 . --select F841 ./describe.py:310:5: F841 local variable 'resource_name' is assigned to but never used ./describe.py:350:13: F841 local variable 'dname'...