feat: implement more CRUD operations for character management with API endpoints

UltimateForm · UltimateForm · commit f397390ebf8d · 2025-06-24T00:35:29.000+01:00
diff --git a/api/bruno/character create.bru b/api/bruno/character create.bru
@@ -0,0 +1,30 @@
+meta {
+  name: character create
+  type: http
+  seq: 5
+}
+
+post {
+  url: {{BASE_URL}}/characters
+  body: json
+  auth: inherit
+}
+
+params:query {
+  ~limit: 10
+  ~skip: 0
+}
+
+body:json {
+  {
+    "name":  "{{NAME}}",
+    "description": "{{DESCR}}",
+    "debut": {{DEBUT}}
+  }
+}
+
+vars:pre-request {
+  NAME: Cristoph Rot
+  DESCR: lorem ipsum
+  DEBUT: 1
+}
diff --git a/api/bruno/character delete.bru b/api/bruno/character delete.bru
@@ -0,0 +1,20 @@
+meta {
+  name: character delete
+  type: http
+  seq: 7
+}
+
+delete {
+  url: {{BASE_URL}}/characters/:id
+  body: none
+  auth: inherit
+}
+
+params:query {
+  ~limit: 10
+  ~skip: 0
+}
+
+params:path {
+  id: f3496428e9e86b9e5df510
+}
diff --git a/api/bruno/character update.bru b/api/bruno/character update.bru
@@ -0,0 +1,35 @@
+meta {
+  name: character update
+  type: http
+  seq: 6
+}
+
+put {
+  url: {{BASE_URL}}/characters/:id
+  body: json
+  auth: inherit
+}
+
+params:query {
+  ~limit: 10
+  ~skip: 0
+}
+
+params:path {
+  id: f3496428e9e86b9e5df510
+}
+
+
+body:json {
+  {
+    "name":  "{{NAME}}",
+    "description": "{{DESCR}}",
+    "debut": {{DEBUT}}
+  }
+}
+
+vars:pre-request {
+  NAME: Cristoph Rot
+  DESCR: lorem ipsum
+  DEBUT: 1
+}
diff --git a/api/bruno/character.bru b/api/bruno/character.bru
@@ -5,7 +5,7 @@ meta {
 }
 
 get {
-  url: {{BASE_URL}}/character/qyeufszVydeucS2CRGfKjC
+  url: {{BASE_URL}}/characters/:id
   body: none
   auth: inherit
 }
@@ -15,6 +15,10 @@ params:query {
   ~skip: 0
 }
 
+params:path {
+  id: 80a7b6a6765d7dffc1b199
+}
+
 body:json {
   {
     "limit": 10,
diff --git a/cmd/api/charactersapi/character.go b/cmd/api/charactersapi/character.go
@@ -20,10 +20,12 @@ func HandleGetCharacter(res http.ResponseWriter, req *http.Request) {
 		return
 	}
 	var char Character = Character{
-		Id:          character.Id,
-		Name:        character.Name,
-		Description: character.Description,
-		Debut:       character.Debut,
+		Id: character.Id,
+		CharacterInfo: CharacterInfo{
+			Name:        character.Name,
+			Description: character.Description,
+			Debut:       character.Debut,
+		},
 	}
 	core.RespondOk(res, char)
 }
diff --git a/cmd/api/charactersapi/characters.go b/cmd/api/charactersapi/characters.go
@@ -28,10 +28,12 @@ func HandleGetCharacters(res http.ResponseWriter, req *http.Request) {
 	// NOTE: eh should i be mapping at all? golang type integrity does mean i dont need to worry so much about leakage
 	for i, char := range characters {
 		charactersMapped[i] = Character{
-			Id:          char.Id,
-			Name:        char.Name,
-			Description: char.Description,
-			Debut:       char.Debut,
+			Id: char.Id,
+			CharacterInfo: CharacterInfo{
+				Name:        char.Name,
+				Description: char.Description,
+				Debut:       char.Debut,
+			},
 		}
 	}
 	core.RespondOk(
diff --git a/cmd/api/charactersapi/create_character.go b/cmd/api/charactersapi/create_character.go
@@ -0,0 +1,41 @@
+package charactersapi
+
+import (
+	"net/http"
+
+	"github.com/UltimateForm/gopen-api/internal/core"
+	"github.com/UltimateForm/gopen-api/internal/repository/charrep"
+	"github.com/UltimateForm/gopen-api/internal/util"
+)
+
+func HandleCreateCharacter(res http.ResponseWriter, req *http.Request) {
+	var bodyChar CharacterInfo
+	bodyErr := core.ParseBody(req, &bodyChar)
+	if bodyErr != nil {
+		core.RespondError(res, req, bodyErr)
+		return
+	}
+	id, idError := util.RandomHex(11)
+	if idError != nil {
+		core.RespondError(res, req, idError)
+		return
+	}
+	createChar, createErr := charrep.WriteOneCharacter(req.Context(), charrep.Character{
+		Id:          id,
+		Name:        bodyChar.Name,
+		Description: bodyChar.Description,
+		Debut:       bodyChar.Debut,
+	})
+	if createErr != nil {
+		core.RespondError(res, req, createErr)
+		return
+	}
+	core.RespondCreated(res, Character{
+		Id: createChar.Id,
+		CharacterInfo: CharacterInfo{
+			Name:        createChar.Name,
+			Description: createChar.Description,
+			Debut:       createChar.Debut,
+		},
+	})
+}
diff --git a/cmd/api/charactersapi/delete_charater.go b/cmd/api/charactersapi/delete_charater.go
@@ -0,0 +1,27 @@
+package charactersapi
+
+import (
+	"net/http"
+
+	"github.com/UltimateForm/gopen-api/internal/core"
+	"github.com/UltimateForm/gopen-api/internal/repository/charrep"
+	"github.com/go-chi/chi/v5"
+)
+
+func HandleDeleteCharacter(res http.ResponseWriter, req *http.Request) {
+	id := chi.URLParam(req, "id")
+	if id == "" {
+		core.RespondKnownError(res, core.BadRequest("id required"))
+		return
+	}
+	deleted, deleteError := charrep.DeleteOneCharacter(req.Context(), id)
+	if deleteError != nil {
+		core.RespondError(res, req, deleteError)
+		return
+	}
+	if !deleted {
+		core.RespondKnownError(res, core.NotFound())
+		return
+	}
+	core.RespondNoContent(res)
+}
diff --git a/cmd/api/charactersapi/types.go b/cmd/api/charactersapi/types.go
@@ -1,12 +1,25 @@
 package charactersapi
 
-type Character struct {
-	Id          string `json:"id"`
+import "github.com/UltimateForm/gopen-api/internal/core"
+
+type CharacterInfo struct {
 	Name        string `json:"name"`
 	Description string `json:"description"`
 	Debut       int    `json:"debut"`
 }
 
+func (src CharacterInfo) Validate() error {
+	if src.Name == "" || src.Description == "" || src.Debut == 0 {
+		return core.BadRequest("missing one or more parameters: name, description, debut")
+	}
+	return nil
+}
+
+type Character struct {
+	Id string `json:"id"`
+	CharacterInfo
+}
+
 type Offset struct {
 	Limit int `json:"limit"`
 	Skip  int `json:"skip"`
diff --git a/cmd/api/charactersapi/update_character.go b/cmd/api/charactersapi/update_character.go
@@ -0,0 +1,41 @@
+package charactersapi
+
+import (
+	"net/http"
+
+	"github.com/UltimateForm/gopen-api/internal/core"
+	"github.com/UltimateForm/gopen-api/internal/repository/charrep"
+	"github.com/go-chi/chi/v5"
+)
+
+func HandleUpdateCharacter(res http.ResponseWriter, req *http.Request) {
+	id := chi.URLParam(req, "id")
+	if id == "" {
+		core.RespondError(res, req, core.BadRequest("id required"))
+		return
+	}
+	var bodyChar CharacterInfo
+	bodyErr := core.ParseBody(req, &bodyChar)
+	if bodyErr != nil {
+		core.RespondError(res, req, bodyErr)
+		return
+	}
+	updatedChar, updateErr := charrep.UpdateOneCharacter(req.Context(), charrep.Character{
+		Id:          id,
+		Name:        bodyChar.Name,
+		Description: bodyChar.Description,
+		Debut:       bodyChar.Debut,
+	})
+	if updateErr != nil {
+		core.RespondError(res, req, updateErr)
+		return
+	}
+	core.RespondOk(res, Character{
+		Id: updatedChar.Id,
+		CharacterInfo: CharacterInfo{
+			Name:        updatedChar.Name,
+			Description: updatedChar.Description,
+			Debut:       updatedChar.Debut,
+		},
+	})
+}
diff --git a/cmd/api/main.go b/cmd/api/main.go
@@ -20,6 +20,9 @@ func Start() http.Handler {
 		r.Use(core.AuthMiddleware)
 		r.Get("/", charactersapi.HandleGetCharacters)
 		r.Get("/{id}", charactersapi.HandleGetCharacter)
+		r.Post("/", charactersapi.HandleCreateCharacter)
+		r.Put("/{id}", charactersapi.HandleUpdateCharacter)
+		r.Delete("/{id}", charactersapi.HandleDeleteCharacter)
 	})
 	log.Println("ROUTER CREATED")
 	return router
diff --git a/internal/core/success.go b/internal/core/success.go
@@ -10,6 +10,12 @@ func RespondOk(res http.ResponseWriter, data any) {
 	json.NewEncoder(res).Encode(data)
 }
 
+func RespondCreated(res http.ResponseWriter, data any) {
+	res.WriteHeader(http.StatusCreated)
+	res.Header().Add("Content-Type", "application/json")
+	json.NewEncoder(res).Encode(data)
+}
+
 func RespondNoContent(res http.ResponseWriter) {
 	res.WriteHeader(http.StatusNoContent)
 }
diff --git a/internal/repository/charrep/main.go b/internal/repository/charrep/main.go
diff --git a/internal/repository/charrep/types.go b/internal/repository/charrep/types.go
diff --git a/internal/repository/main.go b/internal/repository/main.go
diff --git a/internal/util/random.go b/internal/util/random.go

Original file line number	Diff line number	Diff line change
`@@ -5,7 +5,7 @@ meta {`
`5`	`5`	`}`
`6`	`6`
`7`	`7`	`get {`
`8`		`- url: {{BASE_URL}}/character/qyeufszVydeucS2CRGfKjC`
	`8`	`+ url: {{BASE_URL}}/characters/:id`
`9`	`9`	`body: none`
`10`	`10`	`auth: inherit`
`11`	`11`	`}`
`@@ -15,6 +15,10 @@ params:query {`
`15`	`15`	`~skip: 0`
`16`	`16`	`}`
`17`	`17`
	`18`	`+params:path {`
	`19`	`+ id: 80a7b6a6765d7dffc1b199`
	`20`	`+}`
	`21`	`+`
`18`	`22`	`body:json {`
`19`	`23`	`{`
`20`	`24`	`"limit": 10,`
Original file line number	Diff line number	Diff line change
`@@ -20,10 +20,12 @@ func HandleGetCharacter(res http.ResponseWriter, req *http.Request) {`
`20`	`20`	`return`
`21`	`21`	`}`
`22`	`22`	`var char Character = Character{`
`23`		`- Id: character.Id,`
`24`		`- Name: character.Name,`
`25`		`- Description: character.Description,`
`26`		`- Debut: character.Debut,`
	`23`	`+ Id: character.Id,`
	`24`	`+ CharacterInfo: CharacterInfo{`
	`25`	`+ Name: character.Name,`
	`26`	`+ Description: character.Description,`
	`27`	`+ Debut: character.Debut,`
	`28`	`+ },`
`27`	`29`	`}`
`28`	`30`	`core.RespondOk(res, char)`
`29`	`31`	`}`