As long as its readable (it is) and easy to see what it does than it should be good. I would say to make it explicit in QualityFixture though, makes it more readable when we need to see what its doing, though if we add a new quality it means we have to add it (but I think I'd be happier with that).