mirror of
https://github.com/hanzo-fi/payments.git
synced 2026-10-30 20:31:20 +00:00
CrimsonandGitHub
b54eb950a2
* Use validation library in v3 API payments initiation endpoint * Add stricter validation for payment initiations in v3 API * Provide user friendly validation errors * Fix validation errors in e2e tests * Use validation library for v2 payment initiation response * Add custom error strings for custom validation types and simplify validator interface * Add validation to payment initiation reversal endpoints * Update open API specs to reflect stricter validation
90 lines
3.1 KiB
Go
90 lines
3.1 KiB
Go
package v2
|
|
|
|
import (
|
|
"errors"
|
|
"math/big"
|
|
"net/http"
|
|
"net/http/httptest"
|
|
|
|
"github.com/formancehq/payments/internal/api/backend"
|
|
"github.com/formancehq/payments/internal/api/validation"
|
|
"github.com/formancehq/payments/internal/models"
|
|
"github.com/google/uuid"
|
|
. "github.com/onsi/ginkgo/v2"
|
|
"go.uber.org/mock/gomock"
|
|
)
|
|
|
|
var _ = Describe("API v2 Transfer Initiation Reversal", func() {
|
|
var (
|
|
handlerFn http.HandlerFunc
|
|
paymentID models.PaymentInitiationID
|
|
)
|
|
BeforeEach(func() {
|
|
connID := models.ConnectorID{Reference: uuid.New(), Provider: "psp"}
|
|
paymentID = models.PaymentInitiationID{Reference: "ref", ConnectorID: connID}
|
|
})
|
|
|
|
Context("reverse transfer initiation", func() {
|
|
var (
|
|
w *httptest.ResponseRecorder
|
|
m *backend.MockBackend
|
|
)
|
|
BeforeEach(func() {
|
|
w = httptest.NewRecorder()
|
|
ctrl := gomock.NewController(GinkgoT())
|
|
m = backend.NewMockBackend(ctrl)
|
|
handlerFn = transferInitiationsReverse(m, validation.NewValidator())
|
|
})
|
|
|
|
It("should return a bad request error when transferInitiationID is invalid", func(ctx SpecContext) {
|
|
req := prepareQueryRequest(http.MethodGet, "transferInitiationID", "invalid")
|
|
handlerFn(w, req)
|
|
|
|
assertExpectedResponse(w.Result(), http.StatusBadRequest, ErrInvalidID)
|
|
})
|
|
|
|
It("should return a bad request error when body is missing", func(ctx SpecContext) {
|
|
handlerFn(w, prepareQueryRequest(http.MethodGet, "transferInitiationID", paymentID.String()))
|
|
|
|
assertExpectedResponse(w.Result(), http.StatusBadRequest, ErrMissingOrInvalidBody)
|
|
})
|
|
|
|
DescribeTable("validation errors",
|
|
func(r reverseTransferInitiationRequest) {
|
|
handlerFn(w, prepareJSONRequestWithQuery(http.MethodPost, "transferInitiationID", paymentID.String(), &r))
|
|
assertExpectedResponse(w.Result(), http.StatusBadRequest, ErrValidation)
|
|
},
|
|
Entry("reference missing", reverseTransferInitiationRequest{}),
|
|
Entry("amount missing", reverseTransferInitiationRequest{Reference: "amount"}),
|
|
Entry("asset missing", reverseTransferInitiationRequest{Reference: "asset", Amount: big.NewInt(1313)}),
|
|
)
|
|
|
|
It("should return an internal server error when backend returns error", func(ctx SpecContext) {
|
|
expectedErr := errors.New("payment initiation reverse err")
|
|
m.EXPECT().PaymentInitiationReversalsCreate(gomock.Any(), gomock.Any(), true).Return(
|
|
models.Task{},
|
|
expectedErr,
|
|
)
|
|
handlerFn(w, prepareJSONRequestWithQuery(http.MethodPost, "transferInitiationID", paymentID.String(), &reverseTransferInitiationRequest{
|
|
Reference: "ref1",
|
|
Amount: big.NewInt(1313),
|
|
Asset: "USD/2",
|
|
}))
|
|
assertExpectedResponse(w.Result(), http.StatusInternalServerError, "INTERNAL")
|
|
})
|
|
|
|
It("should return status no content on success", func(ctx SpecContext) {
|
|
m.EXPECT().PaymentInitiationReversalsCreate(gomock.Any(), gomock.Any(), true).Return(
|
|
models.Task{},
|
|
nil,
|
|
)
|
|
handlerFn(w, prepareJSONRequestWithQuery(http.MethodPost, "transferInitiationID", paymentID.String(), &reverseTransferInitiationRequest{
|
|
Reference: "ref1",
|
|
Amount: big.NewInt(1313),
|
|
Asset: "USD/2",
|
|
}))
|
|
assertExpectedResponse(w.Result(), http.StatusNoContent, "")
|
|
})
|
|
})
|
|
})
|