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

Utils test #404

Merged
merged 11 commits into from
Dec 27, 2024
Merged

Utils test #404

merged 11 commits into from
Dec 27, 2024

Conversation

rohansen856
Copy link
Contributor

Description

Please include a summary of the change and which issue is fixed. List any dependencies that are required for this change.

Fixes #388

Replace issue_no with the issue number which is fixed in this PR

Screenshots

Summary

  • this PR adds a total of 187 tests for all files under lib/app/utils directory. all are observed to be passing.
  • skipped: a total of 2 folders were skipped lib/app/utils/gen and lib/app/utils/home_path. the first one is auto generated while building and the second one contains device specific configurations (same ap the previous PR)
  • Note: there may be a log: Error requesting permissions: type 'Future<PermissionStatus>' is not a subtype of type 'int' in type cast, (cause is autogenerated file: test/utils/permissions/permissions_manager_test.mocks.dart) this is not an error as all the tests pass around it.
  • Test output:
00:03 +0: /home/rcsen/Documents/open-source/taskwarrior_test/test/utils/permissions/permissions_manager_test.dart: PermissionsManager requestAllPermissions handles permission denied                
Error requesting permissions: type 'Future<PermissionStatus>' is not a subtype of type 'int' in type cast
00:24 +187: All tests passed!  

Checklist

  • Tests have been added or updated to cover the changes
  • Documentation has been updated to reflect the changes
  • Code follows the established coding style guidelines
  • All tests are passing

@rohansen856
Copy link
Contributor Author

@BrawlerXull @Pavel401 any reviews on this...? if this is ok, i would start writing tests for other folders too.

@Pavel401 Pavel401 merged commit 2a91fe1 into CCExtractor:main Dec 27, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

tests: add unit tests for all files
2 participants