packets1: Add full-coverage unit tests for `{R,S,U}*` packets

energostack / bisquitt

Bisquitt, a transparent MQTT-SN gateway

Eclipse Public License 2.0

19 stars 3 forks source link

packets1: Add full-coverage unit tests for `{R,S,U}*` packets #51

Closed mprymek closed 1 year ago

mprymek commented 1 year ago

The first commit improves the way {S,Uns}ubscribe packets are stringified by String(). The other commits make packets1.{R,S,U}* code fully covered by unit tests.

dmajda commented 1 year ago

In addition, I just noticed a small mistake: The message of first commit contains stray “sub2” at the very end.

mprymek commented 1 year ago

In addition, I just noticed a small mistake: The message of first commit contains stray “sub2” at the very end.

Oh, I'm sorry, overlooked when squashing commits... Fixed.