CW 882: Integrate Seed Verification Flow To Integration Tests

.github/workflows/pr_test_build_android.yml

@@ -159,6 +159,7 @@ jobs:
           echo "const tronTestWalletSeeds = '${{ secrets.TRON_TEST_WALLET_SEEDS }}';" >> lib/.secrets.g.dart
           echo "const nanoTestWalletSeeds =  '${{ secrets.NANO_TEST_WALLET_SEEDS }}';" >> lib/.secrets.g.dart
           echo "const wowneroTestWalletSeeds = '${{ secrets.WOWNERO_TEST_WALLET_SEEDS }}';" >> lib/.secrets.g.dart
+          echo "const zanoTestWalletSeeds = '${{ secrets.ZANO_TEST_WALLET_SEEDS }}';" >> lib/.secrets.g.dart
           echo "const moneroTestWalletReceiveAddress = '${{ secrets.MONERO_TEST_WALLET_RECEIVE_ADDRESS }}';" >> lib/.secrets.g.dart
           echo "const bitcoinTestWalletReceiveAddress =  '${{ secrets.BITCOIN_TEST_WALLET_RECEIVE_ADDRESS }}';" >> lib/.secrets.g.dart
           echo "const ethereumTestWalletReceiveAddress =  '${{ secrets.ETHEREUM_TEST_WALLET_RECEIVE_ADDRESS }}';" >> lib/.secrets.g.dart

.github/workflows/pr_test_build_linux.yml

@@ -152,6 +152,8 @@ jobs:
           echo "const tronTestWalletSeeds = '${{ secrets.TRON_TEST_WALLET_SEEDS }}';" >> lib/.secrets.g.dart
           echo "const nanoTestWalletSeeds =  '${{ secrets.NANO_TEST_WALLET_SEEDS }}';" >> lib/.secrets.g.dart
           echo "const wowneroTestWalletSeeds = '${{ secrets.WOWNERO_TEST_WALLET_SEEDS }}';" >> lib/.secrets.g.dart
+          echo "const zanoTestWalletSeeds = '${{ secrets.ZANO_TEST_WALLET_SEEDS }}';" >> lib/.secrets.g.dart
+          echo "const decredTestWalletSeeds = '${{ secrets.DECRED_TEST_WALLET_SEEDS }}';" >> lib/.secrets.g.dart
           echo "const moneroTestWalletReceiveAddress = '${{ secrets.MONERO_TEST_WALLET_RECEIVE_ADDRESS }}';" >> lib/.secrets.g.dart
           echo "const bitcoinTestWalletReceiveAddress =  '${{ secrets.BITCOIN_TEST_WALLET_RECEIVE_ADDRESS }}';" >> lib/.secrets.g.dart
           echo "const ethereumTestWalletReceiveAddress =  '${{ secrets.ETHEREUM_TEST_WALLET_RECEIVE_ADDRESS }}';" >> lib/.secrets.g.dart
@@ -162,6 +164,8 @@ jobs:
           echo "const tronTestWalletReceiveAddress =  '${{ secrets.TRON_TEST_WALLET_RECEIVE_ADDRESS }}';" >> lib/.secrets.g.dart
           echo "const nanoTestWalletReceiveAddress = '${{ secrets.NANO_TEST_WALLET_RECEIVE_ADDRESS }}';" >> lib/.secrets.g.dart
           echo "const wowneroTestWalletReceiveAddress = '${{ secrets.WOWNERO_TEST_WALLET_RECEIVE_ADDRESS }}';" >> lib/.secrets.g.dart
+          echo "const zanoTestWalletReceiveAddress = '${{ secrets.ZANO_TEST_WALLET_RECEIVE_ADDRESS }}';" >> lib/.secrets.g.dart
+          echo "const decredTestWalletReceiveAddress = '${{ secrets.DECRED_TEST_WALLET_RECEIVE_ADDRESS }}';" >> lib/.secrets.g.dart
           echo "const moneroTestWalletBlockHeight =  '${{ secrets.MONERO_TEST_WALLET_BLOCK_HEIGHT }}';" >> lib/.secrets.g.dart
           # end of test secrets
           echo "const chainflipApiKey = '${{ secrets.CHAINFLIP_API_KEY }}';" >> lib/.secrets.g.dart
@@ -240,7 +244,7 @@ jobs:
           name: cakewallet_linux
 
       - name: Prepare virtual desktop
-        if: ${{ contains(env.message, 'run tests') }}
+        if: ${{ !contains(env.message, 'skip tests') }}
         run: |
             nohup Xvfb :99 -screen 0 720x1280x16 &
             echo DISPLAY=:99 | sudo tee -a $GITHUB_ENV
@@ -256,28 +260,28 @@ jobs:
       #   isn't much in those wallets anyway, we still wouldn't like to leak it to anyone who is able to access github.
 
       - name: Test [confirm_seeds_flow_test]
-        if: ${{ contains(env.message, 'run tests') }}
+        if: ${{ !contains(env.message, 'skip tests') }}
         timeout-minutes: 20
         run: |
           xmessage -timeout 30 "confirm_seeds_flow_test" &
           rm -rf ~/.local/share/com.example.cake_wallet/ ~/Documents/cake_wallet/ ~/cake_wallet
           exec timeout --signal=SIGKILL 900 flutter drive --driver=test_driver/integration_test.dart --target=integration_test/test_suites/confirm_seeds_flow_test.dart
       - name: Test [create_wallet_flow_test]
-        if: ${{ contains(env.message, 'run tests') }}
+        if: ${{ !contains(env.message, 'skip tests') }}
         timeout-minutes: 20
         run: |
           xmessage -timeout 30 "create_wallet_flow_test" &
           rm -rf ~/.local/share/com.example.cake_wallet/ ~/Documents/cake_wallet/ ~/cake_wallet
           exec timeout --signal=SIGKILL 900 flutter drive --driver=test_driver/integration_test.dart --target=integration_test/test_suites/create_wallet_flow_test.dart
-      - name: Test [exchange_flow_test]
-        if: ${{ contains(env.message, 'run tests') }}
-        timeout-minutes: 20
-        run: |
-          xmessage -timeout 30 "exchange_flow_test" &
-          rm -rf ~/.local/share/com.example.cake_wallet/ ~/Documents/cake_wallet/ ~/cake_wallet
-          exec timeout --signal=SIGKILL 900 flutter drive --driver=test_driver/integration_test.dart --target=integration_test/test_suites/exchange_flow_test.dart
+      # - name: Test [exchange_flow_test]
+      #   if: ${{ !contains(env.message, 'skip tests') }}
+      #   timeout-minutes: 20
+      #   run: |
+      #     xmessage -timeout 30 "exchange_flow_test" &
+      #     rm -rf ~/.local/share/com.example.cake_wallet/ ~/Documents/cake_wallet/ ~/cake_wallet
+      #     exec timeout --signal=SIGKILL 900 flutter drive --driver=test_driver/integration_test.dart --target=integration_test/test_suites/exchange_flow_test.dart
       - name: Test [restore_wallet_through_seeds_flow_test]
-        if: ${{ contains(env.message, 'run tests') }}
+        if: ${{ !contains(env.message, 'skip tests') }}
         timeout-minutes: 20
         run: |
           xmessage -timeout 30 "restore_wallet_through_seeds_flow_test" &

integration_test/components/common_test_cases.dart

@@ -1,5 +1,6 @@
 import 'package:flutter/material.dart';
 import 'package:flutter_test/flutter_test.dart';
+import 'package:integration_test/integration_test.dart';
 
 class CommonTestCases {
   WidgetTester tester;
@@ -15,7 +16,7 @@ class CommonTestCases {
     bool shouldPumpAndSettle = true,
     int pumpDuration = 100,
   }) async {
-    final widget = find.byKey(ValueKey(key));
+    final widget = find.byKey(ValueKey(key)).first;
     await tester.tap(widget);
     shouldPumpAndSettle
         ? await tester.pumpAndSettle(Duration(milliseconds: pumpDuration))
@@ -69,7 +70,33 @@ class CommonTestCases {
     await tester.pumpAndSettle();
   }
 
-  Future<void> dragUntilVisible(String childKey, String parentKey) async {
+  Future<void> startGesture(String key, Offset gestureOffset) async {
+    await tester.pumpAndSettle();
+
+    final hasKey = isKeyPresent(key);
+
+    tester.printToConsole("Has gestureKey: $hasKey");
+
+    if (!hasKey) return;
+
+    final gesture = await tester.startGesture(tester.getCenter(find.byKey(ValueKey(key))));
+
+    // Drag to the left
+    await gesture.moveBy(gestureOffset);
+
+    // End the gesture
+    await gesture.up();
+
+    await tester.pump();
+  }
+
+  Future<void> dragUntilVisible(
+    String childKey,
+    String parentKey, {
+    int maxScrolls = 100,
+    double scrollStep = 50.0,
+    int maxReverseScrolls = 50,
+  }) async {
     await tester.pumpAndSettle();
 
     final itemFinder = find.byKey(ValueKey(childKey));
@@ -82,18 +109,6 @@ class CommonTestCases {
       return;
     }
 
-    // We can adjust this as needed
-    final maxScrolls = 200;
-
-    int scrolls = 0;
-    bool found = false;
-
-    // We start by scrolling down
-    bool scrollDown = true;
-
-    // Flag to check if we've already reversed direction
-    bool reversedDirection = false;
-
     // Find the Scrollable associated with the Parent Ad
     final scrollableFinder = find.descendant(
       of: listFinder,
@@ -110,48 +125,90 @@ class CommonTestCases {
     // Get the initial scroll position
     final scrollableState = tester.state<ScrollableState>(scrollableFinder);
     double previousScrollPosition = scrollableState.position.pixels;
+    bool scrollDown = true;
+    bool reversedDirection = false;
+    bool found = false;
+
+    int reverseScrollCount = 0;
 
-    while (!found && scrolls < maxScrolls) {
+    for (int scrolls = 0; scrolls < maxScrolls; scrolls++) {
+      await tester.pumpAndSettle();
+
+      // Check if the widget is visible
+      if (tester.any(itemFinder)) {
+        found = true;
+        break;
+      }
+
+      // Log current state for debugging
       tester.printToConsole('Scrolling ${scrollDown ? 'down' : 'up'}, attempt $scrolls');
 
-      // Perform the drag in the current direction
+      // Stop if reverse scroll limit is exceeded
+      if (!scrollDown && reverseScrollCount >= maxReverseScrolls) {
+        tester.printToConsole('Maximum reverse scrolls reached. Widget not found.');
+        break;
+      }
+
+      // Perform scrolling in the current direction
       await tester.drag(
         scrollableFinder,
-        scrollDown ? const Offset(0, -100) : const Offset(0, 100),
+        scrollDown ? Offset(0, -scrollStep) : Offset(0, scrollStep),
       );
       await tester.pumpAndSettle();
-      scrolls++;
 
       // Update the scroll position after the drag
       final currentScrollPosition = scrollableState.position.pixels;
 
       if (currentScrollPosition == previousScrollPosition) {
-        // Cannot scroll further in this direction
+        // Cannot scroll further in the current direction
         if (reversedDirection) {
           // We've already tried both directions
-          tester.printToConsole('Cannot scroll further in both directions. Widget not found.');
+          tester.printToConsole('Reached the scroll limit in both directions. Widget not found.');
           break;
         } else {
-          // Reverse the scroll direction
+          // Reverse the scroll direction and reset reverse scroll count
           scrollDown = !scrollDown;
           reversedDirection = true;
+          reverseScrollCount = 0;
           tester.printToConsole('Reached the end, reversing direction');
         }
       } else {
-        // Continue scrolling in the current direction
+        // Update scroll position and reverse scroll count, incrementing only for reverse scrolling
         previousScrollPosition = currentScrollPosition;
+        if (!scrollDown) reverseScrollCount++;
       }
-
-      // Check if the widget is now in the widget tree
-      found = tester.any(itemFinder);
     }
 
     if (!found) {
-      tester.printToConsole('Widget not found after scrolling in both directions.');
-      return;
+      tester.printToConsole('Widget not found after $maxScrolls scrolls.');
     }
   }
 
+  Future<void> scrollItemIntoView(
+    String itemKeyId,
+    double scrollPixels,
+    String scrollableFinderKey,
+  ) async {
+    final Finder itemFinder = find.byKey(ValueKey(itemKeyId));
+
+    final scrollableFinder = find.descendant(
+      of: find.byKey(ValueKey(scrollableFinderKey)),
+      matching: find.byType(Scrollable),
+    );
+
+    try {
+      await tester.scrollUntilVisible(
+        itemFinder,
+        scrollPixels,
+        scrollable: scrollableFinder,
+      );
+    } catch (e) {
+      tester.printToConsole('Could not find $itemKeyId');
+    }
+
+    await tester.pumpAndSettle();
+  }
+
   Future<void> enterText(String text, String editableTextKey) async {
     final editableTextWidget = find.byKey(ValueKey((editableTextKey)));
 
@@ -171,4 +228,9 @@ class CommonTestCases {
 
   Future<void> defaultSleepTime({int seconds = 2}) async =>
       await Future.delayed(Duration(seconds: seconds));
+
+  Future<void> takeScreenshots(String screenshotName) async {
+    // Pausing this for now
+    // await (tester.binding as IntegrationTestWidgetsFlutterBinding).takeScreenshot(screenshotName);
+  }
 }

integration_test/components/common_test_constants.dart

@@ -7,7 +7,8 @@ class CommonTestConstants {
   static final String exchangeTestAmount = '0.01';
   static final WalletType testWalletType = WalletType.solana;
   static final String testWalletName = 'Integrated Testing Wallet';
-  static final CryptoCurrency testReceiveCurrency = CryptoCurrency.usdtSol;
-  static final CryptoCurrency testDepositCurrency = CryptoCurrency.sol;
+  static final CryptoCurrency sendTestReceiveCurrency = CryptoCurrency.sol;
+  static final CryptoCurrency exchangeTestReceiveCurrency = CryptoCurrency.usdtSol;
+  static final CryptoCurrency exchangeTestDepositCurrency = CryptoCurrency.sol;
   static final String testWalletAddress = '5v9gTW1yWPffhnbNKuvtL2frevAf4HpBMw8oYnfqUjhm';
 }