search

vanitasvitae / Smack

A modular and portable open source XMPP client library written in Java for Android and Java (SE) VMs
https://igniterealtime.org/projects/smack/
Apache License 2.0
16 stars 3 forks source link

Integration tests: deviceList is not cleared properly #28

Closed vanitasvitae closed 7 years ago

vanitasvitae commented 7 years ago

deviceList node contains deviceIds of older tests despite the fact all items are deleted as well as the node itself.

Maybe this is caused by delayed PEP updates (race condition)?

Its not fatal, but its ugly to have all those exceptions in the log.

vanitasvitae commented 7 years ago

It WAS cleared properly. The issue was, that OmemoManagers of earlier tests were still existent and had stanzaListeners registered, which added their deviceIds to the list when updates were received.

I added a shutdown() method to the OmemoManager that removes all active listeners of the OmemoManager. Now it works as intended :)