olvrwn / CashBud

0 stars 0 forks source link

Add unit tests for TransactionMapper #86

Closed sweep-ai[bot] closed 1 year ago

sweep-ai[bot] commented 1 year ago

PR Feedback (click)

Summary of Changes

Please review and merge this PR to ensure the TransactionMapper is thoroughly tested.

Fixes #82.


šŸŽ‰ Latest improvements to Sweep:


šŸ’” To get Sweep to edit this pull request, you can:

sonarcloud[bot] commented 1 year ago

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

olvrwn commented 1 year ago

Add unit tests to test every function and each case of the TransactionMapper.

sweep-ai[bot] commented 1 year ago

šŸš€ Wrote Changes

Add unit tests to test every function and each case of the TransactionMapper.

Hi @olvrwn,

I decided to make the following changes:

File Path Proposed Changes
CashBudTests/Tests/Mappers/TransactionMapperTest.swift Modify CashBudTests/Tests/Mappers/TransactionMapperTest.swift with contents:
ā€¢ Add unit tests to test every function and each case of the TransactionMapper.
olvrwn commented 1 year ago

write unit tests

sweep-ai[bot] commented 1 year ago

šŸš€ Wrote Changes

write unit tests

Hi @olvrwn,

I decided to make the following changes:

File Path Proposed Changes
CashBudTests/Tests/Mappers/TransactionMapperTest.swift Modify CashBudTests/Tests/Mappers/TransactionMapperTest.swift with contents:
ā€¢ Add a new unit test case class for the TransactionMapper.
ā€¢ Add test methods to validate the functionality of the TransactionMapper.
ā€¢ Modify the import statement to include the necessary files and dependencies for testing the TransactionMapper.