Add book deleter

Author: rafaelhl

README.md

@@ -1,10 +1,11 @@
 ## Sample project to validate the [REL lib](https://fs02.github.io/rel/#/) as database layer
 
-This project implements a book library with four operations:
+This project implements a book library with five operations:
 
  - POST /books (Insert a new book on the library)
  - GET /books/{bookID} (Get a book by your ID on the library)
  - PUT /books/{bookID} (Update a book on the library)
+ - DELETE /books/{bookID} (Delete a book on the library)
  - GET /books (Get all books on the library)
 
 ## How to run

books/bookdeleter/bookdeleter.go

@@ -0,0 +1,41 @@
+//go:generate mockery -name=Finder
+//go:generate mockery -name=Deleter
+
+package bookdeleter
+
+import (
+	"context"
+
+	"github.com/rafaelhl/library-rel-lib/books"
+)
+
+type (
+	Finder interface {
+		FindBookByID(ctx context.Context, bookID int) (books.Book, error)
+	}
+
+	Deleter interface {
+		DeleteBook(ctx context.Context, book books.Book) error
+	}
+
+	BookDeleter struct {
+		finder  Finder
+		deleter Deleter
+	}
+)
+
+func New(finder Finder, deleter Deleter) BookDeleter {
+	return BookDeleter{
+		finder:  finder,
+		deleter: deleter,
+	}
+}
+
+func (d BookDeleter) DeleteBook(ctx context.Context, id int) error {
+	book, err := d.finder.FindBookByID(ctx, id)
+	if err != nil || book.ID == 0 {
+		return err
+	}
+
+	return d.deleter.DeleteBook(ctx, book)
+}

books/bookdeleter/bookdeleter_test.go

@@ -0,0 +1,32 @@
+package bookdeleter_test
+
+import (
+	"context"
+	"testing"
+
+	"github.com/stretchr/testify/assert"
+	"github.com/stretchr/testify/mock"
+
+	"github.com/rafaelhl/library-rel-lib/books"
+	"github.com/rafaelhl/library-rel-lib/books/bookdeleter"
+	"github.com/rafaelhl/library-rel-lib/books/bookdeleter/mocks"
+)
+
+func TestBookDeleter_DeleteBook(t *testing.T) {
+	finder := new(mocks.Finder)
+	book := books.Book{
+		ID:          1,
+		Title:       "Deleting a book",
+		Author:      "Test",
+		Description: "Book of test delete",
+		Edition:     1,
+		ShelfID:     1,
+	}
+	finder.On("FindBookByID", mock.Anything, 1).Return(book, nil)
+	deleter := new(mocks.Deleter)
+	deleter.On("DeleteBook", mock.Anything, book).Return(nil)
+	d := bookdeleter.New(finder, deleter)
+
+	err := d.DeleteBook(context.Background(), 1)
+	assert.NoError(t, err)
+}

books/bookdeleter/mocks/Deleter.go

@@ -14,6 +14,7 @@ type (
 		ShelfID     int       `db:"shelf_id" json:"-"`
 		CreatedAt   time.Time `json:"-"`
 		UpdatedAt   time.Time `json:"-"`
+		DeletedAt   time.Time `json:"-"`
 		BookShelf   Shelf     `json:"shelf" ref:"shelf_id" fk:"id"`
 	}
 

books/bookdeleter/mocks/Finder.go

@@ -0,0 +1,42 @@
+//go:generate mockery -name=BookDeleter
+
+package bookdelete
+
+import (
+	"context"
+	"net/http"
+	"strconv"
+
+	"github.com/go-chi/chi"
+)
+
+type (
+	BookDeleter interface {
+		DeleteBook(ctx context.Context, id int) error
+	}
+
+	handler struct {
+		deleter BookDeleter
+	}
+)
+
+func NewHandler(deleter BookDeleter) handler {
+	return handler{
+		deleter: deleter,
+	}
+}
+
+func (h handler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
+	param := chi.URLParam(r, "bookID")
+	bookID, err := strconv.Atoi(param)
+	if err != nil {
+		w.WriteHeader(http.StatusInternalServerError)
+		return
+	}
+
+	err = h.deleter.DeleteBook(r.Context(), bookID)
+	if err != nil {
+		w.WriteHeader(http.StatusInternalServerError)
+		return
+	}
+}

books/entities.go

@@ -0,0 +1,30 @@
+package bookdelete_test
+
+import (
+	"net/http"
+	"net/http/httptest"
+	"testing"
+
+	"github.com/go-chi/chi"
+	"github.com/stretchr/testify/assert"
+	"github.com/stretchr/testify/mock"
+
+	"github.com/rafaelhl/library-rel-lib/books/handler/bookdelete"
+	"github.com/rafaelhl/library-rel-lib/books/handler/bookdelete/mocks"
+)
+
+func TestHandler_ServeHTTP(t *testing.T) {
+	deleter := new(mocks.BookDeleter)
+	deleter.On("DeleteBook", mock.Anything, 1).Return(nil)
+	h := bookdelete.NewHandler(deleter)
+
+	mux := chi.NewMux()
+	mux.Method(http.MethodDelete, "/books/{bookID}", h)
+
+	req := httptest.NewRequest(http.MethodDelete, "/books/1", nil)
+	recorder := httptest.NewRecorder()
+	mux.ServeHTTP(recorder, req)
+
+	deleter.AssertExpectations(t)
+	assert.Equal(t, http.StatusOK, recorder.Code)
+}

books/handler/bookdelete/bookdelete.go

@@ -76,3 +76,7 @@ func (r BooksRepository) FindAllBooks(ctx context.Context) ([]books.Book, error)
 	}
 	return bookList, err
 }
+
+func (r BooksRepository) DeleteBook(ctx context.Context, book books.Book) error {
+	return r.repo.Delete(ctx, &book)
+}

books/handler/bookdelete/bookdelete_test.go

@@ -156,3 +156,23 @@ func TestBooksRepository_FindAllBooks(t *testing.T) {
 	assert.NoError(t, err)
 	assert.Equal(t, expectedBooks, allBooks)
 }
+
+func TestBooksRepository_DeleteBook(t *testing.T) {
+	repo := reltest.New()
+	booksRepository := repository.New(repo)
+	book := books.Book{
+		ID:          1,
+		Title:       "Livro de Teste",
+		Description: "Esse livro é de teste",
+		Author:      "Rafael Holanda",
+		Edition:     1,
+		BookShelf: books.Shelf{
+			ID: 1,
+		},
+	}
+
+	repo.ExpectDelete().For(&book)
+	err := booksRepository.DeleteBook(context.Background(), book)
+
+	assert.NoError(t, err)
+}

books/handler/bookdelete/mocks/BookDeleter.go

@@ -11,10 +11,12 @@ import (
 	"github.com/go-chi/chi/middleware"
 	"github.com/go-chi/render"
 
+	"github.com/rafaelhl/library-rel-lib/books/bookdeleter"
 	"github.com/rafaelhl/library-rel-lib/books/bookfinder"
 	"github.com/rafaelhl/library-rel-lib/books/booklistfinder"
 	"github.com/rafaelhl/library-rel-lib/books/booksinserter"
 	"github.com/rafaelhl/library-rel-lib/books/bookupdater"
+	"github.com/rafaelhl/library-rel-lib/books/handler/bookdelete"
 	"github.com/rafaelhl/library-rel-lib/books/handler/bookfind"
 	"github.com/rafaelhl/library-rel-lib/books/handler/bookinsert"
 	"github.com/rafaelhl/library-rel-lib/books/handler/booklistfind"
@@ -44,6 +46,7 @@ func main() {
 	router.Method(http.MethodPost, "/books", bookinsert.NewHandler(booksinserter.New(repository, repository)))
 	router.Method(http.MethodGet, "/books/{bookID}", bookfind.NewHandler(bookfinder.New(repository)))
 	router.Method(http.MethodPut, "/books/{bookID}", bookupdate.NewHandler(bookupdater.New(repository)))
+	router.Method(http.MethodDelete, "/books/{bookID}", bookdelete.NewHandler(bookdeleter.New(repository, repository)))
 	router.Method(http.MethodGet, "/books", booklistfind.NewHandler(booklistfinder.New(repository)))
 
 	err = http.ListenAndServe(":8080", router)