Unit tests upgrade.

[kkeroo]

This PR refactors creators unit tests and updates the unit tests with the tests for message types. Unittests folder is now structured like:

unittests
-- test_creators
-- test_messages

I cleaned the creators tests and did some changes so they follow the creator functions. Newly added unittests for message types extends our CI testing.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 35.94%. Comparing base (37a2d92) to head (2964afc).
Report is 20 commits behind head on main.

✅ All tests successful. No failed tests found.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #145      +/-   ##
==========================================
+ Coverage   33.40%   35.94%   +2.54%     
==========================================
  Files          68       69       +1     
  Lines        3739     3836      +97     
==========================================
+ Hits         1249     1379     +130     
+ Misses       2490     2457      -33     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[klemen1999]

Generally LGTM, @jkbmrz could you take a look as well

[jkbmrz]

Generally LGTM! Adding some non-blocking comments and suggesting a small stylistic change:
We could increase readability of the tests by defining the correct argument values as variables instead of re-defining them anew for each test function. For example, we could change:

def test_invalid_keypoints_type():
    with pytest.raises(ValueError):
        create_detection_message(
            np.array([[0.5, 0.5, 0.2, 0.2]]), np.array([0.9]), keypoints=[[[0.1, 0.1]]]
        )

to:

BBOXES = np.array([[0.5, 0.5, 0.2, 0.2]])
SCORES = np.array([0.9])

def test_invalid_keypoints_type():
    with pytest.raises(ValueError):
        create_detection_message(
            BBOXES, SCORES, keypoints=[[[0.1, 0.1]]]
        )

and re-use the defined variables throughout the tests.This would increase the readability as the the reader could better focus on the tested values (and ignore the correct ones).

[jkbmrz]

Same here and above, I think we could simplify these tests and have the assert checks in one test function only.

[kkeroo]

@jkbmrz thanks for detailed review. I fixed your suggestions and added global variables so we dont define vars inside each test.