1. Please follow the pattern of a test class to a production class. Yeah, the Role class is big, I know, it will be refactored, but breaking only tests into separate classes makes the overall code harder to refactor in the future;

I disagree, here is a lot of costum patching in set_up and tear down. which i a good thing. I would have much of copypasting in the tests, which is allways bad. I'll change the other ones, but I disagree with having one test class for sake of having one test class.