I was just looking at this test. Changes made to this test does not make sense to me. result of #cached_user_roles are cached records, you should probably call ```.map(&:role)```. Also the second assertion checks for array to be included among roles. Here's the test snippet that I think does what we want

```ruby
user_roles = cached_user_roles.map(&:role)
user.roles.each do |role|
  assert_include user_roles, role
end
```