Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Setup Testing #1207

Open
wants to merge 170 commits into
base: main
Choose a base branch
from
Open

Setup Testing #1207

wants to merge 170 commits into from

Conversation

jigar-f
Copy link
Contributor

@jigar-f jigar-f commented Oct 7, 2024

Overview of This PR:

Initial Test Environment Setup-:Establishes the foundation for testing across the app.
Reusable Widget Testing Structure-: Creates a framework that can be reused for widget tests in future test cases.

Upcoming Work:

  • Widget Tests
  • Unit Tests (Models)
  • UI Tests (Cross-Platform)
  • Integration Tests
  • Payment Flow Tests (Introduce payment flow tests at a later stage)

Test Updates

List of app workflow test

  • App start up sequecne
  • Home flow with mock
  • VPN flow
  • Account flow
  • Setting flow
  • Report issue flow
  • Others are still redevelopment

Run Tests

Note: To run this integration on mobile make sure you have the device connected

To run Single test cases use this will work with widget and integration test
make runTest testfileName

To run all integration/app workflow tests use
make appWorkflowTest

To run all desktop test
make desktopWorkflowTest

To run a single test on a desktop
make runDesktopTest testfileName

@jigar-f jigar-f self-assigned this Oct 7, 2024
@atavism
Copy link
Contributor

atavism commented Oct 25, 2024

The changes here look amazing, @jigar-f!!! Sorry I haven't had a chance to take a closer look at this one yet, but I will do that first thing in the morning

@jigar-f
Copy link
Contributor Author

jigar-f commented Oct 25, 2024

@atavism I updated PR description to help to run the test across all devices, do take a look

@jigar-f
Copy link
Contributor Author

jigar-f commented Dec 3, 2024

@atavism This is ready for review/

@jigar-f
Copy link
Contributor Author

jigar-f commented Dec 3, 2024

Also now we are running tets on windows as well.

@jigar-f
Copy link
Contributor Author

jigar-f commented Dec 5, 2024

@atavism Did you get chance to take look at this one?

@jigar-f jigar-f requested a review from atavism December 10, 2024 09:25
@jigar-f
Copy link
Contributor Author

jigar-f commented Dec 17, 2024

@atavism Can you review this? I like this to get merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants