refactor(user): interface-ize UserService and implement with userService

- Define UserService interface for better abstraction and testability- Create userService struct to implement UserService interface
- Update client.go to use new UserService interface and NewUserService function
- Improve code organization and readability in api_user.go

Signed-off-by: Flc <[email protected]>

Author: flc1125

api_user.go

@@ -5,25 +5,38 @@ import (
 	"net/http"
 )
 
-type UserRole struct {
-	ID   string `json:"id"`
-	Name string `json:"name"`
+type (
+	UserRole struct {
+		ID   string `json:"id"`
+		Name string `json:"name"`
+	}
+
+	// GetRolesRequest represents a request to get roles
+	GetRolesRequest struct {
+		WorkspaceID *int `url:"workspace_id,omitempty"` // 项目 ID
+	}
+)
+
+type UserService interface {
+	// GetRoles 获取角色ID对照关系
+	//
+	// https://open.tapd.cn/document/api-doc/API%E6%96%87%E6%A1%A3/api_reference/user/get_roles.html
+	GetRoles(ctx context.Context, request *GetRolesRequest, opts ...RequestOption) ([]*UserRole, *Response, error)
 }
 
-// UserService is a service to interact with user related API
-type UserService struct {
+type userService struct {
 	client *Client
 }
 
-// GetRolesRequest represents a request to get roles
-type GetRolesRequest struct {
-	WorkspaceID *int `url:"workspace_id,omitempty"` // 项目 ID
+var _ UserService = (*userService)(nil)
+
+func NewUserService(client *Client) UserService {
+	return &userService{
+		client: client,
+	}
 }
 
-// GetRoles 获取角色ID对照关系
-//
-// https://open.tapd.cn/document/api-doc/API%E6%96%87%E6%A1%A3/api_reference/user/get_roles.html
-func (s *UserService) GetRoles(
+func (s *userService) GetRoles(
 	ctx context.Context, request *GetRolesRequest, opts ...RequestOption,
 ) ([]*UserRole, *Response, error) {
 	req, err := s.client.NewRequest(ctx, http.MethodGet, "roles", request, opts)

client.go

@@ -45,7 +45,7 @@ type Client struct {
 	WorkspaceService  *WorkspaceService
 	LabelService      *LabelService
 	MeasureService    *MeasureService
-	UserService       *UserService
+	UserService       UserService
 	WorkflowService   *WorkflowService
 	SettingService    *SettingService
 }
@@ -91,7 +91,7 @@ func newClient(opts ...ClientOption) (*Client, error) {
 	c.WorkspaceService = &WorkspaceService{client: c}
 	c.LabelService = &LabelService{client: c}
 	c.MeasureService = &MeasureService{client: c}
-	c.UserService = &UserService{client: c}
+	c.UserService = NewUserService(c)
 	c.WorkflowService = &WorkflowService{client: c}
 	c.SettingService = &SettingService{client: c}