CW 882: Integrate Seed Verification Flow To Integration Tests

cw_solana/lib/solana_client.dart

@@ -151,7 +151,7 @@ class SolanaWalletClient {
         transactionDetails.addAll(response);
 
         // to avoid reaching the node RPS limit
-        await Future.delayed(Duration(milliseconds: 500));
+        await Future.delayed(const Duration(milliseconds: 500));
       }
 
       for (final tx in transactionDetails) {
@@ -379,18 +379,16 @@ class SolanaWalletClient {
     required double solBalance,
     required double fee,
   }) async {
-    return true;
-    // TODO: this is not doing what the name inclines
-    // final rent =
-    //     await _client!.getMinimumBalanceForMintRentExemption(commitment: Commitment.confirmed);
-    //
-    // final rentInSol = (rent / lamportsPerSol).toDouble();
-    //
-    // final remnant = solBalance - (inputAmount + fee);
-    //
-    // if (remnant > rentInSol) return true;
-    //
-    // return false;
+    final rent =
+        await _client!.getMinimumBalanceForMintRentExemption(commitment: Commitment.confirmed);
+
+    final rentInSol = (rent / lamportsPerSol).toDouble();
+
+    final remnant = solBalance - (inputAmount + fee);
+
+    if (remnant > rentInSol) return true;
+
+    return false;
   }
 
   Future<PendingSolanaTransaction> _signNativeTokenTransaction({
@@ -542,7 +540,7 @@ class SolanaWalletClient {
           ),
         );
 
-        await Future.delayed(Duration(seconds: 5));
+        await Future.delayed(const Duration(seconds: 5));
       }
     } catch (e) {
       throw SolanaCreateAssociatedTokenAccountException(e.toString());
@@ -569,7 +567,7 @@ class SolanaWalletClient {
     );
 
     bool hasSufficientFundsLeft = await hasSufficientFundsLeftForRent(
-      inputAmount: inputAmount,
+      inputAmount: 0,
       fee: fee,
       solBalance: solBalance,
     );
@@ -586,12 +584,12 @@ class SolanaWalletClient {
     );
 
     sendTx() async {
-      await Future.delayed(Duration(seconds: 3));
+      await Future.delayed(const Duration(seconds: 3));
 
       return await sendTransaction(
-          signedTransaction: signedTx,
-          commitment: commitment,
-        );
+        signedTransaction: signedTx,
+        commitment: commitment,
+      );
     }
 
     final pendingTransaction = PendingSolanaTransaction(

integration_test/components/common_test_flows.dart

@@ -17,6 +17,7 @@ import '../robots/new_wallet_type_page_robot.dart';
 import '../robots/pre_seed_page_robot.dart';
 import '../robots/restore_from_seed_or_key_robot.dart';
 import '../robots/restore_options_page_robot.dart';
+import '../robots/seed_verification_page_robot.dart';
 import '../robots/setup_pin_code_robot.dart';
 import '../robots/wallet_group_description_page_robot.dart';
 import '../robots/wallet_list_page_robot.dart';
@@ -40,6 +41,7 @@ class CommonTestFlows {
         _walletListPageRobot = WalletListPageRobot(_tester),
         _newWalletTypePageRobot = NewWalletTypePageRobot(_tester),
         _restoreOptionsPageRobot = RestoreOptionsPageRobot(_tester),
+        _seedVerificationPageRobot = SeedVerificationPageRobot(_tester),
         _createPinWelcomePageRobot = CreatePinWelcomePageRobot(_tester),
         _restoreFromSeedOrKeysPageRobot = RestoreFromSeedOrKeysPageRobot(_tester),
         _walletGroupDescriptionPageRobot = WalletGroupDescriptionPageRobot(_tester);
@@ -58,6 +60,7 @@ class CommonTestFlows {
   final NewWalletTypePageRobot _newWalletTypePageRobot;
   final RestoreOptionsPageRobot _restoreOptionsPageRobot;
   final CreatePinWelcomePageRobot _createPinWelcomePageRobot;
+  final SeedVerificationPageRobot _seedVerificationPageRobot;
   final RestoreFromSeedOrKeysPageRobot _restoreFromSeedOrKeysPageRobot;
   final WalletGroupDescriptionPageRobot _walletGroupDescriptionPageRobot;
 
@@ -87,7 +90,8 @@ class CommonTestFlows {
     await _confirmPreSeedInfo();
 
     await _confirmWalletDetails();
-    await _commonTestCases.defaultSleepTime();
+
+    await _verifyWalletSeed();
   }
 
   //* ========== Handles flow from welcome to restoring wallet from seeds ===============
@@ -150,6 +154,9 @@ class CommonTestFlows {
     await _confirmPreSeedInfo();
 
     await _confirmWalletDetails();
+
+    await _verifyWalletSeed();
+
     await _commonTestCases.defaultSleepTime();
   }
 
@@ -240,8 +247,10 @@ class CommonTestFlows {
 
     if (Platform.isLinux) {
       // manual pin input
-      await _restoreFromSeedOrKeysPageRobot.enterPasswordForWalletRestore(CommonTestConstants.pin.join(""));
-      await _restoreFromSeedOrKeysPageRobot.enterPasswordRepeatForWalletRestore(CommonTestConstants.pin.join(""));
+      await _restoreFromSeedOrKeysPageRobot
+          .enterPasswordForWalletRestore(CommonTestConstants.pin.join(""));
+      await _restoreFromSeedOrKeysPageRobot
+          .enterPasswordRepeatForWalletRestore(CommonTestConstants.pin.join(""));
     }
 
     await _newWalletPageRobot.onNextButtonPressed();
@@ -264,13 +273,17 @@ class CommonTestFlows {
 
     // await _walletSeedPageRobot.onCopySeedsButtonPressed();
 
-    await _walletSeedPageRobot.onSeedPageVerifyButtonPressed();
-    // Turns out the popup about "Copied to clipboard" prevents
-    //the button from being pressed on the first try, by just
-    //tapping it again we fix it.
-    // await _walletSeedPageRobot.onSeedPageVerifyButtonPressed();
-
-    await _walletSeedPageRobot.onOpenWalletButtonPressed();
+    await _walletSeedPageRobot.onVerifySeedButtonPressed();
+  }
+
+  //* ============ Handles Wallet Seed Verification Page ==================
+
+  Future<void> _verifyWalletSeed() async {
+    await _seedVerificationPageRobot.isSeedVerificationPage();
+
+    _seedVerificationPageRobot.hasTitle();
+
+    await _seedVerificationPageRobot.verifyWalletSeeds();
   }
 
   //* Main Restore Actions - On the RestoreFromSeed/Keys Page - Restore from Seeds Action
@@ -293,8 +306,10 @@ class CommonTestFlows {
 
     if (Platform.isLinux) {
       // manual pin input
-      await _restoreFromSeedOrKeysPageRobot.enterPasswordForWalletRestore(CommonTestConstants.pin.join(""));
-      await _restoreFromSeedOrKeysPageRobot.enterPasswordRepeatForWalletRestore(CommonTestConstants.pin.join(""));
+      await _restoreFromSeedOrKeysPageRobot
+          .enterPasswordForWalletRestore(CommonTestConstants.pin.join(""));
+      await _restoreFromSeedOrKeysPageRobot
+          .enterPasswordRepeatForWalletRestore(CommonTestConstants.pin.join(""));
     }
 
     await _restoreFromSeedOrKeysPageRobot.onRestoreWalletButtonPressed();

integration_test/robots/seed_verification_page_robot.dart

@@ -0,0 +1,41 @@
+import 'package:cake_wallet/generated/i18n.dart';
+import 'package:cake_wallet/src/screens/seed/seed_verification/seed_verification_page.dart';
+import 'package:flutter_test/flutter_test.dart';
+
+import '../components/common_test_cases.dart';
+
+class SeedVerificationPageRobot {
+  SeedVerificationPageRobot(this.tester) : commonTestCases = CommonTestCases(tester);
+
+  final WidgetTester tester;
+  final CommonTestCases commonTestCases;
+
+  Future<void> isSeedVerificationPage() async {
+    await commonTestCases.isSpecificPage<SeedVerificationPage>();
+  }
+
+  void hasTitle() {
+    commonTestCases.hasText(S.current.verify_seed);
+  }
+
+  Future<void> verifyWalletSeeds() async {
+    final seedVerificationPage =
+        tester.widget<SeedVerificationPage>(find.byType(SeedVerificationPage));
+
+    final walletSeedViewModel = seedVerificationPage.walletSeedViewModel;
+
+    while (!walletSeedViewModel.isVerificationComplete) {
+      final currentCorrectWord = walletSeedViewModel.currentCorrectWord;
+
+      await commonTestCases.tapItemByKey(
+        'seed_verification_option_${currentCorrectWord}_button_key',
+      );
+
+      await commonTestCases.defaultSleepTime(seconds: 1);
+    }
+
+    await commonTestCases.tapItemByKey('wallet_seed_page_open_wallet_button_key');
+
+    await commonTestCases.defaultSleepTime();
+  }
+}

integration_test/robots/wallet_seed_page_robot.dart

@@ -14,13 +14,13 @@ class WalletSeedPageRobot {
     await commonTestCases.isSpecificPage<WalletSeedPage>();
   }
 
-  Future<void> onSeedPageVerifyButtonPressed() async {
+  Future<void> onVerifySeedButtonPressed() async {
     await commonTestCases.tapItemByKey('wallet_seed_page_verify_seed_button_key');
     await commonTestCases.defaultSleepTime();
   }
 
-  Future<void> onOpenWalletButtonPressed() async {
-    await commonTestCases.tapItemByKey('wallet_seed_page_open_wallet_button_key');
+  Future<void> onSaveSeedButtonPressed() async {
+    await commonTestCases.tapItemByKey('wallet_seed_page_save_seeds_button_key');
     await commonTestCases.defaultSleepTime();
   }
 
@@ -40,12 +40,10 @@ class WalletSeedPageRobot {
     final walletSeedViewModel = walletSeedPage.walletSeedViewModel;
 
     final walletName = walletSeedViewModel.name;
-    final walletSeeds = walletSeedViewModel.seed;
-
+    final walletSeeds = walletSeedViewModel.seedSplit;
     commonTestCases.hasText(walletName);
-    final seedList = walletSeeds.trim().split(" ");
-    for (final seedWord in seedList) {
-      commonTestCases.hasTextAtLestOnce(seedWord);
+    for (var seed in walletSeeds) {
+      commonTestCases.hasTextAtLestOnce(seed);
     }
   }
 

lib/main.dart

@@ -23,6 +23,7 @@ import 'package:cake_wallet/routes.dart';
 import 'package:cake_wallet/src/screens/root/root.dart';
 import 'package:cake_wallet/store/app_store.dart';
 import 'package:cake_wallet/store/authentication_store.dart';
+import 'package:cake_wallet/test_asset_bundles.dart';
 import 'package:cake_wallet/themes/theme_base.dart';
 import 'package:cake_wallet/utils/device_info.dart';
 import 'package:cake_wallet/utils/exception_handler.dart';
@@ -80,8 +81,18 @@ Future<void> runAppWithZone({Key? topLevelKey}) async {
         ledgerFile.writeAsStringSync("$content\n${event.message}");
       });
     }
+    // Basically when we're running a test
+    if (topLevelKey != null) {
+      runApp(
+        DefaultAssetBundle(
+          bundle: TestAssetBundle(),
+          child: App(key: topLevelKey),
+        ),
+      );
+    } else {
+      runApp(App(key: topLevelKey));
+    }
 
-    runApp(App(key: topLevelKey));
     isAppRunning = true;
   }, (error, stackTrace) async {
     if (!isAppRunning) {

lib/src/screens/seed/seed_verification/seed_verification_page.dart

@@ -23,9 +23,11 @@ class SeedVerificationPage extends BasePage {
           child: walletSeedViewModel.isVerificationComplete ||
                   walletSeedViewModel.verificationIndices.isEmpty
               ? SeedVerificationSuccessView(
+                  key: ValueKey('seed_verification_success_view_page'),
                   imageColor: titleColor(context),
                 )
               : SeedVerificationStepView(
+                  key: ValueKey('seed_verification_step_view_page'),
                   walletSeedViewModel: walletSeedViewModel,
                   questionTextColor: titleColor(context),
                 ),

lib/src/screens/seed/seed_verification/seed_verification_step_view.dart

@@ -69,6 +69,7 @@ class SeedVerificationStepView extends StatelessWidget {
                   children: walletSeedViewModel.currentOptions.map(
                     (option) {
                       return GestureDetector(
+                        key: ValueKey('seed_verification_option_${option}_button_key'),
                         onTap: () async {
                           if (walletSeedViewModel.wrongEntries > 2) return;
 

lib/test_asset_bundles.dart

@@ -0,0 +1,15 @@
+import 'dart:convert';
+
+import 'package:flutter/services.dart';
+
+class TestAssetBundle extends CachingAssetBundle {
+  @override
+  Future<String> loadString(String key, {bool cache = true}) async {
+    final ByteData data = await load(key);
+
+    return utf8.decode(data.buffer.asUint8List());
+  }
+
+  @override
+  Future<ByteData> load(String key) async => rootBundle.load(key);
+}

lib/utils/feature_flag.dart

@@ -5,5 +5,5 @@ class FeatureFlag {
   static const bool isExolixEnabled = true;
   static const bool isInAppTorEnabled = false;
   static const bool isBackgroundSyncEnabled = false;
-  static const int verificationWordsCount = kDebugMode ? 0 : 2;
-}
+  static const int verificationWordsCount = 2;
+}