feat: enhance transaction detail page to refresh on txid change with tests

Author: j-kon

bdk_demo/lib/features/wallet_setup/transaction_detail_page.dart

@@ -19,16 +19,29 @@ class TransactionDetailPage extends ConsumerStatefulWidget {
 }
 
 class _TransactionDetailPageState extends ConsumerState<TransactionDetailPage> {
-  late final Future<TxDetails?> _transactionFuture;
+  late Future<TxDetails?> _transactionFuture;
 
-  @override
-  void initState() {
-    super.initState();
+  void _loadTransactionFuture() {
     _transactionFuture = ref
         .read(walletServiceProvider)
         .loadTransactionByTxid(widget.txid);
   }
 
+  @override
+  void initState() {
+    super.initState();
+    _loadTransactionFuture();
+  }
+
+  @override
+  void didUpdateWidget(covariant TransactionDetailPage oldWidget) {
+    super.didUpdateWidget(oldWidget);
+
+    if (oldWidget.txid != widget.txid) {
+      _loadTransactionFuture();
+    }
+  }
+
   String _formatAmount(TxDetails transaction) {
     final amount = transaction.netAmount;
     final prefix = amount >= 0 ? '+' : '-';

bdk_demo/test/widget_test.dart

@@ -190,6 +190,61 @@ void main() {
     },
   );
 
+  testWidgets('Transaction detail page refreshes when txid changes', (
+    tester,
+  ) async {
+    final fakeWalletService = FakeWalletService(
+      walletInfo: _testWalletInfo,
+      transactions: _placeholderTransactions,
+    );
+
+    Future<void> pumpDetail(String txid) async {
+      await tester.pumpWidget(
+        ProviderScope(
+          overrides: [
+            walletServiceProvider.overrideWithValue(fakeWalletService),
+          ],
+          child: MaterialApp(
+            home: TransactionDetailPage(
+              key: const ValueKey('detail-page'),
+              txid: txid,
+            ),
+          ),
+        ),
+      );
+    }
+
+    await pumpDetail(_placeholderTransactions.first.txid);
+    await tester.pumpAndSettle();
+
+    expect(
+      find.text(
+        '1234567890abcdef1234567890abcdef1234567890abcdef1234567890abcd',
+      ),
+      findsOneWidget,
+    );
+    expect(find.text('January 2 2024 03:04'), findsOneWidget);
+
+    await pumpDetail(_placeholderTransactions.last.txid);
+    await tester.pumpAndSettle();
+
+    expect(
+      find.text(
+        'abcdef1234567890abcdef1234567890abcdef1234567890abcdef1234567890',
+      ),
+      findsOneWidget,
+    );
+    expect(find.text('-1600 sat'), findsNWidgets(2));
+    expect(find.text('pending'), findsNWidgets(2));
+    expect(
+      find.text(
+        '1234567890abcdef1234567890abcdef1234567890abcdef1234567890abcd',
+      ),
+      findsNothing,
+    );
+    expect(find.text('January 2 2024 03:04'), findsNothing);
+  });
+
   testWidgets('Transaction detail page handles a missing tx gracefully', (
     tester,
   ) async {