I had not caught this before and this is really nitpicking, but I don't like those names `self.get`, `self.post` and `self.delete`. I would have preferred to a) not pre-create them b) give them a name that indicates it's a `RequestFactory` - when reading this test I thought `self.get` would be a method.