My understanding is that the only reason we're exporting this is for unit-testing.  If that's true, I'd propose that we call both the exported symbol and the implementation _usersToArray() to make it more clear to developers that they shouldn't be messing around with this.  (Same thing applies to findNewNick, but since your patch isn't touching that...).