Yeah I added a bunch of superfluous comments so it was easy to get the flow of the tests as a 3rd party reader. Good or bad idea?

Could be horrible but I thought it helped when comparing it to the FindFile test, for example, as in that scenario we don't inspect the resulting BSONObj again and look at the retrieved file instead.