From 7d2795a9e0ee36bb6bec0743051e608f4071e1d4 Mon Sep 17 00:00:00 2001
From: Florian Sylvain <floriansylvain@mailo.com>
Date: Sun, 19 May 2024 17:12:48 +0200
Subject: [PATCH] feat: post deletion

---
 adapters/secondary/gateways/postRepository.go |  3 +--
 .../gateways/web/templates/posts.html         | 26 +++++++++++++++++--
 api/dependecyInjection.go                     |  2 ++
 api/page.go                                   |  1 +
 api/pagePostDelete.go                         | 24 +++++++++++++++++
 api/post.go                                   | 16 ++++++++++++
 test/post_test.go                             | 19 ++++++++++++++
 useCases/DeletePostUseCase.go                 | 20 ++++++++++++++
 8 files changed, 107 insertions(+), 4 deletions(-)
 create mode 100644 api/pagePostDelete.go
 create mode 100644 useCases/DeletePostUseCase.go

diff --git a/adapters/secondary/gateways/postRepository.go b/adapters/secondary/gateways/postRepository.go
index 9f098ac..a3ec04c 100644
--- a/adapters/secondary/gateways/postRepository.go
+++ b/adapters/secondary/gateways/postRepository.go
@@ -86,8 +86,7 @@ func (a *PostRepository) UpdateBody(id uint32, body string) error {
 }
 
 func (a *PostRepository) Delete(id uint32) error {
-	//TODO implement me
-	panic("implement me")
+	return a.db.Delete(&entity.Post{}, id).Error
 }
 
 var _ gateways.IPostRepository = &PostRepository{}
diff --git a/adapters/secondary/gateways/web/templates/posts.html b/adapters/secondary/gateways/web/templates/posts.html
index 938e83a..2db5db6 100644
--- a/adapters/secondary/gateways/web/templates/posts.html
+++ b/adapters/secondary/gateways/web/templates/posts.html
@@ -44,16 +44,38 @@
                         <use xlink:href="/static/bootstrap-icons.svg#pen"/>
                     </svg>
                 </a>
-                <a href="/post/{{ $post.Title }}/delete" class="btn btn-outline-danger btn-sm">
-                    <svg width="12px" height="12px" fill="currentColor">
+                <a href="/post/{{ $post.Title }}/delete" class="btn btn-outline-danger btn-sm" data-bs-toggle="modal"
+                   data-bs-target="#{{ $post.ID }}">
+                    <svg width=" 12px" height="12px" fill="currentColor">
                         <use xlink:href="/static/bootstrap-icons.svg#trash"/>
                     </svg>
                 </a>
+                <div class="modal fade" id="{{ $post.ID }}" tabindex="-1" aria-labelledby="{{ $post.ID }}Label"
+                     aria-hidden="true">
+                    <div class="modal-dialog">
+                        <div class="modal-content">
+                            <div class="modal-header">
+                                <h1 class="modal-title fs-5" id="{{ $post.ID }}Label">Delete post</h1>
+                                <button type="button" class="btn-close" data-bs-dismiss="modal"
+                                        aria-label="Close"></button>
+                            </div>
+                            <div class="modal-body">
+                                <p>You are about to delete the post <b>« {{ $post.Title }} »</b>, this is definitive, are you sure?</p>
+                            </div>
+                            <div class="modal-footer">
+                                <button type="button" class="btn btn-outline-secondary" data-bs-dismiss="modal">Cancel
+                                </button>
+                                <a href="/post/{{$post.ID}}/delete" class="btn btn-danger">Delete</a>
+                            </div>
+                        </div>
+                    </div>
+                </div>
             </td>
         </tr>
         {{ end }}
         </tbody>
     </table>
+
 </div>
 
 <script>
diff --git a/api/dependecyInjection.go b/api/dependecyInjection.go
index 14eddc7..21d83e2 100644
--- a/api/dependecyInjection.go
+++ b/api/dependecyInjection.go
@@ -15,6 +15,7 @@ type UseCases struct {
 	GetPostUseCase    *useCases.GetPostUseCase
 	ListPostsUseCase  *useCases.ListPostsUseCase
 	UpdatePostUseCase *useCases.UpdatePostUseCase
+	DeletePostUseCase *useCases.DeletePostUseCase
 	GetUserUseCase    *useCases.GetUserUseCase
 	CreateUserUseCase *useCases.CreateUserUseCase
 	UpdateUserUseCase *useCases.UpdateUserUseCase
@@ -51,6 +52,7 @@ func InitContainer() {
 			GetPostUseCase:    useCases.NewGetPostUseCase(db),
 			ListPostsUseCase:  useCases.NewListPostsUseCase(db),
 			UpdatePostUseCase: useCases.NewUpdatePostUseCase(db),
+			DeletePostUseCase: useCases.NewDeletePostUseCase(db),
 			GetUserUseCase:    useCases.NewGetUserUseCase(db),
 			CreateUserUseCase: useCases.NewCreateUserUseCase(db),
 			UpdateUserUseCase: useCases.NewUpdateUserUseCase(db),
diff --git a/api/page.go b/api/page.go
index 60a2b64..27e1ddd 100644
--- a/api/page.go
+++ b/api/page.go
@@ -123,6 +123,7 @@ func NewPageRouter() http.Handler {
 		r.Get("/home", GetHomePage)
 		r.Get("/post", GetPostsPage)
 		r.Get("/post/{name}/edit", GetPostEditPage)
+		r.Get("/post/{id}/delete", GetPostDeletePage)
 		r.Post("/post/{name}/edit", PostPostEditPage)
 		r.Get("/post/create", GetPostCreatePage)
 		r.Post("/post/create", PostPostCreatePage)
diff --git a/api/pagePostDelete.go b/api/pagePostDelete.go
new file mode 100644
index 0000000..a4a1199
--- /dev/null
+++ b/api/pagePostDelete.go
@@ -0,0 +1,24 @@
+package api
+
+import (
+	"github.com/go-chi/chi/v5"
+	"net/http"
+	"strconv"
+)
+
+func GetPostDeletePage(w http.ResponseWriter, r *http.Request) {
+	postId := chi.URLParam(r, "id")
+	id, err := strconv.Atoi(postId)
+	if err != nil {
+		http.Error(w, http.StatusText(400), http.StatusBadRequest)
+		return
+	}
+
+	err = Container.DeletePostUseCase.DeletePost(uint32(id))
+	if err != nil {
+		http.Error(w, http.StatusText(400), http.StatusBadRequest)
+		return
+	}
+
+	http.Redirect(w, r, "/post", http.StatusSeeOther)
+}
diff --git a/api/post.go b/api/post.go
index 0277bc8..c3c8416 100644
--- a/api/post.go
+++ b/api/post.go
@@ -64,10 +64,26 @@ func listPosts(w http.ResponseWriter, _ *http.Request) {
 	_, _ = w.Write(postsJson)
 }
 
+func deletePost(w http.ResponseWriter, r *http.Request) {
+	idParam := chi.URLParam(r, "id")
+	id, err := strconv.Atoi(idParam)
+	if err != nil {
+		http.Error(w, err.Error(), http.StatusBadRequest)
+	}
+
+	err = Container.DeletePostUseCase.DeletePost(uint32(id))
+	if err != nil {
+		http.Error(w, err.Error(), http.StatusNotFound)
+	}
+
+	_, _ = w.Write([]byte("post deleted"))
+}
+
 func NewPostRouter() http.Handler {
 	r := chi.NewRouter()
 	r.Get("/{id}", getPost)
 	r.Post("/", postPost)
 	r.Get("/", listPosts)
+	r.Delete("/{id}", deletePost)
 	return r
 }
diff --git a/test/post_test.go b/test/post_test.go
index 9df52cd..c784b88 100644
--- a/test/post_test.go
+++ b/test/post_test.go
@@ -120,6 +120,20 @@ var TestGetAllPostsSuccess = func(t *testing.T) {
 	assert.Equal(t, createdPost.Body, response[0].Body)
 }
 
+var TestDeletePostSuccess = func(t *testing.T) {
+	var createdPost post.Post
+	var postToCreate = post.Post{
+		Title: "Test Title",
+		Body:  "Test Body",
+	}
+	db := GetDb()
+	db.Create(&postToCreate).Scan(&createdPost)
+
+	r, _ := ApiRequest("DELETE", "/post/"+strconv.Itoa(int(createdPost.ID)), nil)
+
+	assert.Equal(t, http.StatusOK, r.StatusCode)
+}
+
 var TestPostCreate = func(t *testing.T) {
 	t.Run("Should return a post with the given title and body", TestCreatePostSuccess)
 	t.Run("Should return an error if the title is missing", TestCreatePostFailTitleMissing)
@@ -134,6 +148,10 @@ var TestPostGetAll = func(t *testing.T) {
 	t.Run("Should return all posts", TestGetAllPostsSuccess)
 }
 
+var TestPostDelete = func(t *testing.T) {
+	t.Run("Should return success", TestDeletePostSuccess)
+}
+
 func TestPost(t *testing.T) {
 	StartServerIfNotAlready()
 	WaitForServer()
@@ -141,4 +159,5 @@ func TestPost(t *testing.T) {
 	t.Run("Create", TestPostCreate)
 	t.Run("Get", TestPostGet)
 	t.Run("GetAll", TestPostGetAll)
+	t.Run("Delete", TestPostDelete)
 }
diff --git a/useCases/DeletePostUseCase.go b/useCases/DeletePostUseCase.go
new file mode 100644
index 0000000..4f96190
--- /dev/null
+++ b/useCases/DeletePostUseCase.go
@@ -0,0 +1,20 @@
+package useCases
+
+import (
+	"GoCMS/adapters/secondary/gateways"
+	"gorm.io/gorm"
+)
+
+type DeletePostUseCase struct {
+	postRepository gateways.PostRepository
+}
+
+func NewDeletePostUseCase(db *gorm.DB) *DeletePostUseCase {
+	return &DeletePostUseCase{
+		postRepository: *gateways.NewPostRepository(db),
+	}
+}
+
+func (g *DeletePostUseCase) DeletePost(userId uint32) error {
+	return g.postRepository.Delete(userId)
+}