redundant function fix

Author: aayushsingh2502

src/tfe/resources/oauth_client.py

@@ -15,7 +15,6 @@
     OAuthClientUpdateOptions,
 )
 from ..utils import (
-    valid_oauth_client_id,
     valid_string_id,
     validate_oauth_client_add_projects_options,
     validate_oauth_client_create_options,
@@ -89,7 +88,7 @@ def read_with_options(
         self, oauth_client_id: str, options: OAuthClientReadOptions | None
     ) -> OAuthClient:
         """Read an OAuth client by its ID with options."""
-        if not valid_oauth_client_id(oauth_client_id):
+        if not valid_string_id(oauth_client_id):
             raise ValueError(ERR_INVALID_OAUTH_CLIENT_ID)
 
         path = f"/api/v2/oauth-clients/{quote(oauth_client_id)}"
@@ -107,7 +106,7 @@ def update(
         self, oauth_client_id: str, options: OAuthClientUpdateOptions
     ) -> OAuthClient:
         """Update an OAuth client by its ID."""
-        if not valid_oauth_client_id(oauth_client_id):
+        if not valid_string_id(oauth_client_id):
             raise ValueError(ERR_INVALID_OAUTH_CLIENT_ID)
 
         body = {
@@ -129,7 +128,7 @@ def update(
 
     def delete(self, oauth_client_id: str) -> None:
         """Delete an OAuth client by its ID."""
-        if not valid_oauth_client_id(oauth_client_id):
+        if not valid_string_id(oauth_client_id):
             raise ValueError(ERR_INVALID_OAUTH_CLIENT_ID)
 
         path = f"/api/v2/oauth-clients/{quote(oauth_client_id)}"
@@ -139,7 +138,7 @@ def add_projects(
         self, oauth_client_id: str, options: OAuthClientAddProjectsOptions
     ) -> None:
         """Add projects to a given OAuth client."""
-        if not valid_oauth_client_id(oauth_client_id):
+        if not valid_string_id(oauth_client_id):
             raise ValueError(ERR_INVALID_OAUTH_CLIENT_ID)
 
         validate_oauth_client_add_projects_options(options)
@@ -151,7 +150,7 @@ def remove_projects(
         self, oauth_client_id: str, options: OAuthClientRemoveProjectsOptions
     ) -> None:
         """Remove projects from an OAuth client."""
-        if not valid_oauth_client_id(oauth_client_id):
+        if not valid_string_id(oauth_client_id):
             raise ValueError(ERR_INVALID_OAUTH_CLIENT_ID)
 
         validate_oauth_client_remove_projects_options(options)

src/tfe/utils.py

@@ -214,11 +214,6 @@ def validate_workspace_update_options(options: WorkspaceUpdateOptions) -> None:
             raise UnsupportedBothTagsRegexAndFileTriggersEnabledError()
 
 
-def valid_oauth_client_id(v: str | None) -> bool:
-    """Validate OAuth client ID format."""
-    return valid_string_id(v)
-
-
 def validate_oauth_client_create_options(options: OAuthClientCreateOptions) -> None:
     """
     Validate OAuth client create options similar to Go implementation.