Closed sweep-ai[bot] closed 1 year ago
The variables should not be parameters of the functions but instead a class variable. The functions should conform to the TransactionsManagerProtocol.
The variables should not be parameters of the functions but instead a class variable. The functions should conform to the TransactionsManagerProtocol.
Hi @olvrwn,
I decided to make the following changes:
File Path | Proposed Changes |
---|---|
CashBud/TransactionsManagerMock.swift |
Modify CashBud/TransactionsManagerMock.swift with contents: • Remove the parameters from the function signatures and define the necessary variables as class variables. • Add the necessary function signatures from the TransactionsManagerProtocol. |
Kudos, SonarCloud Quality Gate passed!
0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells
No Coverage information
0.0% Duplication
Description
This PR updates the TransactionsManagerMock file in the CashBud project. The file now includes all necessary imports and implements the TransactionsManagerMock class that conforms to the TransactionsManagerProtocol.
For functions that can throw or return an error, a result variable has been added to hold either the successful return object or the error to throw or return. For other functions, a response variable has been implemented to hold the type the function returns.
All functions have been implemented to either return successfully if the variable for the function was set accordingly or to throw or return the set error.
Summary of Changes
Fixes #34.
🎉 Latest improvements to Sweep:
💡 To get Sweep to edit this pull request, you can: