I think it's not. We should throw NonexistentConversationException before we associate the conversation which was ended. I've updated the pull request.