Files
fgueryandGitHub a63e3a20a8 feat(EN-214): create events via storage instead of manually calling event activity (#594)
* feat(EN-214): add outbox table migration

* feat(EN-214): add outbox models and storage layer

* feat(EN-214): implement outbox publisher activity

* feat(EN-214): implement outbox schedule activity

* feat(EN-214): add outbox publisher workflow

* feat(EN-214): migrate accounts to outbox pattern

* feat(EN-214): migrate balances to outbox pattern

* feat(EN-214): migrate payments to outbox pattern

* feat(EN-214): migrate bank accounts to outbox pattern

* feat(EN-214): migrate pools to outbox pattern

* feat(EN-214): migrate payment initiations to outbox pattern

* feat(EN-214): migrate PSU and connections to outbox pattern

* feat(EN-214): migrate connectors to outbox pattern

* feat(EN-214): remove legacy send_events workflow

* feat(EN-214): update workflows to use outbox pattern

* feat(EN-214): improve outbox idempotency and error handling

* feat(EN-214): update storage tasks and tests

* test(EN-214): update e2e tests for outbox pattern

* test(EN-214): clean up workflow tests

* chore(EN-214): update docker compose configuration

* feat(EN-214): add e2e run config in ide

* fix: add back docker-compose-dev run config

* chore: improve connector uninstall comments & remove connector id being passed

* fix: ensure events are not pushed if balances are not created

* fix: ensure err is not overwritten when using rollback

* fix: ensure we use v3 provider format

* chore: apply the nitpick comments from coderabbit

* chore: consistentely use e() error handling

* feat: optionally disable publisher schedule to make e2e test more consistent

* fix: improve error handling on uninstall connector

* avoid using custom payload and rely on existing interfaces; also avoid using different types for events

* fix: remove unecessary unmarshalling

* fix: ensure that rollbackOnTxError picks up errors on tx.Commit()

* fix: add extra validation on iban & account number in the test

* test: cover marshall/unmarshalling of event payload

* fix: increase test timeout

* try to make test more resiliant

* fix: wrong pointer use in loop

* fix: error formatting issue

* fix: move oubtox schedule creation out of an activity and do it in the worker itself

* feat: OutboxEventDeleteAndSend as batch

* change outbox_event to have id only instead of idempotency key

* make e2e test less brittle, save eventID as idempotency key in publication

* feat: make outbox_event keep an history

* feat(EN-214): added a schedule to cleanup outbox events

* test: make cleanup actually clean

* ignore linting rules for now

* feat: do not add psu metadata to event anymore as it's irrelevant

* fix: create local variable instead of relying on pointers, even if not 100% necessary

* fix: remove temporal schedule describe calls
2025-11-24 16:05:42 +00:00

237 lines
5.7 KiB
Go

package events
import (
"encoding/json"
"math/big"
"testing"
"time"
"github.com/stretchr/testify/assert"
"github.com/stretchr/testify/require"
)
func TestPaymentMessagePayload_MarshalJSON(t *testing.T) {
t.Parallel()
t.Run("with amounts", func(t *testing.T) {
t.Parallel()
initialAmount := big.NewInt(1000000)
amount := big.NewInt(999999)
payload := PaymentMessagePayload{
ID: "pay123",
ConnectorID: "conn123",
Provider: "test",
Reference: "ref123",
CreatedAt: time.Date(2024, 1, 1, 12, 0, 0, 0, time.UTC),
Type: "PAYIN",
Status: "SUCCEEDED",
Scheme: "SEPA",
Asset: "USD/2",
InitialAmount: initialAmount,
Amount: amount,
}
data, err := json.Marshal(&payload)
require.NoError(t, err)
var result map[string]interface{}
err = json.Unmarshal(data, &result)
require.NoError(t, err)
assert.Equal(t, "pay123", result["id"])
assert.Equal(t, "1000000", result["initialAmount"])
assert.Equal(t, "999999", result["amount"])
})
t.Run("with nil amounts", func(t *testing.T) {
t.Parallel()
payload := PaymentMessagePayload{
ID: "pay123",
ConnectorID: "conn123",
Provider: "test",
Reference: "ref123",
CreatedAt: time.Date(2024, 1, 1, 12, 0, 0, 0, time.UTC),
Type: "PAYIN",
Status: "SUCCEEDED",
Scheme: "SEPA",
Asset: "USD/2",
InitialAmount: nil,
Amount: nil,
}
data, err := json.Marshal(&payload)
require.NoError(t, err)
var result map[string]interface{}
err = json.Unmarshal(data, &result)
require.NoError(t, err)
assert.Nil(t, result["initialAmount"])
assert.Nil(t, result["amount"])
})
t.Run("with one nil amount", func(t *testing.T) {
t.Parallel()
amount := big.NewInt(500000)
payload := PaymentMessagePayload{
ID: "pay123",
ConnectorID: "conn123",
Provider: "test",
Reference: "ref123",
CreatedAt: time.Date(2024, 1, 1, 12, 0, 0, 0, time.UTC),
Type: "PAYIN",
Status: "SUCCEEDED",
Scheme: "SEPA",
Asset: "USD/2",
InitialAmount: nil,
Amount: amount,
}
data, err := json.Marshal(&payload)
require.NoError(t, err)
var result map[string]interface{}
err = json.Unmarshal(data, &result)
require.NoError(t, err)
assert.Nil(t, result["initialAmount"])
assert.Equal(t, "500000", result["amount"])
})
}
func TestPaymentMessagePayload_UnmarshalJSON(t *testing.T) {
t.Parallel()
t.Run("with valid amount strings", func(t *testing.T) {
t.Parallel()
jsonData := `{
"id": "pay123",
"connectorID": "conn123",
"provider": "test",
"reference": "ref123",
"createdAt": "2024-01-01T12:00:00Z",
"type": "PAYIN",
"status": "SUCCEEDED",
"scheme": "SEPA",
"asset": "USD/2",
"rawData": {},
"initialAmount": "1000000",
"amount": "999999"
}`
var payload PaymentMessagePayload
err := json.Unmarshal([]byte(jsonData), &payload)
require.NoError(t, err)
assert.Equal(t, "pay123", payload.ID)
assert.NotNil(t, payload.InitialAmount)
assert.Equal(t, "1000000", payload.InitialAmount.String())
assert.NotNil(t, payload.Amount)
assert.Equal(t, "999999", payload.Amount.String())
})
t.Run("with nil amounts", func(t *testing.T) {
t.Parallel()
jsonData := `{
"id": "pay123",
"connectorID": "conn123",
"provider": "test",
"reference": "ref123",
"createdAt": "2024-01-01T12:00:00Z",
"type": "PAYIN",
"status": "SUCCEEDED",
"scheme": "SEPA",
"asset": "USD/2",
"rawData": {}
}`
var payload PaymentMessagePayload
err := json.Unmarshal([]byte(jsonData), &payload)
require.NoError(t, err)
assert.Nil(t, payload.InitialAmount)
assert.Nil(t, payload.Amount)
})
t.Run("with invalid initialAmount string", func(t *testing.T) {
t.Parallel()
jsonData := `{
"id": "pay123",
"connectorID": "conn123",
"provider": "test",
"reference": "ref123",
"createdAt": "2024-01-01T12:00:00Z",
"type": "PAYIN",
"status": "SUCCEEDED",
"scheme": "SEPA",
"asset": "USD/2",
"rawData": {},
"initialAmount": "invalid-number"
}`
var payload PaymentMessagePayload
err := json.Unmarshal([]byte(jsonData), &payload)
require.Error(t, err)
assert.Contains(t, err.Error(), "invalid initialAmount string")
})
t.Run("with invalid amount string", func(t *testing.T) {
t.Parallel()
jsonData := `{
"id": "pay123",
"connectorID": "conn123",
"provider": "test",
"reference": "ref123",
"createdAt": "2024-01-01T12:00:00Z",
"type": "PAYIN",
"status": "SUCCEEDED",
"scheme": "SEPA",
"asset": "USD/2",
"rawData": {},
"amount": "invalid-number"
}`
var payload PaymentMessagePayload
err := json.Unmarshal([]byte(jsonData), &payload)
require.Error(t, err)
assert.Contains(t, err.Error(), "invalid amount string")
})
t.Run("round-trip marshalling", func(t *testing.T) {
t.Parallel()
original := PaymentMessagePayload{
ID: "pay123",
ConnectorID: "conn123",
Provider: "test",
Reference: "ref123",
CreatedAt: time.Date(2024, 1, 1, 12, 0, 0, 0, time.UTC),
Type: "PAYIN",
Status: "SUCCEEDED",
Scheme: "SEPA",
Asset: "USD/2",
InitialAmount: big.NewInt(1000000),
Amount: big.NewInt(999999),
}
data, err := json.Marshal(&original)
require.NoError(t, err)
var unmarshaled PaymentMessagePayload
err = json.Unmarshal(data, &unmarshaled)
require.NoError(t, err)
assert.Equal(t, original.ID, unmarshaled.ID)
assert.Equal(t, original.InitialAmount.String(), unmarshaled.InitialAmount.String())
assert.Equal(t, original.Amount.String(), unmarshaled.Amount.String())
})
}