feat(SPV-1059) fee and change

.golangci-lint.yml

@@ -96,12 +96,17 @@ issues:
     # Exclude some linters from running on tests files.
     - path: _test\.go
       linters:
-        - gocyclo
         - errcheck
         - gosec
         - wrapcheck
         - bodyclose
 
+    - path: /testabilities/
+      linters:
+        - gosec
+        - wrapcheck
+        - bodyclose
+
     - path: tester/
       linters:
         - gocyclo

actions/testabilities/assert_spvwallet_application.go

@@ -8,6 +8,7 @@ import (
 	"testing"
 
 	"github.com/bitcoin-sv/spv-wallet/actions/testabilities/apierror"
+	testpaymail "github.com/bitcoin-sv/spv-wallet/engine/paymail/testabilities"
 	testengine "github.com/bitcoin-sv/spv-wallet/engine/testabilities"
 	"github.com/bitcoin-sv/spv-wallet/engine/tester/fixtures"
 	"github.com/bitcoin-sv/spv-wallet/engine/tester/jsonrequire"
@@ -19,7 +20,7 @@ import (
 type SPVWalletApplicationAssertions interface {
 	Response(response *resty.Response) SPVWalletResponseAssertions
 	User(user fixtures.User) SPVWalletAppUserAssertions
-	ExternalPaymailHost() testengine.PaymailExternalAssertions
+	ExternalPaymailHost() testpaymail.PaymailExternalAssertions
 }
 
 type SPVWalletResponseAssertions interface {
@@ -73,7 +74,7 @@ func (a *appAssertions) User(user fixtures.User) SPVWalletAppUserAssertions {
 	}
 }
 
-func (a *appAssertions) ExternalPaymailHost() testengine.PaymailExternalAssertions {
+func (a *appAssertions) ExternalPaymailHost() testpaymail.PaymailExternalAssertions {
 	return a.engineAssertions.ExternalPaymailHost()
 }
 

actions/v2/transactions/internal/mapping/outline_tx.go

@@ -80,6 +80,14 @@ func outlineOutputEntryToResponse(index int, value *transaction.OutputAnnotation
 func outlineOutputToResponse(from *transaction.OutputAnnotation) api.ModelsOutputAnnotation {
 	return api.ModelsOutputAnnotation{
 		Bucket: api.ModelsOutputAnnotationBucket(from.Bucket),
+		CustomInstructions: lo.IfF(
+			from.CustomInstructions != nil,
+			func() *api.ModelsSPVWalletCustomInstructions {
+				return lo.ToPtr(
+					lo.Map(*from.CustomInstructions, lox.MappingFn(customInstructionsToResponse)),
+				)
+			},
+		).Else(nil),
 		Paymail: lo.IfF(
 			from.Paymail != nil,
 			func() *api.ModelsPaymailAnnotationDetails {

actions/v2/transactions/internal/testabilities/assert_transaction_details.go

@@ -0,0 +1,51 @@
+package testabilities
+
+import (
+	"testing"
+
+	sdk "github.com/bitcoin-sv/go-sdk/transaction"
+	"github.com/bitcoin-sv/spv-wallet/engine/tester/fixtures"
+	"github.com/bitcoin-sv/spv-wallet/engine/v2/transaction"
+	"github.com/bitcoin-sv/spv-wallet/models/bsv"
+	"github.com/stretchr/testify/require"
+)
+
+type TransactionDetailsAssertions interface {
+	WithOutputValues(values ...bsv.Satoshis) TransactionDetailsAssertions
+	OutputUnlockableBy(vout int, user fixtures.User) TransactionDetailsAssertions
+}
+
+type transactionAssertions struct {
+	t           testing.TB
+	tx          *sdk.Transaction
+	require     *require.Assertions
+	annotations transaction.Annotations
+}
+
+func (a *transactionAssertions) WithOutputValues(values ...bsv.Satoshis) TransactionDetailsAssertions {
+	a.t.Helper()
+	a.require.Lenf(a.tx.Outputs, len(values), "Tx has less outputs then expected values")
+	for i, v := range values {
+		a.require.Equal(v, bsv.Satoshis(a.tx.Outputs[i].Satoshis), "output value mismatch")
+	}
+	return a
+}
+
+func (a *transactionAssertions) OutputUnlockableBy(vout int, user fixtures.User) TransactionDetailsAssertions {
+	a.t.Helper()
+	a.require.Less(vout, len(a.tx.Outputs), "output index out of range")
+
+	outputAnnotation, ok := a.annotations.Outputs[vout]
+	if !ok {
+		a.t.Fatalf("output %d has no required annotation", vout)
+	}
+	a.require.NotNil(outputAnnotation.CustomInstructions, "output %d has no custom instructions", vout)
+
+	fixtures.GivenTX(a.t).
+		WithSender(user).
+		WithInputFromUTXO(a.tx, uint32(vout), *outputAnnotation.CustomInstructions...).
+		WithOPReturn("dummy data").
+		TX() // during TX call, the transaction is signed. Should fail if the UTXO cannot be unlocked by the user.
+
+	return a
+}

actions/v2/transactions/internal/testabilities/assert_transactions_endpoint.go

@@ -16,8 +16,7 @@ type TransactionsEndpointAssertions interface {
 
 type TransactionsResponseAssertions interface {
 	testabilities.SPVWalletResponseAssertions
-	ContainsValidBEEFHexInField(field string) TransactionsResponseAssertions
-	ContainsValidRawTxHexInField(field string) TransactionsResponseAssertions
+	ContainsValidTransaction(format string) TransactionDetailsAssertions
 }
 
 func Then(t testing.TB, app testabilities.SPVWalletApplicationFixture) TransactionsEndpointAssertions {
@@ -50,16 +49,23 @@ type transactionResponseAssertions struct {
 	assert   *assert.Assertions
 }
 
-func (a *transactionResponseAssertions) ContainsValidBEEFHexInField(field string) TransactionsResponseAssertions {
-	txHex := a.SPVWalletResponseAssertions.JSONValue().GetString(field)
-	_, err := sdk.NewTransactionFromBEEFHex(txHex)
-	a.require.NoError(err, "hex is not valid BEEF")
-	return a
-}
-
-func (a *transactionResponseAssertions) ContainsValidRawTxHexInField(field string) TransactionsResponseAssertions {
-	txHex := a.SPVWalletResponseAssertions.JSONValue().GetString(field)
-	_, err := sdk.NewTransactionFromHex(txHex)
-	a.require.NoError(err, "hex is not valid raw tx")
-	return a
+func (a *transactionResponseAssertions) ContainsValidTransaction(format string) TransactionDetailsAssertions {
+	a.t.Helper()
+	txHex := a.SPVWalletResponseAssertions.JSONValue().GetString("hex")
+	var tx *sdk.Transaction
+	var err error
+	switch format {
+	case "BEEF":
+		tx, err = sdk.NewTransactionFromBEEFHex(txHex)
+	case "RAW":
+		tx, err = sdk.NewTransactionFromHex(txHex)
+	default:
+		a.t.Fatalf("unsupported format: %s", format)
+	}
+	a.require.NoError(err, "hex is not valid tx in format %s", format)
+	return &transactionAssertions{
+		t:       a.t,
+		require: a.require,
+		tx:      tx,
+	}
 }

actions/v2/transactions/internal/testabilities/fixture_transactions_endpoints.go

@@ -6,6 +6,7 @@ import (
 
 	"github.com/bitcoin-sv/spv-wallet/actions/testabilities"
 	"github.com/bitcoin-sv/spv-wallet/engine/tester/fixtures"
+	"github.com/bitcoin-sv/spv-wallet/engine/tester/paymailmock"
 )
 
 // ExpectedCustomInstructions represents json array with custom instructions of UTXO created with Faucet().TopUp() method
@@ -18,7 +19,7 @@ const ExpectedCustomInstructions = `[
 
 type TransactionsEndpointFixture interface {
 	testabilities.SPVWalletApplicationFixture
-	OutlineResponseContext(additionalParams map[string]any) map[string]any
+	OutlineResponseContext(format string, additionalParams map[string]any) map[string]any
 }
 
 func Given(t testing.TB) TransactionsEndpointFixture {
@@ -31,11 +32,14 @@ type fixture struct {
 	testabilities.SPVWalletApplicationFixture
 }
 
-func (f *fixture) OutlineResponseContext(paramsOverride map[string]any) map[string]any {
-	params := make(map[string]any)
-	params["ReceiverPaymail"] = fixtures.RecipientExternal.DefaultPaymail()
-	params["SenderPaymail"] = fixtures.Sender.DefaultPaymail()
-	params["CustomInstructions"] = ExpectedCustomInstructions
+func (f *fixture) OutlineResponseContext(format string, paramsOverride map[string]any) map[string]any {
+	params := map[string]any{
+		"Format":             format,
+		"ReceiverPaymail":    fixtures.RecipientExternal.DefaultPaymail(),
+		"SenderPaymail":      fixtures.Sender.DefaultPaymail(),
+		"CustomInstructions": ExpectedCustomInstructions,
+		"Reference":          paymailmock.MockedReferenceID,
+	}
 
 	maps.Copy(params, paramsOverride)