Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Testing #5

Open
alsm opened this issue Nov 2, 2018 · 1 comment
Open

Testing #5

alsm opened this issue Nov 2, 2018 · 1 comment

Comments

@alsm
Copy link
Contributor

alsm commented Nov 2, 2018

The code of the library is undertested, both for correctness and performance. A v5 broker implementation for testing is here https://github.com/eclipse/paho.mqtt.testing/ and instance of which is running and available at paho8181.cloudapp.net:1883 (this server supports v5, v3 and websockets for both on the same port)
alsm added a commit to alsm/paho.golang that referenced this issue Jan 30, 2019
@alsm
Start to add tests and fix issues they're throwing up
a85a59a 
Adding tests are overdue and it's showing up the areas that I haven't
got right, especially in the packets library

eclipse-paho#5
bkneis pushed a commit to bkneis/paho.golang that referenced this issue Aug 9, 2022
@alsm
Start to add tests and fix issues they're throwing up
89b49ec 
Adding tests are overdue and it's showing up the areas that I haven't
got right, especially in the packets library

eclipse-paho#5
@MattBrittan
Copy link
Contributor

Added a range of AutoPaho tests in #156 and #158; these exercise a range of functions in the core paho library too. These use a new test server (autopaho/internal/testserver) which enables connection/disconnection logic to be exercised. This may not be a perfect approach but it avoids the need to run external tools.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@alsm @MattBrittan