123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355 |
- package image
- import (
- "bytes"
- "context"
- "mime/multipart"
- "net/http/httptest"
- "strconv"
- "testing"
- "github.com/brianvoe/gofakeit/v6"
- "github.com/gofiber/fiber/v2"
- "github.com/gojuno/minimock/v3"
- "github.com/stretchr/testify/assert"
- "git.dmitriygnatenko.ru/dima/homethings/internal/api/v1/image/mocks"
- "git.dmitriygnatenko.ru/dima/homethings/internal/dto"
- "git.dmitriygnatenko.ru/dima/homethings/internal/helpers/test"
- "git.dmitriygnatenko.ru/dima/homethings/internal/models"
- )
- // nolint:errcheck
- func TestAddImageHandler(t *testing.T) {
- t.Parallel()
- type req struct {
- method string
- route string
- contentType string
- body []byte
- }
- var (
- placeID = gofakeit.Uint64()
- thingID = gofakeit.Uint64()
- testError = gofakeit.Error()
- txMockFunc = func(ctx context.Context, f func(ctx context.Context) error) error {
- return f(ctx)
- }
- )
- // Correct request for adding place image
- addPlaceCorrectBody := &bytes.Buffer{}
- addPlaceCorrectWriter := multipart.NewWriter(addPlaceCorrectBody)
- addPlaceCorrectWriter.WriteField("place_id", strconv.FormatUint(placeID, 10))
- 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.FormatUint(thingID, 10))
- 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{}
- tmMock func(mc *minimock.Controller) TransactionManager
- fileRepoMock func(mc *minimock.Controller) FileRepository
- placeImageRepoMock func(mc *minimock.Controller) PlaceImageRepository
- thingImageRepoMock func(mc *minimock.Controller) ThingImageRepository
- }{
- {
- name: "negative case - bad request",
- req: req{
- method: fiber.MethodPost,
- route: "/v1/images",
- },
- resCode: fiber.StatusBadRequest,
- tmMock: func(mc *minimock.Controller) TransactionManager {
- return mocks.NewTransactionManagerMock(mc)
- },
- placeImageRepoMock: func(mc *minimock.Controller) PlaceImageRepository {
- return mocks.NewPlaceImageRepositoryMock(mc)
- },
- thingImageRepoMock: func(mc *minimock.Controller) ThingImageRepository {
- return mocks.NewThingImageRepositoryMock(mc)
- },
- fileRepoMock: func(mc *minimock.Controller) FileRepository {
- return mocks.NewFileRepositoryMock(mc)
- },
- },
- {
- name: "negative case - incorrect request",
- req: req{
- method: fiber.MethodPost,
- route: "/v1/images",
- body: addPlaceIncorrectBody.Bytes(),
- contentType: addPlaceIncorrectContentType,
- },
- resCode: fiber.StatusBadRequest,
- tmMock: func(mc *minimock.Controller) TransactionManager {
- return mocks.NewTransactionManagerMock(mc)
- },
- placeImageRepoMock: func(mc *minimock.Controller) PlaceImageRepository {
- return mocks.NewPlaceImageRepositoryMock(mc)
- },
- thingImageRepoMock: func(mc *minimock.Controller) ThingImageRepository {
- return mocks.NewThingImageRepositoryMock(mc)
- },
- fileRepoMock: func(mc *minimock.Controller) FileRepository {
- return mocks.NewFileRepositoryMock(mc)
- },
- },
- {
- name: "negative case - incorrect request",
- req: req{
- method: fiber.MethodPost,
- route: "/v1/images",
- body: addThingIncorrectBody.Bytes(),
- contentType: addThingIncorrectContentType,
- },
- resCode: fiber.StatusBadRequest,
- tmMock: func(mc *minimock.Controller) TransactionManager {
- return mocks.NewTransactionManagerMock(mc)
- },
- placeImageRepoMock: func(mc *minimock.Controller) PlaceImageRepository {
- return mocks.NewPlaceImageRepositoryMock(mc)
- },
- thingImageRepoMock: func(mc *minimock.Controller) ThingImageRepository {
- return mocks.NewThingImageRepositoryMock(mc)
- },
- fileRepoMock: func(mc *minimock.Controller) FileRepository {
- return mocks.NewFileRepositoryMock(mc)
- },
- },
- {
- name: "negative case - empty request",
- req: req{
- method: fiber.MethodPost,
- route: "/v1/images",
- body: emptyBody.Bytes(),
- contentType: emptyContentType,
- },
- resCode: fiber.StatusBadRequest,
- tmMock: func(mc *minimock.Controller) TransactionManager {
- return mocks.NewTransactionManagerMock(mc)
- },
- placeImageRepoMock: func(mc *minimock.Controller) PlaceImageRepository {
- return mocks.NewPlaceImageRepositoryMock(mc)
- },
- thingImageRepoMock: func(mc *minimock.Controller) ThingImageRepository {
- return mocks.NewThingImageRepositoryMock(mc)
- },
- fileRepoMock: func(mc *minimock.Controller) FileRepository {
- return mocks.NewFileRepositoryMock(mc)
- },
- },
- {
- name: "negative case - repository error (add image)",
- req: req{
- method: fiber.MethodPost,
- route: "/v1/images",
- body: addPlaceCorrectBody.Bytes(),
- contentType: addPlaceCorrectContentType,
- },
- resCode: fiber.StatusInternalServerError,
- tmMock: func(mc *minimock.Controller) TransactionManager {
- mock := mocks.NewTransactionManagerMock(mc)
- mock.ReadCommittedMock.Set(txMockFunc)
- return mock
- },
- placeImageRepoMock: func(mc *minimock.Controller) PlaceImageRepository {
- mock := mocks.NewPlaceImageRepositoryMock(mc)
- mock.AddMock.Inspect(func(ctx context.Context, req models.AddPlaceImageRequest) {
- assert.Equal(mc, placeID, req.PlaceID)
- }).Return(testError)
- return mock
- },
- thingImageRepoMock: func(mc *minimock.Controller) ThingImageRepository {
- return mocks.NewThingImageRepositoryMock(mc)
- },
- fileRepoMock: func(mc *minimock.Controller) FileRepository {
- mock := mocks.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{},
- tmMock: func(mc *minimock.Controller) TransactionManager {
- mock := mocks.NewTransactionManagerMock(mc)
- mock.ReadCommittedMock.Set(txMockFunc)
- return mock
- },
- placeImageRepoMock: func(mc *minimock.Controller) PlaceImageRepository {
- mock := mocks.NewPlaceImageRepositoryMock(mc)
- mock.AddMock.Inspect(func(ctx context.Context, req models.AddPlaceImageRequest) {
- assert.Equal(mc, placeID, req.PlaceID)
- }).Return(nil)
- return mock
- },
- thingImageRepoMock: func(mc *minimock.Controller) ThingImageRepository {
- return mocks.NewThingImageRepositoryMock(mc)
- },
- fileRepoMock: func(mc *minimock.Controller) FileRepository {
- mock := mocks.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,
- tmMock: func(mc *minimock.Controller) TransactionManager {
- mock := mocks.NewTransactionManagerMock(mc)
- mock.ReadCommittedMock.Set(txMockFunc)
- return mock
- },
- placeImageRepoMock: func(mc *minimock.Controller) PlaceImageRepository {
- return mocks.NewPlaceImageRepositoryMock(mc)
- },
- thingImageRepoMock: func(mc *minimock.Controller) ThingImageRepository {
- mock := mocks.NewThingImageRepositoryMock(mc)
- mock.AddMock.Inspect(func(ctx context.Context, req models.AddThingImageRequest) {
- assert.Equal(mc, thingID, req.ThingID)
- }).Return(testError)
- return mock
- },
- fileRepoMock: func(mc *minimock.Controller) FileRepository {
- mock := mocks.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{},
- tmMock: func(mc *minimock.Controller) TransactionManager {
- mock := mocks.NewTransactionManagerMock(mc)
- mock.ReadCommittedMock.Set(txMockFunc)
- return mock
- },
- placeImageRepoMock: func(mc *minimock.Controller) PlaceImageRepository {
- return mocks.NewPlaceImageRepositoryMock(mc)
- },
- thingImageRepoMock: func(mc *minimock.Controller) ThingImageRepository {
- mock := mocks.NewThingImageRepositoryMock(mc)
- mock.AddMock.Inspect(func(ctx context.Context, req models.AddThingImageRequest) {
- assert.Equal(mc, thingID, req.ThingID)
- }).Return(nil)
- return mock
- },
- fileRepoMock: func(mc *minimock.Controller) FileRepository {
- mock := mocks.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,
- tmMock: func(mc *minimock.Controller) TransactionManager {
- return mocks.NewTransactionManagerMock(mc)
- },
- placeImageRepoMock: func(mc *minimock.Controller) PlaceImageRepository {
- return mocks.NewPlaceImageRepositoryMock(mc)
- },
- thingImageRepoMock: func(mc *minimock.Controller) ThingImageRepository {
- return mocks.NewThingImageRepositoryMock(mc)
- },
- fileRepoMock: func(mc *minimock.Controller) FileRepository {
- mock := mocks.NewFileRepositoryMock(mc)
- mock.SaveMock.Return(testError)
- return mock
- },
- },
- }
- for _, tt := range tests {
- t.Run(tt.name, func(t *testing.T) {
- t.Parallel()
- mc := minimock.NewController(t)
- fiberApp := fiber.New()
- fiberApp.Post("/v1/images", AddImageHandler(
- tt.tmMock(mc),
- tt.fileRepoMock(mc),
- tt.thingImageRepoMock(mc),
- tt.placeImageRepoMock(mc),
- ))
- 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, test.TestTimeout)
- assert.Equal(t, tt.resCode, fiberRes.StatusCode)
- if tt.resBody != nil {
- assert.Equal(t, test.MarshalResponse(tt.resBody), test.ConvertBodyToString(fiberRes.Body))
- }
- })
- }
- }
|