add team management features

Signed-off-by: Maximilian <burr.maximilian97@gmail.com>

Author: DerBurri

api/team.go

@@ -104,3 +104,23 @@ func (c *Client) DeleteTeam(orgId string, teamId int) error {
 	r, err := c.api.TeamsAPI.TeamsDeleteTeam(c.ctx, float32(teamId)).Execute()
 	return cserrors.FormatAPIError(r, err)
 }
+
+func (c *Client) AddTeamMember(teamId int, email string, role int) error {
+	r, err := c.api.TeamsAPI.TeamsInviteMember(c.ctx, float32(teamId)).
+		TeamsInviteMemberRequest(openapi_client.TeamsInviteMemberRequest{
+			UserEmail: email,
+			Role:      role,
+		}).Execute()
+	if err != nil {
+		return cserrors.FormatAPIError(r, err)
+	}
+	return nil
+}
+
+func (c *Client) RemoveTeamMember(teamId int, userId int) error {
+	r, err := c.api.TeamsAPI.TeamsRemoveMember(c.ctx, float32(teamId), float32(userId)).Execute()
+	if err != nil {
+		return cserrors.FormatAPIError(r, err)
+	}
+	return nil
+}

cli/cmd/client.go

@@ -36,6 +36,8 @@ type Client interface {
 	DeployLandscape(wsId int, profile string) error
 	CreateTeam(orgId string, name string, dcId int) (*api.Team, error)
 	DeleteTeam(orgId string, teamId int) error
+	AddTeamMember(teamId int, email string, role int) error
+	RemoveTeamMember(teamId int, userId int) error
 }
 
 // CommandExecutor abstracts command execution for testing

cli/cmd/mocks.go

@@ -4,12 +4,23 @@
 package cmd
 
 import (
+	"errors"
+	"fmt"
+
 	"github.com/spf13/cobra"
 )
 
 type AddTeamMemberCmd struct {
-	cmd  *cobra.Command
-	Opts CreateWorkspaceOpts
+	cmd           *cobra.Command
+	Opts          AddTeamMemberOpts
+	ClientFactory func(GlobalOptions) (Client, error)
+}
+
+type AddTeamMemberOpts struct {
+	*GlobalOptions
+	Email  string
+	Role   int
+	TeamId int
 }
 
 func AddAddTeamMemberCmd(team *cobra.Command, opts *GlobalOptions) {
@@ -19,14 +30,44 @@ func AddAddTeamMemberCmd(team *cobra.Command, opts *GlobalOptions) {
 			Short: "Add team member",
 			Long:  `Add team member to a team`,
 		},
-		Opts: CreateWorkspaceOpts{
+		Opts: AddTeamMemberOpts{
 			GlobalOptions: opts,
 		},
+		ClientFactory: NewClient,
 	}
 	t.cmd.RunE = t.RunE
+	t.cmd.Flags().StringVarP(&t.Opts.Email, "email", "e", "", "Team member email")
+	t.cmd.Flags().IntVarP(&t.Opts.Role, "role", "r", 0, "Team member role 0=admin, 1=member")
 	AddCmd(team, t.cmd)
 }
 
 func (c *AddTeamMemberCmd) RunE(_ *cobra.Command, args []string) error {
+	client, err := c.ClientFactory(*c.Opts.GlobalOptions)
+	if err != nil {
+		return fmt.Errorf("failed to create Codespehre client: %w", err)
+	}
+
+	teamId, err := c.Opts.GetTeamId()
+	if err != nil {
+		return err
+	}
+
+	err = c.AddTeamMember(client, teamId, c.Opts.Email, c.Opts.Role)
+	return err
+
+}
+
+func (c *AddTeamMemberCmd) AddTeamMember(client Client, teamId int, email string, role int) error {
+	if email == "" {
+		return errors.New("email cannot be empty")
+	}
+
+	fmt.Printf("add member: %s to team %d with role: %d", email, teamId, role)
+
+	err := client.AddTeamMember(teamId, email, role)
+	if err != nil {
+		return fmt.Errorf("failed to add member to team: %w", err)
+	}
+
 	return nil
 }

cli/cmd/team_add_member.go

@@ -0,0 +1,105 @@
+// Copyright (c) Codesphere Inc.
+// SPDX-License-Identifier: Apache-2.0
+
+package cmd_test
+
+import (
+	"errors"
+
+	"github.com/codesphere-cloud/cs-go/cli/cmd"
+	. "github.com/onsi/ginkgo/v2"
+	. "github.com/onsi/gomega"
+)
+
+var _ = Describe("AddTeamMember", func() {
+	var (
+		mockEnv    *cmd.MockEnv
+		mockClient *cmd.MockClient
+		c          *cmd.AddTeamMemberCmd
+		teamId     int
+		dcId       int
+		email      string
+	)
+
+	BeforeEach(func() {
+		mockClient = cmd.NewMockClient(GinkgoT())
+		mockEnv = cmd.NewMockEnv(GinkgoT())
+		teamId = 42
+		email = "test@test.com"
+		dcId = 1 // Default data center ID for testing
+		c = &cmd.AddTeamMemberCmd{
+			Opts: cmd.AddTeamMemberOpts{
+				GlobalOptions: &cmd.GlobalOptions{
+					Env:    mockEnv,
+					TeamId: teamId,
+					// OrgId is intentionally left empty here, will be set in BeforeEach for specific contexts
+				},
+				Email:  email,
+				TeamId: teamId,
+			},
+			ClientFactory: func(opts cmd.GlobalOptions) (cmd.Client, error) {
+				return mockClient, nil
+			},
+		}
+		// Mock common environment calls needed for client creation
+	})
+
+	AfterEach(func() {
+		mockEnv.AssertExpectations(GinkgoT())
+		mockClient.AssertExpectations(GinkgoT())
+	})
+
+	Context("Validation", func() {
+		It("should fail if the mail is empty", func() {
+
+			err := c.AddTeamMember(mockClient, teamId, "", dcId)
+
+			Expect(err).To(HaveOccurred())
+			Expect(err.Error()).To(Equal("email cannot be empty"))
+		})
+	})
+
+	Context("RunE execution flow", func() {
+		It("should successfully add a member to a team", func() {
+			mockClient.EXPECT().AddTeamMember(teamId, email, 0).Return(nil).Once()
+
+			err := c.RunE(nil, []string{})
+			Expect(err).ToNot(HaveOccurred())
+		})
+
+		It("should fail when the token is not allowed to add a member", func() {
+			mockClient.EXPECT().AddTeamMember(teamId, email, 0).Return(errors.New("failed")).Once()
+
+			err := c.RunE(nil, []string{})
+			Expect(err).To(HaveOccurred())
+			Expect(err.Error()).To(ContainSubstring("failed to add member to team: "))
+		})
+
+		It("should fail when client creation fails", func() {
+			c.ClientFactory = func(opts cmd.GlobalOptions) (cmd.Client, error) {
+				return nil, errors.New("client init failed")
+			}
+
+			err := c.RunE(nil, []string{})
+			Expect(err).To(HaveOccurred())
+			Expect(err.Error()).To(ContainSubstring("failed to create Codespehre client: client init failed"))
+		})
+
+		It("should fail when team ID is unavailable", func() {
+			c.Opts.GlobalOptions.TeamId = -1
+			mockEnv.EXPECT().GetTeamId().Return(-1, errors.New("CS_TEAM_ID env var required, but not set")).Once()
+
+			err := c.RunE(nil, []string{})
+			Expect(err).To(HaveOccurred())
+			Expect(err.Error()).To(Equal("CS_TEAM_ID env var required, but not set"))
+		})
+
+		It("should fail when email is empty", func() {
+			c.Opts.Email = ""
+			err := c.RunE(nil, []string{})
+			Expect(err).To(HaveOccurred())
+			Expect(err.Error()).To(Equal("email cannot be empty"))
+		})
+	})
+	
+})

cli/cmd/team_add_member_test.go

@@ -5,12 +5,15 @@ package cmd
 
 import (
 	"errors"
+	"fmt"
+
 	"github.com/spf13/cobra"
 )
 
 type RemoveTeamCmd struct {
-	cmd  *cobra.Command
-	Opts RemoveTeamOpts
+	cmd           *cobra.Command
+	Opts          RemoveTeamOpts
+	ClientFactory func(GlobalOptions) (Client, error)
 }
 
 type RemoveTeamOpts struct {
@@ -28,6 +31,7 @@ func AddRemoveTeamCmd(team *cobra.Command, opts *GlobalOptions) {
 		Opts: RemoveTeamOpts{
 			GlobalOptions: opts,
 		},
+		ClientFactory: NewClient,
 	}
 	t.cmd.RunE = t.RunE
 	t.cmd.Flags().StringVarP(&t.Opts.name, "name", "n", "", "Team name")
@@ -36,10 +40,9 @@ func AddRemoveTeamCmd(team *cobra.Command, opts *GlobalOptions) {
 }
 
 func (c *RemoveTeamCmd) RunE(_ *cobra.Command, args []string) error {
-	// TODO: Implement team removal logic
-	client, err := NewClient(*c.Opts.GlobalOptions)
+	client, err := c.ClientFactory(*c.Opts.GlobalOptions)
 	if err != nil {
-		return err
+		return fmt.Errorf("failed to create Codespehre client: %w", err)
 	}
 
 	orgId, err := c.Opts.GetOrgId()
@@ -52,12 +55,9 @@ func (c *RemoveTeamCmd) RunE(_ *cobra.Command, args []string) error {
 		return errors.New("team ID not set, use -T or CS_TEAM_ID to set it")
 	}
 
-	//
-
 	err = client.DeleteTeam(orgId, teamId)
 	if err != nil {
-		return err
+		return fmt.Errorf("failed to delete team: %w", err)
 	}
 	return nil
-
 }