Skip to content

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