search

aabarmin / epam-microservices-training-2022

Apache License 2.0
14 stars 15 forks source link

Increase Test Coverage for monolithic-layered app #11

Closed naXa777 closed 12 months ago

naXa777 commented 1 year ago

The monolithic/monolithic-layered module currently has 0% test coverage. While performing manual testing, I encountered a few bugs. This issue aims to address the lack of automated tests for this module and ensure better reliability.

  1. Write unit tests for individual components:

    • Services
    • Controllers
    • Repositories

  2. Write integration tests to cover end-to-end flows of the app.

Please provide a PR with added tests, any bug fixes, and a brief description of the changes. Address any bugs or discrepancies found during testing. Any found bugs should be documented, and if they are outside the scope of this issue, separate issues should be opened.

Benefits

naXa777 commented 1 year ago

Let me add some tests! Please assign the ticket, @aabarmin.

naXa777 commented 1 year ago

Hey, @aabarmin, so I took the "act now, apologize later" approach and just went ahead and added those tests! I know you are a good person, cause you never said "NO" when I asked you to assign a ticket 😄 Anyway, here's my pull request, ready for your magic touch: https://github.com/aabarmin/epam-microservices-training-2022/pull/30