depthai-ros icon indicating copy to clipboard operation
depthai-ros copied to clipboard

[Feature-Request] {Add unit tests}

Open Serafadam opened this issue 1 year ago • 1 comments

Start with the why:

Well, it's a general good practice.

Move to the what:

  • depthai_bridge - Adding tests to verify Converters properly output messages based on some mock input DAI data.

  • depthai_ros_driver - Adding several test cases to verify bare C++ code for PipelineGenerator to verify that parameters/objects are properly created.

Move to the how:

  • depthai_bridge: For each converter add a test case with some example data. Verify that different options work.
  • depthai_ros_driver: Test out pipeline generation based on different devices (mock device sensor list).

I guess best way to start is to start with one test case for each of those packages and add more later. cc @saching13

Serafadam avatar Jun 30 '23 07:06 Serafadam

Yes. We can start with unit for each of the converters. Followed by integration tests where we run it on our devices with rpi, Jetson platforms.

saching13 avatar Jul 04 '23 12:07 saching13