In our tests, we are currently mocking sys.exit. Due to this mocking, the application logic does not exit immediately in the tests, resulting in many errors failing. Therefore, this issue is being addressed through this PR by removing the mock and checking for the SystemExit exception to be raised.
Related Issue
N/A
Type of Change
Please mark the appropriate option below to describe the type of change your pull request introduces:
[ ] Bug fix
[ ] New feature
[ ] Enhancement
[ ] Documentation update
[x] Refactor
[x] Other (Tests)
Checklist
[x] My pull request has a clear title and description.
[x] I have used semantic commit messages.
Examples: "fix: Fixed foobar bug", "feat(accounts): Added foobar feature".
[ ] I have added/updated the necessary documentation on README.md.
[ ] I have added appropriate test cases (if applicable) to ensure the changes are functioning correctly.
Additional Notes
[Add any additional notes or context that you think the reviewers should know about.]
By submitting this pull request, I confirm that I have read and complied with the contribution guidelines of this project.
Description
In our tests, we are currently mocking
sys.exit
. Due to this mocking, the application logic does not exit immediately in the tests, resulting in many errors failing. Therefore, this issue is being addressed through this PR by removing the mock and checking for theSystemExit
exception to be raised.Related Issue
N/A
Type of Change
Please mark the appropriate option below to describe the type of change your pull request introduces:
Checklist
"fix: Fixed foobar bug"
,"feat(accounts): Added foobar feature"
.README.md
.Additional Notes
[Add any additional notes or context that you think the reviewers should know about.]
By submitting this pull request, I confirm that I have read and complied with the contribution guidelines of this project.