feat: update tag

internal/adapter/gql/gqlmodel/convert_tag.go

@@ -32,3 +32,16 @@ func ToTagGroup(tg *tag.Group) *TagGroup {
 		Tags:    ids,
 	}
 }
+
+func ToTag(t tag.Tag) Tag {
+	if t == nil {
+		return nil
+	}
+	switch ty := t.(type) {
+	case *tag.Item:
+		return ToTagItem(ty)
+	case *tag.Group:
+		return ToTagGroup(ty)
+	}
+	return nil
+}

internal/adapter/gql/resolver_mutation_tag.go

@@ -57,7 +57,7 @@ func (r *mutationResolver) UpdateTag(ctx context.Context, input gqlmodel.UpdateT
 		return nil, err
 	}
 	return &gqlmodel.UpdateTagPayload{
-		Tag: gqlmodel.ToTagGroup(tag),
+		Tag: gqlmodel.ToTag(*tag),
 	}, nil
 }
 

internal/usecase/interactor/tag.go

@@ -178,7 +178,7 @@ func (i *Tag) DetachItemFromGroup(ctx context.Context, inp interfaces.DetachItem
 	return tg, nil
 }
 
-func (i *Tag) UpdateTag(ctx context.Context, inp interfaces.UpdateTagParam, operator *usecase.Operator) (*tag.Group, error) {
+func (i *Tag) UpdateTag(ctx context.Context, inp interfaces.UpdateTagParam, operator *usecase.Operator) (*tag.Tag, error) {
 	tx, err := i.transaction.Begin()
 	if err != nil {
 		return nil, err
@@ -193,19 +193,20 @@ func (i *Tag) UpdateTag(ctx context.Context, inp interfaces.UpdateTagParam, oper
 		return nil, interfaces.ErrOperationDenied
 	}
 
-	tg, err := i.tagRepo.FindGroupByID(ctx, inp.TagID, []id.SceneID{inp.SceneID})
+	tg, err := i.tagRepo.FindByID(ctx, inp.TagID, []id.SceneID{inp.SceneID})
+	t := *tg
 	if err != nil {
 		return nil, err
 	}
 
 	if inp.Label != nil {
-		tg.Rename(*inp.Label)
+		t.Rename(*inp.Label)
 	}
 
-	err = i.tagRepo.Save(ctx, tg)
+	err = i.tagRepo.Save(ctx, t)
 	if err != nil {
 		return nil, err
 	}
 	tx.Commit()
-	return tg, nil
+	return &t, nil
 }

internal/usecase/interfaces/tag.go

@@ -41,5 +41,5 @@ type Tag interface {
 	CreateGroup(context.Context, CreateTagGroupParam, *usecase.Operator) (*tag.Group, error)
 	AttachItemToGroup(context.Context, AttachItemToGroupParam, *usecase.Operator) (*tag.Group, error)
 	DetachItemFromGroup(context.Context, DetachItemToGroupParam, *usecase.Operator) (*tag.Group, error)
-	UpdateTag(context.Context, UpdateTagParam, *usecase.Operator) (*tag.Group, error)
+	UpdateTag(context.Context, UpdateTagParam, *usecase.Operator) (*tag.Tag, error)
 }

pkg/tag/group.go

@@ -8,7 +8,3 @@ type Group struct {
 func (g *Group) Tags() *List {
 	return g.tags
 }
-
-func (g *Group) Rename(s string) {
-	g.label = s
-}

pkg/tag/tag.go

@@ -21,6 +21,7 @@ type Tag interface {
 	ID() id.TagID
 	Scene() id.SceneID
 	Label() string
+	Rename(string)
 }
 
 func (t *tag) ID() id.TagID {
@@ -35,6 +36,10 @@ func (t *tag) Label() string {
 	return t.label
 }
 
+func (t *tag) Rename(s string) {
+	t.label = s
+}
+
 func ToTagGroup(t Tag) *Group {
 	if tg, ok := t.(*Group); ok {
 		return tg

pkg/tag/tag_test.go

@@ -23,3 +23,11 @@ func TestToTagItem(t *testing.T) {
 	item2 := ToTagItem(&tag2)
 	assert.NotNil(t, item2)
 }
+
+func TestTag_Rename(t *testing.T) {
+	tt := tag{
+		label: "xxx",
+	}
+	tt.Rename("changed")
+	assert.Equal(t, "changed", tt.Label())
+}

schema.graphql

@@ -1327,7 +1327,7 @@ type DetachTagItemFromGroupPayload {
 }
 
 type UpdateTagPayload {
-  tag: TagGroup!
+  tag: Tag!
 }
 
 type AttachTagToLayerPayload{