Crdzbird/unit test flutter #259
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Widget Test Unit Test Integrity Test
|
Awesome to have some tests! |
oxtoacart
reviewed
Jul 29, 2021
| @@ -84,6 +84,43 @@ Once the required tools are installed and the lfs are downloaded, open the proje | |||
| - Edit the run config and add on the flavor option prod | |||
|
|
|||
|
|
|||
| ## Unit Tests | |||
|
|
|||
| To run the unit test you need to input the following setup. | |||
There was a problem hiding this comment.
Is there a way to do this from the command line?
There was a problem hiding this comment.
yep. an example would be:
flutter run test/catcher_test.dart --coverage or
flutter run test/catcher_test.dart
There was a problem hiding this comment.
if you wish to run everything just replace the catcher_test with an * and it should run everything
There was a problem hiding this comment.
Thanks! Could you add that to the readme please?
oxtoacart
reviewed
Jul 29, 2021
README.md
Outdated
|
|
||
| 1. On your `terminal` check if you have installed: `lcov` if not then install. | ||
| 2. Go to on your terminal `android-lantern/coverage` and type: `genhtml coverage/lcov.info -o coverage/html` that will generate a nice html file with the code coverage of all your files. | ||
| #### NOTE: Please avoid to upload the generated files from coverage, this is because the numerous files that are created. |
There was a problem hiding this comment.
Can you please add the necessary directories/files to .gitignore?
There was a problem hiding this comment.
Sure, im gonna omit the coverage and genhtml
oxtoacart
reviewed
Jul 29, 2021
coverage/lcov.info
Outdated
| @@ -0,0 +1,4857 @@ | |||
| SF:lib/config/catcher_setup.dart | |||
There was a problem hiding this comment.
actually... that is also needed to be omitted, im gonna add it to the .gitignore as suggested
|
I can't run the test from the command line |
|
|
I notice that I forget to update one test |
Crdzbird
commented
Jul 30, 2021
| @@ -32,6 +32,10 @@ app.iml | |||
| # is commented out by default. | |||
| .vscode/ | |||
|
|
|||
| coverage | |||
| coverage/html/ | |||
| coverage/lcov.info | |||
There was a problem hiding this comment.
Added coverage folder to .gitignore
Crdzbird
commented
Jul 30, 2021
| ### Run Independent Tests | ||
|
|
||
| On your terminal go to the root of the project and type: | ||
| `flutter test test/my_folder_test.dart` |
There was a problem hiding this comment.
Example of how to run flutter unit test without coverage
Crdzbird
commented
Jul 30, 2021
| `flutter test test/my_folder_test.dart` | ||
|
|
||
| in case that you need the code coverage just add the following argument. | ||
| `flutter test --coverage test/my_folder_test.dart` |
There was a problem hiding this comment.
Example of how to run flutter unit test with coverage
Crdzbird
commented
Jul 30, 2021
| If you wanna visualize the current percentage of code coverage you need to do the following steps. | ||
|
|
||
| 1. On your `terminal` check if you have installed: `lcov` if not then install. | ||
| 2. Go to on your terminal `android-lantern/coverage` and type: `genhtml coverage/lcov.info -o coverage/html` that will generate a nice html file with the code coverage of all your files. |
There was a problem hiding this comment.
Explanation about how to generate a informative graph about the percentage of code coverage on the project.
Crdzbird
commented
Jul 30, 2021
| @@ -109,6 +109,7 @@ dev_dependencies: | |||
| sdk: flutter | |||
|
|
|||
| test: ^1.15.7 | |||
| mockito: ^5.0.12 | |||
There was a problem hiding this comment.
Mockit is used in order to be capable to emulated some specific code ex: auto_route
Thanks! If I run all tests, the catcher_test fails. I don't think that's new to this branch, but since you're in here, would you mind taking a look please? |
Interesting it seems that the error is not actually on the catcher but rather on base_screen, will take a look on that |
Crdzbird
commented
Jul 30, 2021
| }), | ||
| )); | ||
| await tester.pumpAndSettle(); | ||
| expect(find.byType(HomePage), findsOneWidget); |
There was a problem hiding this comment.
The error was due to the widget still wasn't settle and also the multi provider wasn't inside the scope
|
@oxtoacart All tests should be working, also for any navigation test is preferable to use the |
|
Awesome @Crdzbird ! I love the documentation. |
# for free
to join this conversation on GitHub.
Already have an account?
# to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The test at the time has a tiny percentage of code coverage, hope that this will increase at their respective moment, also added documentation about how to run and generate the visual graph.