UbiquityRobotics / move_basic

A minimal navigation node
BSD 3-Clause "New" or "Revised" License
69 stars 21 forks source link

Queue behaviour #58

Closed dorkamotorka closed 4 years ago

dorkamotorka commented 4 years ago

goalCancelling test fixture is not working properly yet.

dorkamotorka commented 4 years ago

Yes, I will work on it.

dorkamotorka commented 4 years ago

So despite removing the sleep functions and adding Helper function, unit testing succed in 7/10 cases. Before moving on to ASSERT-ing variables I want to make sure where am I missing things, that the test is so unreliable.

dorkamotorka commented 4 years ago

The test fixtures are working consistently but lagging a lot due to sleep() that cancelGoal and everything else required to work properly. Tests are time independent, sleep are there just so the threads have enough time to close.