Hello friends! I wanted to contribute to this project, so I did the missing tests for Payment Services. This will improve the code coverage percentage and bring more quality to the project.
📄 Motivation and Context
Motivation:
Currently default test data for payments is being missed. This hinders the development flow.
Context:
Added a file called PaymentSDJpaServiceTest.java based on existing PaymentSDJpaService.java with all tests. Changed - TestUtils.java file at the end with some helper code.
Note: I wanted to implement an efficient UnitTest avoiding an IT, so I had to confirm the method's logic. I capture the parameters in the repository method in order for them to be verified.
Thank you so much for the opportunity to collaborate!
Issue link (https://github.com/jmprathab/MyHome/issues/255)
🧪 How Has This Been Tested?
Basically had to log some parameters after writing the code, and go by test-driven-development. Assertions got me secured that everything was fine.
📷 Screenshots (if appropriate)
📦 Types of changes
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
✅ Checklist
[x] My code follows the code style of this project(Do your best to follow code styles. If none apply just skip this).
[ ] My change requires a change to the documentation.
🚀 Description
Hello friends! I wanted to contribute to this project, so I did the missing tests for Payment Services. This will improve the code coverage percentage and bring more quality to the project.
📄 Motivation and Context
Motivation:
Context:
🧪 How Has This Been Tested?
Basically had to log some parameters after writing the code, and go by test-driven-development. Assertions got me secured that everything was fine.
📷 Screenshots (if appropriate)
📦 Types of changes
✅ Checklist