search

wind-c / comqtt

A lightweight, high-performance go mqtt server(v3.0|v3.1.1|v5.0) supporting distributed cluster
MIT License
869 stars 50 forks source link

TestServerBuildAck fails #54

Closed perbu closed 11 months ago

perbu commented 11 months ago

It looks like the test is broken. It seems to expect the ack to contain some very specific data. The test will pass when replacing properties with:

    properties := packets.Properties{}

which I suspect is correct, but I'm not 100% sure.

https://github.com/wind-c/comqtt/blob/9c91ef6b86e11a7ca5d947dfcf6c0dd8319b86d4/mqtt/server_test.go#L1212