Merge remote-tracking branch 'upstream/main' into test/currency-dropd…

…own-widget
TBD54566975 · Oct 11, 2024 · 4821eb9 · 4821eb9
2 parents 6a2e10f + 89bbc71
commit 4821eb9
Show file tree

Hide file tree

Showing 4 changed files with 116 additions and 5 deletions.
diff --git a/lib/features/payment/payment_details_page.dart b/lib/features/payment/payment_details_page.dart
@@ -66,7 +66,7 @@ class PaymentDetailsPage extends HookConsumerWidget {
 
     return PopScope(
       canPop: !isAwaiting,
-      onPopInvoked: (_) {
+      onPopInvokedWithResult: (_, __) {
         if (isAwaiting) {
           ref.read(quoteProvider.notifier).stopPolling();
           quote.value = null;

diff --git a/lib/features/payment/payment_review_page.dart b/lib/features/payment/payment_review_page.dart
@@ -38,7 +38,7 @@ class PaymentReviewPage extends HookConsumerWidget {
 
     return PopScope(
       canPop: false,
-      onPopInvoked: (didPop) async {
+      onPopInvokedWithResult: (didPop, _) async {
         if (didPop) {
           return;
         }

diff --git a/test/features/account/account_balance_card_test.dart b/test/features/account/account_balance_card_test.dart
@@ -0,0 +1,102 @@
+import 'package:didpay/features/account/account_balance_card.dart';
+import 'package:didpay/features/account/account_balance_notifier.dart';
+import 'package:didpay/features/did/did_provider.dart';
+import 'package:didpay/features/pfis/pfis_notifier.dart';
+import 'package:flutter/material.dart';
+import 'package:flutter_test/flutter_test.dart';
+import 'package:mocktail/mocktail.dart';
+
+import '../../helpers/mocks.dart';
+import '../../helpers/test_data.dart';
+import '../../helpers/widget_helpers.dart';
+
+void main() async {
+  await TestData.initializeDids();
+
+  final accountBalance = TestData.getAccountBalance();
+  final did = TestData.aliceDid;
+  final pfis = TestData.getPfis();
+
+  late MockPfisNotifier mockPfisNotifier;
+  late MockAccountBalanceNotifier mockAccountBalanceNotifier;
+
+  group('AccountBalanceCard', () {
+    setUpAll(() {
+      registerFallbackValue(did);
+      registerFallbackValue(pfis);
+    });
+
+    setUp(() {
+      mockPfisNotifier = MockPfisNotifier(pfis);
+      mockAccountBalanceNotifier = MockAccountBalanceNotifier(accountBalance);
+
+      when(
+        () => mockAccountBalanceNotifier.startPolling(any(), any()),
+      ).thenAnswer((_) async => accountBalance);
+    });
+
+    Widget accountBalanceCardTestWidget({
+      bool pfisIsEmpty = false,
+    }) =>
+        WidgetHelpers.testableWidget(
+          child: const AccountBalanceCard(),
+          overrides: [
+            pfisProvider.overrideWith(
+              (ref) => pfisIsEmpty ? MockPfisNotifier([]) : mockPfisNotifier,
+            ),
+            accountBalanceProvider
+                .overrideWith(() => mockAccountBalanceNotifier),
+            didProvider.overrideWith((ref) => did),
+          ],
+        );
+
+    testWidgets('should show account balance title', (tester) async {
+      await tester.pumpWidget(accountBalanceCardTestWidget());
+      await tester.pumpAndSettle();
+
+      expect(find.text('Account balance'), findsOneWidget);
+    });
+
+    testWidgets('should show account balance amount', (tester) async {
+      await tester.pumpWidget(accountBalanceCardTestWidget());
+      await tester.pumpAndSettle();
+
+      expect(find.text('101'), findsOneWidget);
+      expect(find.text('USD'), findsOneWidget);
+    });
+
+    group('if pfis is empty', () {
+      testWidgets('should not show deposit button', (tester) async {
+        await tester
+            .pumpWidget(accountBalanceCardTestWidget(pfisIsEmpty: true));
+        await tester.pumpAndSettle();
+
+        expect(find.widgetWithText(FilledButton, 'Deposit'), findsNothing);
+      });
+
+      testWidgets('should not show withdraw button', (tester) async {
+        await tester
+            .pumpWidget(accountBalanceCardTestWidget(pfisIsEmpty: true));
+        await tester.pumpAndSettle();
+
+        expect(find.widgetWithText(FilledButton, 'Withdraw'), findsNothing);
+      });
+    });
+
+    group('if pfis is not empty', () {
+      testWidgets('should show deposit button', (tester) async {
+        await tester.pumpWidget(accountBalanceCardTestWidget());
+        await tester.pumpAndSettle();
+
+        expect(find.widgetWithText(FilledButton, 'Deposit'), findsOneWidget);
+      });
+
+      testWidgets('should show withdraw button', (tester) async {
+        await tester.pumpWidget(accountBalanceCardTestWidget());
+        await tester.pumpAndSettle();
+
+        expect(find.widgetWithText(FilledButton, 'Withdraw'), findsOneWidget);
+      });
+    });
+  });
+}
diff --git a/test/helpers/test_data.dart b/test/helpers/test_data.dart
@@ -27,8 +27,16 @@ class TestData {
   static FeatureFlag getFeatureFlag(String name, String description) =>
       FeatureFlag(name: name, description: description);
 
-  static Map<Pfi, List<Offering>> getOfferingsMap() => {
-        Pfi(did: pfiDid.uri): [getOffering()],
+  static Map<Pfi, List<Offering>> getOfferingsMap({
+    List<String>? payoutCurrencies,
+  }) =>
+      {
+        Pfi(did: pfiDid.uri):
+            (payoutCurrencies != null && payoutCurrencies.isNotEmpty)
+                ? payoutCurrencies
+                    .map((currency) => getOffering(payoutCurrency: currency))
+                    .toList()
+                : [getOffering()],
       };
 
   static List<Pfi> getPfis() => [Pfi(did: pfiDid.uri)];
@@ -37,6 +45,7 @@ class TestData {
       AccountBalance(total: '101', currencyCode: 'USD', balancesMap: {});
 
   static Offering getOffering({
+    String? payoutCurrency,
     PresentationDefinition? requiredClaims,
     List<PayinMethod>? payinMethods,
     List<PayoutMethod>? payoutMethods,
@@ -59,7 +68,7 @@ class TestData {
                 ],
           ),
           payout: PayoutDetails(
-            currencyCode: 'USD',
+            currencyCode: payoutCurrency ?? 'USD',
             methods: payoutMethods ??
                 [
                   PayoutMethod(