You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository was archived by the owner on Mar 26, 2026. It is now read-only.
In PR googleapis/python-compute#218 unit tests are failing an assertion for assert client.transport._host where there is logic that adds https:// as a prefix to host but the assertions do not reflect this change. See code snippet below and build log here.
def test_vpn_gateways_host_no_port():
client = VpnGatewaysClient(
credentials=ga_credentials.AnonymousCredentials(),
client_options=client_options.ClientOptions(
api_endpoint="compute.googleapis.com"
),
)
> assert client.transport._host == "compute.googleapis.com:443"
E AssertionError: assert 'https://comp...oogleapis.com/' == 'compute.googleapis.com:443'
E - compute.googleapis.com:443
E ? ----
E + https://compute.googleapis.com/
E ? ++++++++
PR https://github.com/googleapis/gapic-generator-python/pull/1148/files introduced a feature to add
https://as a prefix forhost, however the test below fails when there is a rest transport.gapic-generator-python/gapic/templates/tests/unit/gapic/%name_%version/%sub/test_%service.py.j2
Lines 2326 to 2333 in f59917f
In PR googleapis/python-compute#218 unit tests are failing an assertion for
assert client.transport._hostwhere there is logic that addshttps://as a prefix tohostbut the assertions do not reflect this change. See code snippet below and build log here.