123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431 |
- package image
- import (
- "bytes"
- "context"
- "database/sql"
- "errors"
- "mime/multipart"
- "net/http/httptest"
- "strconv"
- "testing"
- API "git.dmitriygnatenko.ru/dima/homethings/internal/api/v1"
- "git.dmitriygnatenko.ru/dima/homethings/internal/dto"
- "git.dmitriygnatenko.ru/dima/homethings/internal/helpers"
- "git.dmitriygnatenko.ru/dima/homethings/internal/interfaces"
- "git.dmitriygnatenko.ru/dima/homethings/internal/models"
- repoMocks "git.dmitriygnatenko.ru/dima/homethings/internal/repositories/mocks"
- sp "git.dmitriygnatenko.ru/dima/homethings/internal/service_provider"
- "github.com/brianvoe/gofakeit/v6"
- "github.com/gofiber/fiber/v2"
- "github.com/gojuno/minimock/v3"
- "github.com/stretchr/testify/assert"
- )
- // nolint:errcheck
- func Test_AddImageHandler(t *testing.T) {
- type req struct {
- method string
- route string
- contentType string
- body []byte
- }
- var (
- mc = minimock.NewController(t)
- placeID = gofakeit.Number(1, 1000)
- thingID = gofakeit.Number(1, 1000)
- testError = errors.New(gofakeit.Phrase())
- )
- // Correct request for adding place image
- addPlaceCorrectBody := &bytes.Buffer{}
- addPlaceCorrectWriter := multipart.NewWriter(addPlaceCorrectBody)
- addPlaceCorrectWriter.WriteField("place_id", strconv.Itoa(placeID))
- addPlaceCorrectWriter.CreateFormFile("files", gofakeit.Word())
- addPlaceCorrectContentType := addPlaceCorrectWriter.FormDataContentType()
- addPlaceCorrectWriter.Close()
- // Correct request for adding thing image
- addThingCorrectBody := &bytes.Buffer{}
- addThingCorrectWriter := multipart.NewWriter(addThingCorrectBody)
- addThingCorrectWriter.WriteField("thing_id", strconv.Itoa(thingID))
- addThingCorrectWriter.CreateFormFile("files", gofakeit.Word())
- addThingCorrectContentType := addThingCorrectWriter.FormDataContentType()
- addThingCorrectWriter.Close()
- // Incorrect request for adding place image
- addPlaceIncorrectBody := &bytes.Buffer{}
- addPlaceIncorrectWriter := multipart.NewWriter(addPlaceIncorrectBody)
- addPlaceIncorrectWriter.WriteField("place_id", gofakeit.Word())
- addPlaceIncorrectContentType := addPlaceIncorrectWriter.FormDataContentType()
- addPlaceIncorrectWriter.Close()
- // Incorrect request for adding thing image
- addThingIncorrectBody := &bytes.Buffer{}
- addThingIncorrectWriter := multipart.NewWriter(addThingIncorrectBody)
- addThingIncorrectWriter.WriteField("thing_id", gofakeit.Word())
- addThingIncorrectContentType := addThingIncorrectWriter.FormDataContentType()
- addThingIncorrectWriter.Close()
- // Incorrect empty request
- emptyBody := &bytes.Buffer{}
- emptyWriter := multipart.NewWriter(emptyBody)
- emptyContentType := emptyWriter.FormDataContentType()
- emptyWriter.Close()
- tests := []struct {
- name string
- req req
- resCode int
- resBody interface{}
- fileRepoMock func(mc *minimock.Controller) interfaces.FileRepository
- placeImageRepoMock func(mc *minimock.Controller) interfaces.PlaceImageRepository
- thingImageRepoMock func(mc *minimock.Controller) interfaces.ThingImageRepository
- }{
- {
- name: "negative case - bad request",
- req: req{
- method: fiber.MethodPost,
- route: "/v1/images",
- },
- resCode: fiber.StatusBadRequest,
- placeImageRepoMock: func(mc *minimock.Controller) interfaces.PlaceImageRepository {
- return repoMocks.NewPlaceImageRepositoryMock(mc)
- },
- thingImageRepoMock: func(mc *minimock.Controller) interfaces.ThingImageRepository {
- return repoMocks.NewThingImageRepositoryMock(mc)
- },
- fileRepoMock: func(mc *minimock.Controller) interfaces.FileRepository {
- return repoMocks.NewFileRepositoryMock(mc)
- },
- },
- {
- name: "negative case - incorrect request",
- req: req{
- method: fiber.MethodPost,
- route: "/v1/images",
- body: addPlaceIncorrectBody.Bytes(),
- contentType: addPlaceIncorrectContentType,
- },
- resCode: fiber.StatusBadRequest,
- placeImageRepoMock: func(mc *minimock.Controller) interfaces.PlaceImageRepository {
- return repoMocks.NewPlaceImageRepositoryMock(mc)
- },
- thingImageRepoMock: func(mc *minimock.Controller) interfaces.ThingImageRepository {
- return repoMocks.NewThingImageRepositoryMock(mc)
- },
- fileRepoMock: func(mc *minimock.Controller) interfaces.FileRepository {
- return repoMocks.NewFileRepositoryMock(mc)
- },
- },
- {
- name: "negative case - incorrect request",
- req: req{
- method: fiber.MethodPost,
- route: "/v1/images",
- body: addThingIncorrectBody.Bytes(),
- contentType: addThingIncorrectContentType,
- },
- resCode: fiber.StatusBadRequest,
- placeImageRepoMock: func(mc *minimock.Controller) interfaces.PlaceImageRepository {
- return repoMocks.NewPlaceImageRepositoryMock(mc)
- },
- thingImageRepoMock: func(mc *minimock.Controller) interfaces.ThingImageRepository {
- return repoMocks.NewThingImageRepositoryMock(mc)
- },
- fileRepoMock: func(mc *minimock.Controller) interfaces.FileRepository {
- return repoMocks.NewFileRepositoryMock(mc)
- },
- },
- {
- name: "negative case - empty request",
- req: req{
- method: fiber.MethodPost,
- route: "/v1/images",
- body: emptyBody.Bytes(),
- contentType: emptyContentType,
- },
- resCode: fiber.StatusBadRequest,
- placeImageRepoMock: func(mc *minimock.Controller) interfaces.PlaceImageRepository {
- return repoMocks.NewPlaceImageRepositoryMock(mc)
- },
- thingImageRepoMock: func(mc *minimock.Controller) interfaces.ThingImageRepository {
- return repoMocks.NewThingImageRepositoryMock(mc)
- },
- fileRepoMock: func(mc *minimock.Controller) interfaces.FileRepository {
- return repoMocks.NewFileRepositoryMock(mc)
- },
- },
- {
- name: "negative case - repository error (begin tx)",
- req: req{
- method: fiber.MethodPost,
- route: "/v1/images",
- body: addPlaceCorrectBody.Bytes(),
- contentType: addPlaceCorrectContentType,
- },
- resCode: fiber.StatusInternalServerError,
- placeImageRepoMock: func(mc *minimock.Controller) interfaces.PlaceImageRepository {
- mock := repoMocks.NewPlaceImageRepositoryMock(mc)
- mock.BeginTxMock.Return(nil, testError)
- return mock
- },
- thingImageRepoMock: func(mc *minimock.Controller) interfaces.ThingImageRepository {
- return repoMocks.NewThingImageRepositoryMock(mc)
- },
- fileRepoMock: func(mc *minimock.Controller) interfaces.FileRepository {
- mock := repoMocks.NewFileRepositoryMock(mc)
- mock.SaveMock.Return(nil)
- return mock
- },
- },
- {
- name: "negative case - repository error (commit tx)",
- req: req{
- method: fiber.MethodPost,
- route: "/v1/images",
- body: addPlaceCorrectBody.Bytes(),
- contentType: addPlaceCorrectContentType,
- },
- resCode: fiber.StatusInternalServerError,
- placeImageRepoMock: func(mc *minimock.Controller) interfaces.PlaceImageRepository {
- mock := repoMocks.NewPlaceImageRepositoryMock(mc)
- mock.BeginTxMock.Return(nil, nil)
- mock.AddMock.Inspect(func(ctx context.Context, req models.AddPlaceImageRequest, tx *sql.Tx) {
- assert.Equal(mc, placeID, req.PlaceID)
- }).Return(nil)
- mock.CommitTxMock.Return(testError)
- return mock
- },
- thingImageRepoMock: func(mc *minimock.Controller) interfaces.ThingImageRepository {
- return repoMocks.NewThingImageRepositoryMock(mc)
- },
- fileRepoMock: func(mc *minimock.Controller) interfaces.FileRepository {
- mock := repoMocks.NewFileRepositoryMock(mc)
- mock.SaveMock.Return(nil)
- return mock
- },
- },
- {
- name: "negative case - repository error (add image)",
- req: req{
- method: fiber.MethodPost,
- route: "/v1/images",
- body: addPlaceCorrectBody.Bytes(),
- contentType: addPlaceCorrectContentType,
- },
- resCode: fiber.StatusInternalServerError,
- placeImageRepoMock: func(mc *minimock.Controller) interfaces.PlaceImageRepository {
- mock := repoMocks.NewPlaceImageRepositoryMock(mc)
- mock.BeginTxMock.Return(nil, nil)
- mock.AddMock.Inspect(func(ctx context.Context, req models.AddPlaceImageRequest, tx *sql.Tx) {
- assert.Equal(mc, placeID, req.PlaceID)
- }).Return(testError)
- return mock
- },
- thingImageRepoMock: func(mc *minimock.Controller) interfaces.ThingImageRepository {
- return repoMocks.NewThingImageRepositoryMock(mc)
- },
- fileRepoMock: func(mc *minimock.Controller) interfaces.FileRepository {
- mock := repoMocks.NewFileRepositoryMock(mc)
- mock.SaveMock.Return(nil)
- return mock
- },
- },
- {
- name: "positive case - add place image",
- req: req{
- method: fiber.MethodPost,
- route: "/v1/images",
- body: addPlaceCorrectBody.Bytes(),
- contentType: addPlaceCorrectContentType,
- },
- resCode: fiber.StatusOK,
- resBody: dto.EmptyResponse{},
- placeImageRepoMock: func(mc *minimock.Controller) interfaces.PlaceImageRepository {
- mock := repoMocks.NewPlaceImageRepositoryMock(mc)
- mock.BeginTxMock.Return(nil, nil)
- mock.AddMock.Inspect(func(ctx context.Context, req models.AddPlaceImageRequest, tx *sql.Tx) {
- assert.Equal(mc, placeID, req.PlaceID)
- }).Return(nil)
- mock.CommitTxMock.Return(nil)
- return mock
- },
- thingImageRepoMock: func(mc *minimock.Controller) interfaces.ThingImageRepository {
- return repoMocks.NewThingImageRepositoryMock(mc)
- },
- fileRepoMock: func(mc *minimock.Controller) interfaces.FileRepository {
- mock := repoMocks.NewFileRepositoryMock(mc)
- mock.SaveMock.Return(nil)
- return mock
- },
- },
- {
- name: "negative case - repository error (begin tx)",
- req: req{
- method: fiber.MethodPost,
- route: "/v1/images",
- body: addThingCorrectBody.Bytes(),
- contentType: addThingCorrectContentType,
- },
- resCode: fiber.StatusInternalServerError,
- placeImageRepoMock: func(mc *minimock.Controller) interfaces.PlaceImageRepository {
- return repoMocks.NewPlaceImageRepositoryMock(mc)
- },
- thingImageRepoMock: func(mc *minimock.Controller) interfaces.ThingImageRepository {
- mock := repoMocks.NewThingImageRepositoryMock(mc)
- mock.BeginTxMock.Return(nil, testError)
- return mock
- },
- fileRepoMock: func(mc *minimock.Controller) interfaces.FileRepository {
- mock := repoMocks.NewFileRepositoryMock(mc)
- mock.SaveMock.Return(nil)
- return mock
- },
- },
- {
- name: "negative case - repository error (commit tx)",
- req: req{
- method: fiber.MethodPost,
- route: "/v1/images",
- body: addThingCorrectBody.Bytes(),
- contentType: addThingCorrectContentType,
- },
- resCode: fiber.StatusInternalServerError,
- placeImageRepoMock: func(mc *minimock.Controller) interfaces.PlaceImageRepository {
- return repoMocks.NewPlaceImageRepositoryMock(mc)
- },
- thingImageRepoMock: func(mc *minimock.Controller) interfaces.ThingImageRepository {
- mock := repoMocks.NewThingImageRepositoryMock(mc)
- mock.BeginTxMock.Return(nil, nil)
- mock.AddMock.Inspect(func(ctx context.Context, req models.AddThingImageRequest, tx *sql.Tx) {
- assert.Equal(mc, thingID, req.ThingID)
- }).Return(nil)
- mock.CommitTxMock.Return(testError)
- return mock
- },
- fileRepoMock: func(mc *minimock.Controller) interfaces.FileRepository {
- mock := repoMocks.NewFileRepositoryMock(mc)
- mock.SaveMock.Return(nil)
- return mock
- },
- },
- {
- name: "negative case - repository error (add image)",
- req: req{
- method: fiber.MethodPost,
- route: "/v1/images",
- body: addThingCorrectBody.Bytes(),
- contentType: addThingCorrectContentType,
- },
- resCode: fiber.StatusInternalServerError,
- placeImageRepoMock: func(mc *minimock.Controller) interfaces.PlaceImageRepository {
- return repoMocks.NewPlaceImageRepositoryMock(mc)
- },
- thingImageRepoMock: func(mc *minimock.Controller) interfaces.ThingImageRepository {
- mock := repoMocks.NewThingImageRepositoryMock(mc)
- mock.BeginTxMock.Return(nil, nil)
- mock.AddMock.Inspect(func(ctx context.Context, req models.AddThingImageRequest, tx *sql.Tx) {
- assert.Equal(mc, thingID, req.ThingID)
- }).Return(testError)
- return mock
- },
- fileRepoMock: func(mc *minimock.Controller) interfaces.FileRepository {
- mock := repoMocks.NewFileRepositoryMock(mc)
- mock.SaveMock.Return(nil)
- return mock
- },
- },
- {
- name: "positive case - add thing image",
- req: req{
- method: fiber.MethodPost,
- route: "/v1/images",
- body: addThingCorrectBody.Bytes(),
- contentType: addThingCorrectContentType,
- },
- resCode: fiber.StatusOK,
- resBody: dto.EmptyResponse{},
- placeImageRepoMock: func(mc *minimock.Controller) interfaces.PlaceImageRepository {
- return repoMocks.NewPlaceImageRepositoryMock(mc)
- },
- thingImageRepoMock: func(mc *minimock.Controller) interfaces.ThingImageRepository {
- mock := repoMocks.NewThingImageRepositoryMock(mc)
- mock.BeginTxMock.Return(nil, nil)
- mock.AddMock.Inspect(func(ctx context.Context, req models.AddThingImageRequest, tx *sql.Tx) {
- assert.Equal(mc, thingID, req.ThingID)
- }).Return(nil)
- mock.CommitTxMock.Return(nil)
- return mock
- },
- fileRepoMock: func(mc *minimock.Controller) interfaces.FileRepository {
- mock := repoMocks.NewFileRepositoryMock(mc)
- mock.SaveMock.Return(nil)
- return mock
- },
- },
- {
- name: "negative case - save file error",
- req: req{
- method: fiber.MethodPost,
- route: "/v1/images",
- body: addThingCorrectBody.Bytes(),
- contentType: addThingCorrectContentType,
- },
- resCode: fiber.StatusInternalServerError,
- placeImageRepoMock: func(mc *minimock.Controller) interfaces.PlaceImageRepository {
- return repoMocks.NewPlaceImageRepositoryMock(mc)
- },
- thingImageRepoMock: func(mc *minimock.Controller) interfaces.ThingImageRepository {
- return repoMocks.NewThingImageRepositoryMock(mc)
- },
- fileRepoMock: func(mc *minimock.Controller) interfaces.FileRepository {
- mock := repoMocks.NewFileRepositoryMock(mc)
- mock.SaveMock.Return(testError)
- return mock
- },
- },
- }
- for _, tt := range tests {
- t.Run(tt.name, func(t *testing.T) {
- fiberApp := fiber.New()
- serviceProvider := sp.InitMock(tt.placeImageRepoMock(mc), tt.thingImageRepoMock(mc), tt.fileRepoMock(mc))
- fiberApp.Post("/v1/images", AddImageHandler(serviceProvider))
- fiberReq := httptest.NewRequest(tt.req.method, tt.req.route, bytes.NewReader(tt.req.body))
- fiberReq.Header.Add(fiber.HeaderContentType, tt.req.contentType)
- fiberRes, _ := fiberApp.Test(fiberReq, API.DefaultTestTimeOut)
- assert.Equal(t, tt.resCode, fiberRes.StatusCode)
- if tt.resBody != nil {
- assert.Equal(t, helpers.MarshalResponse(tt.resBody), helpers.ConvertBodyToString(fiberRes.Body))
- }
- })
- }
- }
|