lime-packages icon indicating copy to clipboard operation
lime-packages copied to clipboard

lime-utils: get_node_status, Rewrite the functionality in order to be easier to test.

Open germanferrero opened this issue 3 years ago • 1 comments

A recent regression affected this functionality. And it was fixed by https://github.com/libremesh/lime-packages/pull/757 Implementing a unit test for it to avoid future regressions is desired, but not straight forward since currently the function has multiple responsibilities. One possible approach would be to split it in two:

  • most_active neighbor
  • the rest of the "status" And do unit tests for each of it.

germanferrero avatar Sep 16 '20 22:09 germanferrero

@selankon what do you think about this issue? Has it alredy been fixed?

ilario avatar Feb 28 '23 14:02 ilario