While we're at it, I don't see any reason why this shouldn't become:
```.py
for role in roles:
    rn = role.name if isinstance(role, self.role_model) else role
    roles.append(self.find_role(rn)
```