CW 882: Integrate Seed Verification Flow To Integration Tests #1966

Blazebrain · 2025-01-16T13:07:17Z

Issue Number (if Applicable): Fixes #

Description

Please include a summary of the changes and which issue is fixed / feature is added.

Pull Request - Checklist

Initial Manual Tests Passed
Double check modified code and verify it with the feature/task requirements
Format code
Look for code duplication
Clear naming for variables and methods

…Integrate-Seed-Verification-Flow-To-Integration-Tests

…ion-Tests

… test script to add a watch dog to prevent infinite running on CI

https://github.com/cake-tech/cake_wallet into Integrate-Seed-Verification-Flow-To-Integration-Tests

lib/utils/feature_flag.dart

…d flow test

… and resulting errors

OmarHatem28 · 2025-01-28T12:54:38Z

integration_test/robots/transactions_page_robot.dart

-    //* ======Confirm it displays the properly formatted title===========
-    final transactionType = dashboardViewModel.getTransactionType(item.transaction);
-
-    final title = item.formattedTitle + item.formattedStatus + transactionType;
-
-    commonTestCases.findWidgetViaDescendant(
-      of: find.byKey(ValueKey(keyId)),
-      matching: find.text(title),
-    );


why this was removed?

forgot to add a TODO there, there's been some inconsistencies with the wordings for the title, something from the flutter end as it doesn't constantly happen. Happens once, and when you rerun, it doesn't happen again. I think I mentioned earlier on one of the calls.

integration_test/test_suites/exchange_flow_test.dart

…Integrate-Seed-Verification-Flow-To-Integration-Tests

# Conflicts: # lib/src/screens/wallet_keys/wallet_keys_page.dart

fix tests not working in latest main

…Integrate-Seed-Verification-Flow-To-Integration-Tests

https://github.com/cake-tech/cake_wallet into Integrate-Seed-Verification-Flow-To-Integration-Tests

…Integrate-Seed-Verification-Flow-To-Integration-Tests

- Pause screenshots for now, results in a super large json response file - Fix delay on send bottomsheet because of pumpAndSettle animation - Fix issue with integration test runner script

…Integrate-Seed-Verification-Flow-To-Integration-Tests

https://github.com/cake-tech/cake_wallet into Integrate-Seed-Verification-Flow-To-Integration-Tests

…tests. This change: - Handles the slider during the send confirmation flow during the send integration test - Adds the missing seeds for the restore flow to the relevant workflow files

OmarHatem28

and they failed on my local testing

OmarHatem28 · 2025-04-28T12:49:48Z

integration_test/test_suites/transaction_history_flow_test.dart

+    /// For bitcoin/Monero/Wownero WalletTypes.
    await commonTestFlows.welcomePageToRestoreWalletThroughSeedsFlow(
-      WalletType.solana,
-      secrets.solanaTestWalletSeeds,
+      WalletType.bitcoin,
+      secrets.bitcoinTestWalletSeeds,


this only covers bitcoin

OmarHatem28 · 2025-04-28T12:50:01Z

integration_test/test_suites/transaction_history_flow_test.dart

+    /// For Solana/Tron WalletTypes.
+    await dashboardPageRobot.navigateToWalletsListPage();

    await commonTestFlows.restoreWalletFromWalletMenu(
-      WalletType.bitcoin,
-      secrets.bitcoinTestWalletSeeds,
+      WalletType.solana,


same here, it only covers solana

Blazebrain and others added 16 commits December 20, 2024 10:36

fix: Generic fixes

25a8393

Merge branch 'main' of https://github.com/cake-tech/cake_wallet

e3c4f7a

fix: Modify hasSufficientFundsForRent to work for spl transactions

1335627

feat: Integrate seed verification into integration tests

ecc0156

Merge branch 'main' of https://github.com/cake-tech/cake_wallet into …

f5cc30b

…Integrate-Seed-Verification-Flow-To-Integration-Tests

fix: Adjust workflow file

50c82e0

chore: Remove previous working directory

ff3606a

fix: Revert change

4707dc8

chore: Remove step to setup kvm, already existing in new setup

164ae7e

chore: Remove a couple of steps from workflow

45aed18

chore: Try switching test commands

3ca0334

fix: Switch back to flutter drive

5060cbe

fix: Flutter error when running tests with Flutter drive

85f8ebb

Merge branch 'main' into Integrate-Seed-Verification-Flow-To-Integrat…

ee067cc

…ion-Tests

feat: Add screenshots to individual test pages and modify integration…

b0602f8

… test script to add a watch dog to prevent infinite running on CI

Merge branch 'Integrate-Seed-Verification-Flow-To-Integration-Tests' of

3216a87

https://github.com/cake-tech/cake_wallet into Integrate-Seed-Verification-Flow-To-Integration-Tests

OmarHatem28 reviewed Jan 17, 2025

View reviewed changes

lib/utils/feature_flag.dart Outdated Show resolved Hide resolved

Blazebrain added 4 commits January 17, 2025 16:38

feat: Implement transaction success info robot and fix issue with sen…

17285bd

…d flow test

fix: Improve integration testing for exchange flow and clean up logic…

640766d

… and resulting errors

fix: Handle duplicate words for seeds and test restore through seeds

b83cf66

fix: Optimize implementation for transactions history test

35d5a10

OmarHatem28 added the Work in Progress label Jan 27, 2025

OmarHatem28 reviewed Jan 28, 2025

View reviewed changes

Blazebrain added 2 commits February 11, 2025 16:31

Merge branch 'main' of https://github.com/cake-tech/cake_wallet into …

2e8038b

…Integrate-Seed-Verification-Flow-To-Integration-Tests

Skip seed verification when in debug and when not in CI flow

f640062

Blazebrain requested a review from OmarHatem28 February 11, 2025 19:38

Blazebrain and others added 4 commits February 11, 2025 20:42

chore: Remove print statements and add TODOs

adc9f3f

fix: Upgrade version for upload action

33d0033

fix: Add missing secrets to testing workflow file

1ebc087

fix tests not working in latest main [skip slack]

4c9e37c

# Conflicts: # lib/src/screens/wallet_keys/wallet_keys_page.dart

MrCyjaneK and others added 25 commits February 12, 2025 18:03

add zano seed [skip slack] [run tests]

5bfa166

disable exchange_flow_test.dart [run tests] [skip slack]

dda2927

Run tests on linux always (since nobody ever used [run tests])

1396e97

fix tests after ui updates

6881707

[skip slack] tests update

0d2b0a3

Merge pull request #1999 from cake-tech/cyjan-fix-tests

1f60c4e

fix tests not working in latest main

Merge branch 'main' of https://github.com/cake-tech/cake_wallet into …

cd0d049

…Integrate-Seed-Verification-Flow-To-Integration-Tests

Merge branch 'Integrate-Seed-Verification-Flow-To-Integration-Tests' of

bbe72f4

https://github.com/cake-tech/cake_wallet into Integrate-Seed-Verification-Flow-To-Integration-Tests

Fixing tests WIP

b9d6d6d

Merge branch 'main' of https://github.com/cake-tech/cake_wallet into …

0f7c4ee

…Integrate-Seed-Verification-Flow-To-Integration-Tests

Fix create wallet integraiton test flow

d6c7beb

Update and fix Exchange flow test flow

74fc9d7

Switch printV to printToConsole

1711bb1

Merge branch 'main' of https://github.com/cake-tech/cake_wallet into …

aad7320

…Integrate-Seed-Verification-Flow-To-Integration-Tests

Merge branch 'main' of https://github.com/cake-tech/cake_wallet into …

e285236

…Integrate-Seed-Verification-Flow-To-Integration-Tests

Merge branch 'main' of https://github.com/cake-tech/cake_wallet into …

c24d531

…Integrate-Seed-Verification-Flow-To-Integration-Tests

Adjustments to the the flows on Send and Exchange

3eea48d

Merge branch 'main' of https://github.com/cake-tech/cake_wallet into …

9faec1c

…Integrate-Seed-Verification-Flow-To-Integration-Tests

feat: Modify Exchange and send tests to fit new flow

c2e77c0

fix:

26506a9

- Pause screenshots for now, results in a super large json response file - Fix delay on send bottomsheet because of pumpAndSettle animation - Fix issue with integration test runner script

Merge branch 'main' of https://github.com/cake-tech/cake_wallet into …

629cdf0

…Integrate-Seed-Verification-Flow-To-Integration-Tests

Merge branch 'main' of https://github.com/cake-tech/cake_wallet into …

5ff769e

…Integrate-Seed-Verification-Flow-To-Integration-Tests

Merge branch 'main' of https://github.com/cake-tech/cake_wallet into …

9e3d64e

…Integrate-Seed-Verification-Flow-To-Integration-Tests

Merge branch 'Integrate-Seed-Verification-Flow-To-Integration-Tests' of

55d32ab

https://github.com/cake-tech/cake_wallet into Integrate-Seed-Verification-Flow-To-Integration-Tests

fix(integration-tests): Fix issues relating to send flow integration …

717bb0c

…tests. This change: - Handles the slider during the send confirmation flow during the send integration test - Adds the missing seeds for the restore flow to the relevant workflow files

OmarHatem28 requested changes Apr 30, 2025

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

CW 882: Integrate Seed Verification Flow To Integration Tests #1966

CW 882: Integrate Seed Verification Flow To Integration Tests #1966

Blazebrain commented Jan 16, 2025

OmarHatem28 Jan 28, 2025

Blazebrain Jan 28, 2025

OmarHatem28 left a comment

OmarHatem28 Apr 28, 2025

OmarHatem28 Apr 28, 2025

CW 882: Integrate Seed Verification Flow To Integration Tests #1966

Are you sure you want to change the base?

CW 882: Integrate Seed Verification Flow To Integration Tests #1966

Conversation

Blazebrain commented Jan 16, 2025

Description

Pull Request - Checklist

OmarHatem28 Jan 28, 2025

Choose a reason for hiding this comment

Blazebrain Jan 28, 2025

Choose a reason for hiding this comment

OmarHatem28 left a comment

Choose a reason for hiding this comment

OmarHatem28 Apr 28, 2025

Choose a reason for hiding this comment

OmarHatem28 Apr 28, 2025

Choose a reason for hiding this comment