Don't miscapitalize members.

agronholm / mqttproto

Sans-IO MQTT v5 protocol implementation for Python

MIT License

10 stars 3 forks source link

Don't miscapitalize members. #17

Closed smurfix closed 2 weeks ago

smurfix commented 2 weeks ago

… and I was wondering why the tests fail.

coveralls commented 2 weeks ago

Pull Request Test Coverage Report for Build 11742448360

Details

0 of 0 changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 82.816%

Totals
Change from base Build 11689559730:	0.0%
Covered Lines:	6699
Relevant Lines:	8089

💛 - Coveralls

agronholm commented 2 weeks ago

Did you add more tests on your own? The test suite was passing here on GitHub an for me locally.

smurfix commented 2 weeks ago

I tested against mosquitto and flashmq. Both happen to check this field. They kill the connection when that happens, with rather interesting (if unexpected) results on the mqttproto side.