-
-
Notifications
You must be signed in to change notification settings - Fork 0
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.
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
test: transfer aedes tests to the project #7
Conversation
getlarge
commented
Oct 5, 2023
•
edited
Loading
edited
- Fixes [bug] Strange behaviour with retained packets #6
The current approach is not the most elegant as i duplicated aedes tests inside the codebase to have reliably running tests. Albeit by now i could move those changes back to the local aedes node_modules and trigger |
That's so good! 🎉
Awesome, let me know when this is ready 💪🏼 |
Found it. It seems like the |
Yeah in fact as I said there was some really weird behaviour and this explains it. I think that adding the tests should cover us on almost all this kind of issues 💪🏼 |