Just one comment: test_listing_seen_with_permission needs a comment at the start of it stating the expected behaviour (it's obvious from the test, but I don't want to have to parse the test to understand what it's testing).
Other than that this is great; r=me.
« Back to merge proposal
Just one comment: test_listing_ seen_with_ permission needs a comment at the start of it stating the expected behaviour (it's obvious from the test, but I don't want to have to parse the test to understand what it's testing).
Other than that this is great; r=me.