search

trebol-ecommerce / trebol-backend-monolith

Monolithic eCommerce backend web application that exposes a RESTful API.
MIT License
16 stars 21 forks source link

Write unit tests for `org.trebol.operation.services` #88

Closed bglamadrid closed 1 year ago

bglamadrid commented 2 years ago

As the subject says, it is important to have the basics covered for all service classes within the package

org.trebol.operation.services

This issue may be blocked by future refactorings, nonetheless it will remain here as long as it doesn't have at least 60% code coverage.

mepox commented 2 years ago

@bglamadrid Can we split this up? Can I make separate issues for each class/test?

bglamadrid commented 2 years ago

Can we split this up? Can I make separate issues for each class/test?

Yes, you can take one of these services and work on it. Does any one pick your interest right now?

mepox commented 2 years ago

Can we split this up? Can I make separate issues for each class/test?

Yes, you can take one of these services and work on it. Does any one pick your interest right now?

I will have a look and pick one tomorrow.

NyorJa commented 2 years ago

Im gonna take also @bglamadrid

mepox commented 2 years ago
bglamadrid commented 2 years ago

Im gonna take also @bglamadrid

Sure, @mepox just created the proper issues (thank you again!) and I wrote a little description for each of them with the intention and what must the unit tests do in each case.

@mepox also please pick, and request me to assign you, one of those.