f1nal3 / Juniorgram

A C++ messenger for learning purposes in EPAM Brest C++ R&D Lab
14 stars 1 forks source link

Write unit-tests on a Test Client #668

Open memorisecodead opened 1 year ago

memorisecodead commented 1 year ago

Workflow:

f1nal3 commented 1 year ago

image I'd refrain from bothering with this since:

memorisecodead commented 1 year ago

image I'd refrain from bothering with this since:

  • It is not used in the statistics
  • Will soon be reworked
  • There is a reason test things are not considered during coverage report

At this point, there are no tests for the current implementation. Regardless of the sonar, we stick to the rule written code - covered by tests. Even if there are changes to the test client in the future, the tests will be rewritten for the new implementation =)