wallet: add tx store edge-case itests

Add additive integration coverage for the remaining CreateTx and UpdateTx edge conditions that were still below the db-only coverage target. These tests keep the branch focused on public store behavior while pushing the split backend files above the current 90 percent goal.

Author: yyforyongyu

wallet/internal/db/itest/tx_store_create_edge_cases_test.go

@@ -91,3 +91,45 @@ func TestCreateTxRejectsDuplicateConfirmedTransaction(t *testing.T) {
 	err = store.CreateTx(t.Context(), params)
 	require.ErrorIs(t, err, db.ErrTxAlreadyExists)
 }
+
+// TestCreateTxRejectsMissingConfirmingBlockForExistingUnminedRow verifies that
+// re-confirming an existing unmined row still requires the confirming block to
+// exist in block history.
+func TestCreateTxRejectsMissingConfirmingBlockForExistingUnminedRow(t *testing.T) {
+	t.Parallel()
+
+	store := NewTestStore(t)
+	walletID := newWallet(t, store, "wallet-missing-confirming-block")
+	createDerivedAccount(t, store, walletID, db.KeyScopeBIP0084, "default")
+
+	addr := newDerivedAddress(
+		t, store, walletID, db.KeyScopeBIP0084, "default", false,
+	)
+	tx := newRegularTx(
+		[]wire.OutPoint{randomOutPoint()},
+		[]*wire.TxOut{{Value: 4600, PkScript: addr.ScriptPubKey}},
+	)
+
+	err := store.CreateTx(t.Context(), db.CreateTxParams{
+		WalletID: walletID,
+		Tx:       tx,
+		Received: time.Unix(1710000860, 0),
+		Status:   db.TxStatusPending,
+		Credits:  map[uint32]btcutil.Address{0: nil},
+	})
+	require.NoError(t, err)
+
+	err = store.CreateTx(t.Context(), db.CreateTxParams{
+		WalletID: walletID,
+		Tx:       tx,
+		Received: time.Unix(1710000861, 0),
+		Block: &db.Block{
+			Hash:      RandomHash(),
+			Height:    999,
+			Timestamp: time.Unix(1710000862, 0),
+		},
+		Status:  db.TxStatusPublished,
+		Credits: map[uint32]btcutil.Address{0: nil},
+	})
+	require.ErrorContains(t, err, "require confirming block")
+}

wallet/internal/db/itest/tx_store_edge_cases_test.go

@@ -3,6 +3,7 @@
 package itest
 
 import (
+	"strings"
 	"testing"
 	"time"
 
@@ -161,3 +162,32 @@ func TestTxReadsReturnQueryErrorsWhenClosed(t *testing.T) {
 	})
 	require.ErrorContains(t, err, "list transactions by height")
 }
+
+// TestUpdateTxRejectsTooLongLabel verifies that backend label updates surface
+// query errors when the requested label exceeds the stored column limit.
+func TestUpdateTxRejectsTooLongLabel(t *testing.T) {
+	t.Parallel()
+
+	store := NewTestStore(t)
+	walletID := newWallet(t, store, "wallet-update-label-too-long")
+
+	tx := newRegularTx(
+		[]wire.OutPoint{randomOutPoint()},
+		[]*wire.TxOut{{Value: 9100, PkScript: []byte{0x51}}},
+	)
+	err := store.CreateTx(t.Context(), db.CreateTxParams{
+		WalletID: walletID,
+		Tx:       tx,
+		Received: time.Unix(1710001600, 0),
+		Status:   db.TxStatusPending,
+	})
+	require.NoError(t, err)
+
+	label := strings.Repeat("x", 501)
+	err = store.UpdateTx(t.Context(), db.UpdateTxParams{
+		WalletID: walletID,
+		Txid:     tx.TxHash(),
+		Label:    &label,
+	})
+	require.ErrorContains(t, err, "update transaction label")
+}