chore: fix src lint

Author: jonathanedey

firebase_admin/__init__.py

@@ -79,11 +79,11 @@ def initialize_app(credential=None, options=None, name=_DEFAULT_APP_NAME):
             'apps, pass a second argument to initialize_app() to give each app '
             'a unique name.'))
 
-    raise ValueError((
-        'Firebase app named "{0}" already exists. This means you called '
+    raise ValueError(
+        f'Firebase app named "{name}" already exists. This means you called '
         'initialize_app() more than once with the same app name as the '
         'second argument. Make sure you provide a unique name every time '
-        'you call initialize_app().').format(name))
+        'you call initialize_app().')
 
 
 def delete_app(app):
@@ -96,8 +96,7 @@ def delete_app(app):
       ValueError: If the app is not initialized.
     """
     if not isinstance(app, App):
-        raise ValueError('Illegal app argument type: "{}". Argument must be of '
-                         'type App.'.format(type(app)))
+        raise ValueError(f'Illegal app argument type: "{type(app)}". Argument must be of type App.')
     with _apps_lock:
         if _apps.get(app.name) is app:
             del _apps[app.name]
@@ -109,9 +108,9 @@ def delete_app(app):
             'the default app by calling initialize_app().')
 
     raise ValueError(
-        ('Firebase app named "{0}" is not initialized. Make sure to initialize '
-         'the app by calling initialize_app() with your app name as the '
-         'second argument.').format(app.name))
+        f'Firebase app named "{app.name}" is not initialized. Make sure to initialize '
+        'the app by calling initialize_app() with your app name as the '
+        'second argument.')
 
 
 def get_app(name=_DEFAULT_APP_NAME):
@@ -128,8 +127,8 @@ def get_app(name=_DEFAULT_APP_NAME):
           app does not exist.
     """
     if not isinstance(name, str):
-        raise ValueError('Illegal app name argument type: "{}". App name '
-                         'must be a string.'.format(type(name)))
+        raise ValueError(
+            f'Illegal app name argument type: "{type(name)}". App name must be a string.')
     with _apps_lock:
         if name in _apps:
             return _apps[name]
@@ -140,9 +139,9 @@ def get_app(name=_DEFAULT_APP_NAME):
             'the SDK by calling initialize_app().')
 
     raise ValueError(
-        ('Firebase app named "{0}" does not exist. Make sure to initialize '
-         'the SDK by calling initialize_app() with your app name as the '
-         'second argument.').format(name))
+        f'Firebase app named "{name}" does not exist. Make sure to initialize '
+        'the SDK by calling initialize_app() with your app name as the '
+        'second argument.')
 
 
 class _AppOptions:
@@ -153,8 +152,9 @@ def __init__(self, options):
             options = self._load_from_environment()
 
         if not isinstance(options, dict):
-            raise ValueError('Illegal Firebase app options type: {0}. Options '
-                             'must be a dictionary.'.format(type(options)))
+            raise ValueError(
+                f'Illegal Firebase app options type: {type(options)}. '
+                'Options must be a dictionary.')
         self._options = options
 
     def get(self, key, default=None):
@@ -175,15 +175,15 @@ def _load_from_environment(self):
             json_str = config_file
         else:
             try:
-                with open(config_file, 'r') as json_file:
+                with open(config_file, 'r', encoding='utf-8') as json_file:
                     json_str = json_file.read()
             except Exception as err:
-                raise ValueError('Unable to read file {}. {}'.format(config_file, err)) from err
+                raise ValueError(f'Unable to read file {config_file}. {err}') from err
         try:
             json_data = json.loads(json_str)
         except Exception as err:
             raise ValueError(
-                'JSON string "{0}" is not valid json. {1}'.format(json_str, err)) from err
+                f'JSON string "{json_str}" is not valid json. {err}') from err
         return {k: v for k, v in json_data.items() if k in _CONFIG_VALID_KEYS}
 
 
@@ -206,8 +206,9 @@ def __init__(self, name, credential, options):
           ValueError: If an argument is None or invalid.
         """
         if not name or not isinstance(name, str):
-            raise ValueError('Illegal Firebase app name "{0}" provided. App name must be a '
-                             'non-empty string.'.format(name))
+            raise ValueError(
+                f'Illegal Firebase app name "{name}" provided. App name must be a '
+                'non-empty string.')
         self._name = name
 
         if isinstance(credential, GoogleAuthCredentials):
@@ -228,7 +229,7 @@ def __init__(self, name, credential, options):
     def _validate_project_id(cls, project_id):
         if project_id is not None and not isinstance(project_id, str):
             raise ValueError(
-                'Invalid project ID: "{0}". project ID must be a string.'.format(project_id))
+                f'Invalid project ID: "{project_id}". project ID must be a string.')
 
     @property
     def name(self):
@@ -293,11 +294,11 @@ def _get_service(self, name, initializer):
         """
         if not name or not isinstance(name, str):
             raise ValueError(
-                'Illegal name argument: "{0}". Name must be a non-empty string.'.format(name))
+                f'Illegal name argument: "{name}". Name must be a non-empty string.')
         with self._lock:
             if self._services is None:
                 raise ValueError(
-                    'Service requested from deleted Firebase App: "{0}".'.format(self._name))
+                    f'Service requested from deleted Firebase App: "{self._name}".')
             if name not in self._services:
                 self._services[name] = initializer(self)
             return self._services[name]

firebase_admin/_auth_client.py

@@ -38,7 +38,7 @@ def __init__(self, app, tenant_id=None):
             3. set the project ID via the GOOGLE_CLOUD_PROJECT environment variable.""")
 
         credential = None
-        version_header = 'Python/Admin/{0}'.format(firebase_admin.__version__)
+        version_header = f'Python/Admin/{firebase_admin.__version__}'
         timeout = app.options.get('httpTimeout', _http_client.DEFAULT_TIMEOUT_SECONDS)
         # Non-default endpoint URLs for emulator support are set in this dict later.
         endpoint_urls = {}
@@ -48,7 +48,7 @@ def __init__(self, app, tenant_id=None):
         # endpoint URLs to use the emulator. Additionally, use a fake credential.
         emulator_host = _auth_utils.get_emulator_host()
         if emulator_host:
-            base_url = 'http://{0}/identitytoolkit.googleapis.com'.format(emulator_host)
+            base_url = f'http://{emulator_host}/identitytoolkit.googleapis.com'
             endpoint_urls['v1'] = base_url + '/v1'
             endpoint_urls['v2'] = base_url + '/v2'
             credential = _utils.EmulatorAdminCredentials()
@@ -123,15 +123,16 @@ def verify_id_token(self, id_token, check_revoked=False, clock_skew_seconds=0):
         """
         if not isinstance(check_revoked, bool):
             # guard against accidental wrong assignment.
-            raise ValueError('Illegal check_revoked argument. Argument must be of type '
-                             ' bool, but given "{0}".'.format(type(check_revoked)))
+            raise ValueError(
+                'Illegal check_revoked argument. Argument must be of type bool, but given '
+                f'"{type(check_revoked)}".')
 
         verified_claims = self._token_verifier.verify_id_token(id_token, clock_skew_seconds)
         if self.tenant_id:
             token_tenant_id = verified_claims.get('firebase', {}).get('tenant')
             if self.tenant_id != token_tenant_id:
                 raise _auth_utils.TenantIdMismatchError(
-                    'Invalid tenant ID: {0}'.format(token_tenant_id))
+                    f'Invalid tenant ID: {token_tenant_id}')
 
         if check_revoked:
             self._check_jwt_revoked_or_disabled(
@@ -249,7 +250,7 @@ def _matches(identifier, user_record):
                     if identifier.provider_id == user_info.provider_id
                     and identifier.provider_uid == user_info.uid
                 ), False)
-            raise TypeError("Unexpected type: {}".format(type(identifier)))
+            raise TypeError(f"Unexpected type: {type(identifier)}")
 
         def _is_user_found(identifier, user_records):
             return any(_matches(identifier, user_record) for user_record in user_records)
@@ -757,4 +758,4 @@ def _check_jwt_revoked_or_disabled(self, verified_claims, exc_type, label):
         if user.disabled:
             raise _auth_utils.UserDisabledError('The user record is disabled.')
         if verified_claims.get('iat') * 1000 < user.tokens_valid_after_timestamp:
-            raise exc_type('The Firebase {0} has been revoked.'.format(label))
+            raise exc_type(f'The Firebase {label} has been revoked.')

firebase_admin/_auth_providers.py

@@ -181,13 +181,13 @@ class ProviderConfigClient:
     def __init__(self, http_client, project_id, tenant_id=None, url_override=None):
         self.http_client = http_client
         url_prefix = url_override or self.PROVIDER_CONFIG_URL
-        self.base_url = '{0}/projects/{1}'.format(url_prefix, project_id)
+        self.base_url = f'{url_prefix}/projects/{project_id}'
         if tenant_id:
-            self.base_url += '/tenants/{0}'.format(tenant_id)
+            self.base_url += f'/tenants/{tenant_id}'
 
     def get_oidc_provider_config(self, provider_id):
         _validate_oidc_provider_id(provider_id)
-        body = self._make_request('get', '/oauthIdpConfigs/{0}'.format(provider_id))
+        body = self._make_request('get', f'/oauthIdpConfigs/{provider_id}')
         return OIDCProviderConfig(body)
 
     def create_oidc_provider_config(
@@ -218,7 +218,7 @@ def create_oidc_provider_config(
         if response_type:
             req['responseType'] = response_type
 
-        params = 'oauthIdpConfigId={0}'.format(provider_id)
+        params = f'oauthIdpConfigId={provider_id}'
         body = self._make_request('post', '/oauthIdpConfigs', json=req, params=params)
         return OIDCProviderConfig(body)
 
@@ -259,14 +259,14 @@ def update_oidc_provider_config(
             raise ValueError('At least one parameter must be specified for update.')
 
         update_mask = _auth_utils.build_update_mask(req)
-        params = 'updateMask={0}'.format(','.join(update_mask))
-        url = '/oauthIdpConfigs/{0}'.format(provider_id)
+        params = f'updateMask={",".join(update_mask)}'
+        url = f'/oauthIdpConfigs/{provider_id}'
         body = self._make_request('patch', url, json=req, params=params)
         return OIDCProviderConfig(body)
 
     def delete_oidc_provider_config(self, provider_id):
         _validate_oidc_provider_id(provider_id)
-        self._make_request('delete', '/oauthIdpConfigs/{0}'.format(provider_id))
+        self._make_request('delete', f'/oauthIdpConfigs/{provider_id}')
 
     def list_oidc_provider_configs(self, page_token=None, max_results=MAX_LIST_CONFIGS_RESULTS):
         return _ListOIDCProviderConfigsPage(
@@ -277,7 +277,7 @@ def _fetch_oidc_provider_configs(self, page_token=None, max_results=MAX_LIST_CON
 
     def get_saml_provider_config(self, provider_id):
         _validate_saml_provider_id(provider_id)
-        body = self._make_request('get', '/inboundSamlConfigs/{0}'.format(provider_id))
+        body = self._make_request('get', f'/inboundSamlConfigs/{provider_id}')
         return SAMLProviderConfig(body)
 
     def create_saml_provider_config(
@@ -301,7 +301,7 @@ def create_saml_provider_config(
         if enabled is not None:
             req['enabled'] = _auth_utils.validate_boolean(enabled, 'enabled')
 
-        params = 'inboundSamlConfigId={0}'.format(provider_id)
+        params = f'inboundSamlConfigId={provider_id}'
         body = self._make_request('post', '/inboundSamlConfigs', json=req, params=params)
         return SAMLProviderConfig(body)
 
@@ -341,14 +341,14 @@ def update_saml_provider_config(
             raise ValueError('At least one parameter must be specified for update.')
 
         update_mask = _auth_utils.build_update_mask(req)
-        params = 'updateMask={0}'.format(','.join(update_mask))
-        url = '/inboundSamlConfigs/{0}'.format(provider_id)
+        params = f'updateMask={",".join(update_mask)}'
+        url = f'/inboundSamlConfigs/{provider_id}'
         body = self._make_request('patch', url, json=req, params=params)
         return SAMLProviderConfig(body)
 
     def delete_saml_provider_config(self, provider_id):
         _validate_saml_provider_id(provider_id)
-        self._make_request('delete', '/inboundSamlConfigs/{0}'.format(provider_id))
+        self._make_request('delete', f'/inboundSamlConfigs/{provider_id}')
 
     def list_saml_provider_configs(self, page_token=None, max_results=MAX_LIST_CONFIGS_RESULTS):
         return _ListSAMLProviderConfigsPage(
@@ -367,15 +367,15 @@ def _fetch_provider_configs(self, path, page_token=None, max_results=MAX_LIST_CO
         if max_results < 1 or max_results > MAX_LIST_CONFIGS_RESULTS:
             raise ValueError(
                 'Max results must be a positive integer less than or equal to '
-                '{0}.'.format(MAX_LIST_CONFIGS_RESULTS))
+                f'{MAX_LIST_CONFIGS_RESULTS}.')
 
-        params = 'pageSize={0}'.format(max_results)
+        params = f'pageSize={max_results}'
         if page_token:
-            params += '&pageToken={0}'.format(page_token)
+            params += f'&pageToken={page_token}'
         return self._make_request('get', path, params=params)
 
     def _make_request(self, method, path, **kwargs):
-        url = '{0}{1}'.format(self.base_url, path)
+        url = f'{self.base_url}{path}'
         try:
             return self.http_client.body(method, url, **kwargs)
         except requests.exceptions.RequestException as error:
@@ -385,45 +385,42 @@ def _make_request(self, method, path, **kwargs):
 def _validate_oidc_provider_id(provider_id):
     if not isinstance(provider_id, str):
         raise ValueError(
-            'Invalid OIDC provider ID: {0}. Provider ID must be a non-empty string.'.format(
-                provider_id))
+            f'Invalid OIDC provider ID: {provider_id}. Provider ID must be a non-empty string.')
     if not provider_id.startswith('oidc.'):
-        raise ValueError('Invalid OIDC provider ID: {0}.'.format(provider_id))
+        raise ValueError(f'Invalid OIDC provider ID: {provider_id}.')
     return provider_id
 
 
 def _validate_saml_provider_id(provider_id):
     if not isinstance(provider_id, str):
         raise ValueError(
-            'Invalid SAML provider ID: {0}. Provider ID must be a non-empty string.'.format(
-                provider_id))
+            f'Invalid SAML provider ID: {provider_id}. Provider ID must be a non-empty string.')
     if not provider_id.startswith('saml.'):
-        raise ValueError('Invalid SAML provider ID: {0}.'.format(provider_id))
+        raise ValueError(f'Invalid SAML provider ID: {provider_id}.')
     return provider_id
 
 
 def _validate_non_empty_string(value, label):
     """Validates that the given value is a non-empty string."""
     if not isinstance(value, str):
-        raise ValueError('Invalid type for {0}: {1}.'.format(label, value))
+        raise ValueError(f'Invalid type for {label}: {value}.')
     if not value:
-        raise ValueError('{0} must not be empty.'.format(label))
+        raise ValueError(f'{label} must not be empty.')
     return value
 
 
 def _validate_url(url, label):
     """Validates that the given value is a well-formed URL string."""
     if not isinstance(url, str) or not url:
         raise ValueError(
-            'Invalid photo URL: "{0}". {1} must be a non-empty '
-            'string.'.format(url, label))
+            f'Invalid photo URL: "{url}". {label} must be a non-empty string.')
     try:
         parsed = parse.urlparse(url)
         if not parsed.netloc:
-            raise ValueError('Malformed {0}: "{1}".'.format(label, url))
+            raise ValueError(f'Malformed {label}: "{url}".')
         return url
     except Exception as exception:
-        raise ValueError('Malformed {0}: "{1}".'.format(label, url)) from exception
+        raise ValueError(f'Malformed {label}: "{url}".') from exception
 
 
 def _validate_x509_certificates(x509_certificates):