feat: add test for wip

Author: hunjixin

api/jiaozifs.gen.go

@@ -855,13 +855,6 @@ paths:
         - wip
       operationId: createWip
       summary: create working in process
-      parameters:
-        - in: query
-          name: name
-          description: wip name
-          required: true
-          schema:
-            type: string
       responses:
         201:
           description: working in process created

api/swagger.yml

@@ -192,16 +192,14 @@ func (bct BranchController) DeleteBranch(ctx context.Context, w *api.JiaozifsRes
 		return
 	}
 
-	_, err = bct.Repo.RefRepo().Get(ctx, models.NewGetRefParams().SetName(params.RefName).SetRepositoryID(repository.ID))
+	// Delete branch
+	affectedRows, err := bct.Repo.RefRepo().Delete(ctx, models.NewDeleteRefParams().SetName(params.RefName).SetRepositoryID(repository.ID))
 	if err != nil {
 		w.Error(err)
 		return
 	}
-
-	// Delete branch
-	err = bct.Repo.RefRepo().Delete(ctx, models.NewDeleteRefParams().SetName(params.RefName).SetRepositoryID(repository.ID))
-	if err != nil {
-		w.Error(err)
+	if affectedRows == 0 {
+		w.NotFound()
 		return
 	}
 	w.OK()

controller/branch_ctl.go

@@ -61,7 +61,7 @@ func (repositoryCtl RepositoryController) ListRepositoryOfAuthenticatedUser(ctx
 		return
 	}
 
-	repositories, err := repositoryCtl.Repo.RepositoryRepo().List(ctx, models.NewListRepoParams().SetOwnerID(operator.ID))
+	repositories, err := repositoryCtl.Repo.RepositoryRepo().List(ctx, models.NewListRepoParams().SetOwnerID(operator.ID)) //operator is owner
 	if err != nil {
 		w.Error(err)
 		return
@@ -131,7 +131,7 @@ func (repositoryCtl RepositoryController) CreateRepository(ctx context.Context,
 			Name:        body.Name,
 			Description: body.Description,
 			HEAD:        defaultRef.Name,
-			OwnerID:     operator.ID,
+			OwnerID:     operator.ID, // this api only create repo for operator
 			CreatorID:   operator.ID,
 			CreatedAt:   time.Now(),
 			UpdatedAt:   time.Now(),
@@ -174,17 +174,22 @@ func (repositoryCtl RepositoryController) DeleteRepository(ctx context.Context,
 		return
 	}
 
-	repo, err := repositoryCtl.Repo.RepositoryRepo().Get(ctx, models.NewGetRepoParams().SetName(repositoryName).SetOwnerID(operator.ID))
+	repo, err := repositoryCtl.Repo.RepositoryRepo().Get(ctx, models.NewGetRepoParams().SetName(repositoryName).SetOwnerID(owner.ID))
 	if err != nil {
 		w.Error(err)
 		return
 	}
 
-	err = repositoryCtl.Repo.RepositoryRepo().Delete(ctx, models.NewDeleteRepoParams().SetID(repo.ID))
+	affectRows, err := repositoryCtl.Repo.RepositoryRepo().Delete(ctx, models.NewDeleteRepoParams().SetID(repo.ID))
 	if err != nil {
 		w.Error(err)
 		return
 	}
+
+	if affectRows == 0 {
+		w.NotFound()
+		return
+	}
 	w.OK()
 }
 
@@ -247,7 +252,7 @@ func (repositoryCtl RepositoryController) UpdateRepository(ctx context.Context,
 }
 
 func (repositoryCtl RepositoryController) GetCommitsInRepository(ctx context.Context, w *api.JiaozifsResponse, _ *http.Request, ownerName string, repositoryName string, params api.GetCommitsInRepositoryParams) {
-	user, err := auth.GetOperator(ctx)
+	operator, err := auth.GetOperator(ctx)
 	if err != nil {
 		w.Error(err)
 		return
@@ -259,7 +264,7 @@ func (repositoryCtl RepositoryController) GetCommitsInRepository(ctx context.Con
 		return
 	}
 
-	if user.Name != ownerName { //todo check public or private
+	if operator.Name != ownerName { //todo check public or private
 		w.Forbidden()
 		return
 	}