chore: Correct `x-goog-api-client` header logic by jonathanedey · Pull Request #876 · firebase/firebase-admin-python
Expand Up
@@ -197,7 +197,8 @@ def test_get_tenant(self, tenant_mgt_app):
assert req.method == 'GET'
assert req.url == '{0}/tenants/tenant-id'.format(TENANT_MGT_URL_PREFIX)
assert req.headers['X-Client-Version'] == f'Python/Admin/{firebase_admin.__version__}'
assert req.headers['X-GOOG-API-CLIENT'] == _utils.get_metrics_header()
expected_metrics_header = _utils.get_metrics_header() + ' mock-cred-metric-tag'
assert req.headers['x-goog-api-client'] == expected_metrics_header
def test_tenant_not_found(self, tenant_mgt_app): _instrument_tenant_mgt(tenant_mgt_app, 500, TENANT_NOT_FOUND_RESPONSE) Expand Down Expand Up @@ -289,7 +290,8 @@ def _assert_request(self, recorder, body): assert req.method == 'POST' assert req.url == '{0}/tenants'.format(TENANT_MGT_URL_PREFIX) assert req.headers['X-Client-Version'] == f'Python/Admin/{firebase_admin.__version__}' assert req.headers['X-GOOG-API-CLIENT'] == _utils.get_metrics_header() expected_metrics_header = _utils.get_metrics_header() + ' mock-cred-metric-tag' assert req.headers['x-goog-api-client'] == expected_metrics_header got = json.loads(req.body.decode()) assert got == body
Expand Down Expand Up @@ -389,7 +391,8 @@ def _assert_request(self, recorder, body, mask): assert req.url == '{0}/tenants/tenant-id?updateMask={1}'.format( TENANT_MGT_URL_PREFIX, ','.join(mask)) assert req.headers['X-Client-Version'] == f'Python/Admin/{firebase_admin.__version__}' assert req.headers['X-GOOG-API-CLIENT'] == _utils.get_metrics_header() expected_metrics_header = _utils.get_metrics_header() + ' mock-cred-metric-tag' assert req.headers['x-goog-api-client'] == expected_metrics_header got = json.loads(req.body.decode()) assert got == body
Expand All @@ -411,7 +414,8 @@ def test_delete_tenant(self, tenant_mgt_app): assert req.method == 'DELETE' assert req.url == '{0}/tenants/tenant-id'.format(TENANT_MGT_URL_PREFIX) assert req.headers['X-Client-Version'] == f'Python/Admin/{firebase_admin.__version__}' assert req.headers['X-GOOG-API-CLIENT'] == _utils.get_metrics_header() expected_metrics_header = _utils.get_metrics_header() + ' mock-cred-metric-tag' assert req.headers['x-goog-api-client'] == expected_metrics_header
def test_tenant_not_found(self, tenant_mgt_app): _instrument_tenant_mgt(tenant_mgt_app, 500, TENANT_NOT_FOUND_RESPONSE) Expand Down Expand Up @@ -555,7 +559,8 @@ def _assert_request(self, recorder, expected=None): req = recorder[0] assert req.method == 'GET' assert req.headers['X-Client-Version'] == f'Python/Admin/{firebase_admin.__version__}' assert req.headers['X-GOOG-API-CLIENT'] == _utils.get_metrics_header() expected_metrics_header = _utils.get_metrics_header() + ' mock-cred-metric-tag' assert req.headers['x-goog-api-client'] == expected_metrics_header request = dict(parse.parse_qsl(parse.urlsplit(req.url).query)) assert request == expected
Expand Down Expand Up @@ -932,7 +937,8 @@ def _assert_request( assert req.method == method assert req.url == '{0}/tenants/tenant-id{1}'.format(prefix, want_url) assert req.headers['X-Client-Version'] == f'Python/Admin/{firebase_admin.__version__}' assert req.headers['X-GOOG-API-CLIENT'] == _utils.get_metrics_header() expected_metrics_header = _utils.get_metrics_header() + ' mock-cred-metric-tag' assert req.headers['x-goog-api-client'] == expected_metrics_header body = json.loads(req.body.decode()) assert body == want_body
Expand Down
def test_tenant_not_found(self, tenant_mgt_app): _instrument_tenant_mgt(tenant_mgt_app, 500, TENANT_NOT_FOUND_RESPONSE) Expand Down Expand Up @@ -289,7 +290,8 @@ def _assert_request(self, recorder, body): assert req.method == 'POST' assert req.url == '{0}/tenants'.format(TENANT_MGT_URL_PREFIX) assert req.headers['X-Client-Version'] == f'Python/Admin/{firebase_admin.__version__}' assert req.headers['X-GOOG-API-CLIENT'] == _utils.get_metrics_header() expected_metrics_header = _utils.get_metrics_header() + ' mock-cred-metric-tag' assert req.headers['x-goog-api-client'] == expected_metrics_header got = json.loads(req.body.decode()) assert got == body
Expand Down Expand Up @@ -389,7 +391,8 @@ def _assert_request(self, recorder, body, mask): assert req.url == '{0}/tenants/tenant-id?updateMask={1}'.format( TENANT_MGT_URL_PREFIX, ','.join(mask)) assert req.headers['X-Client-Version'] == f'Python/Admin/{firebase_admin.__version__}' assert req.headers['X-GOOG-API-CLIENT'] == _utils.get_metrics_header() expected_metrics_header = _utils.get_metrics_header() + ' mock-cred-metric-tag' assert req.headers['x-goog-api-client'] == expected_metrics_header got = json.loads(req.body.decode()) assert got == body
Expand All @@ -411,7 +414,8 @@ def test_delete_tenant(self, tenant_mgt_app): assert req.method == 'DELETE' assert req.url == '{0}/tenants/tenant-id'.format(TENANT_MGT_URL_PREFIX) assert req.headers['X-Client-Version'] == f'Python/Admin/{firebase_admin.__version__}' assert req.headers['X-GOOG-API-CLIENT'] == _utils.get_metrics_header() expected_metrics_header = _utils.get_metrics_header() + ' mock-cred-metric-tag' assert req.headers['x-goog-api-client'] == expected_metrics_header
def test_tenant_not_found(self, tenant_mgt_app): _instrument_tenant_mgt(tenant_mgt_app, 500, TENANT_NOT_FOUND_RESPONSE) Expand Down Expand Up @@ -555,7 +559,8 @@ def _assert_request(self, recorder, expected=None): req = recorder[0] assert req.method == 'GET' assert req.headers['X-Client-Version'] == f'Python/Admin/{firebase_admin.__version__}' assert req.headers['X-GOOG-API-CLIENT'] == _utils.get_metrics_header() expected_metrics_header = _utils.get_metrics_header() + ' mock-cred-metric-tag' assert req.headers['x-goog-api-client'] == expected_metrics_header request = dict(parse.parse_qsl(parse.urlsplit(req.url).query)) assert request == expected
Expand Down Expand Up @@ -932,7 +937,8 @@ def _assert_request( assert req.method == method assert req.url == '{0}/tenants/tenant-id{1}'.format(prefix, want_url) assert req.headers['X-Client-Version'] == f'Python/Admin/{firebase_admin.__version__}' assert req.headers['X-GOOG-API-CLIENT'] == _utils.get_metrics_header() expected_metrics_header = _utils.get_metrics_header() + ' mock-cred-metric-tag' assert req.headers['x-goog-api-client'] == expected_metrics_header body = json.loads(req.body.decode()) assert body == want_body
Expand Down