I wanted to point out one change in the patch that isn't obvious. I modified org.apache.vysper.xmpp.modules.core.TestUser.createForSession(). Previously it was returning a new TestUser with a stored resource, but the entity that it stored did not have a bound resource. This would cause problems in testing when using the new ResourceRegistry API, since the new API requires Entities with bound resources instead of resource name Strings as parameters.
The change required updates to some other tests as well, but everything seems to be working now.