#3188 Allow forcing logout of all sessions before now
Merged by pingou. Opened by puiterwijk.
puiterwijk/pagure refusesessions  into  master

Download 3188.patch
no initial comment

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.

2 new commits added

  • Allow forcing logout of all sessions before now
  • Mock get_user out

Nice, I read it just backward :D

Looks fine to me, thanks! :)

Pull-Request has been merged by pingou

Metadata