feat: add api for revert wip changes

Author: hunjixin

api/jiaozifs.gen.go

@@ -1070,6 +1070,8 @@ paths:
           description: Unauthorized
         403:
           description: Forbidden
+        500:
+          description: Server Internal Error
 
   /wip/{owner}/{repository}/changes:
     parameters:

api/swagger.yml

@@ -184,6 +184,16 @@ func deleteObject(ctx context.Context, c convey.C, client *api.Client, title str
 	})
 }
 
+func createWip(ctx context.Context, c convey.C, client *api.Client, title string, user string, repoName string, refName string) {
+	c.Convey("create wip "+title, func() {
+		resp, err := client.GetWip(ctx, user, repoName, &api.GetWipParams{
+			RefName: refName,
+		})
+		convey.So(err, convey.ShouldBeNil)
+		convey.So(resp.StatusCode, convey.ShouldEqual, http.StatusCreated)
+	})
+}
+
 func commitWip(ctx context.Context, c convey.C, client *api.Client, title string, user string, repoName string, refName string, msg string) {
 	c.Convey("commit wip "+title, func() {
 		resp, err := client.CommitWip(ctx, user, repoName, &api.CommitWipParams{

integrationtest/helper_test.go

@@ -312,7 +312,6 @@ func WipObjectSpec(ctx context.Context, urlStr string) func(c convey.C) {
 		})
 
 		c.Convey("get wip changes", func(c convey.C) {
-
 			c.Convey("no auth", func() {
 				re := client.RequestEditors
 				client.RequestEditors = nil
@@ -381,5 +380,80 @@ func WipObjectSpec(ctx context.Context, urlStr string) func(c convey.C) {
 				convey.So(*result.JSON200, convey.ShouldHaveLength, 4)
 			})
 		})
+
+		c.Convey("revert wip changes", func(c convey.C) {
+			c.Convey("no auth", func() {
+				re := client.RequestEditors
+				client.RequestEditors = nil
+				resp, err := client.RevertWipChanges(ctx, userName, repoName, &api.RevertWipChangesParams{
+					RefName: branchName,
+				})
+				client.RequestEditors = re
+				convey.So(err, convey.ShouldBeNil)
+				convey.So(resp.StatusCode, convey.ShouldEqual, http.StatusUnauthorized)
+			})
+
+			c.Convey("fail to revert changes in non exit user", func() {
+				resp, err := client.RevertWipChanges(ctx, "mockUser", repoName, &api.RevertWipChangesParams{
+					RefName: branchName,
+				})
+				convey.So(err, convey.ShouldBeNil)
+				convey.So(resp.StatusCode, convey.ShouldEqual, http.StatusNotFound)
+			})
+
+			c.Convey("fail to revert changes in non exit repo", func() {
+				resp, err := client.RevertWipChanges(ctx, userName, "fakeRepo", &api.RevertWipChangesParams{
+					RefName: branchName,
+				})
+				convey.So(err, convey.ShouldBeNil)
+				convey.So(resp.StatusCode, convey.ShouldEqual, http.StatusNotFound)
+			})
+
+			c.Convey("fail to revert changes in non exit branch", func() {
+				resp, err := client.RevertWipChanges(ctx, userName, repoName, &api.RevertWipChangesParams{
+					RefName: "mockref",
+				})
+				convey.So(err, convey.ShouldBeNil)
+				convey.So(resp.StatusCode, convey.ShouldEqual, http.StatusNotFound)
+			})
+
+			c.Convey("forbidden revert changes in others", func() {
+				resp, err := client.RevertWipChanges(ctx, "jimmy", "happygo", &api.RevertWipChangesParams{
+					RefName: branchName,
+				})
+				convey.So(err, convey.ShouldBeNil)
+				convey.So(resp.StatusCode, convey.ShouldEqual, http.StatusForbidden)
+			})
+
+			c.Convey("not exit path", func() {
+				resp, err := client.RevertWipChanges(ctx, userName, repoName, &api.RevertWipChangesParams{
+					RefName:    branchName,
+					PathPrefix: utils.String("a/b/c/d"),
+				})
+				convey.So(err, convey.ShouldBeNil)
+				convey.So(resp.StatusCode, convey.ShouldEqual, http.StatusNotFound)
+			})
+
+			c.Convey("success to revert changes", func() {
+				resp, err := client.RevertWipChanges(ctx, userName, repoName, &api.RevertWipChangesParams{
+					RefName: branchName,
+				})
+				convey.So(err, convey.ShouldBeNil)
+				convey.So(resp.StatusCode, convey.ShouldEqual, http.StatusOK)
+
+				{
+					resp, err = client.GetWipChanges(ctx, userName, repoName, &api.GetWipChangesParams{
+						RefName: branchName,
+					})
+					convey.So(err, convey.ShouldBeNil)
+					convey.So(resp.StatusCode, convey.ShouldEqual, http.StatusOK)
+
+					result, err := api.ParseGetWipChangesResponse(resp)
+					convey.So(err, convey.ShouldBeNil)
+					convey.So(*result.JSON200, convey.ShouldHaveLength, 0)
+				}
+			})
+		})
+
 	}
 }

integrationtest/wip_object_test.go

@@ -189,13 +189,3 @@ func WipSpec(ctx context.Context, urlStr string) func(c convey.C) {
 		})
 	}
 }
-
-func createWip(ctx context.Context, c convey.C, client *api.Client, title string, user string, repoName string, refName string) {
-	c.Convey("create wip "+title, func() {
-		resp, err := client.GetWip(ctx, user, repoName, &api.GetWipParams{
-			RefName: refName,
-		})
-		convey.So(err, convey.ShouldBeNil)
-		convey.So(resp.StatusCode, convey.ShouldEqual, http.StatusCreated)
-	})
-}