I suspect flake8 may not like the two empty lines here
This is used only in one place, can't we just mock it where it's used?
I wonder if we want to add support for this to pagure-admin (though this can be done in another PR)
No, I'm mocking it out in all the other tests that run get_user, because about every test runs it.
Note that the default is keep_get_user=False. There's only one place where this does not get mocked.
keep_get_user=False
2 new commits added
Nice, I read it just backward :D
Looks fine to me, thanks! :)
Pull-Request has been merged by pingou