From 4c9301268092f2d93ae3e34ee4ffec04d36b3f40 Mon Sep 17 00:00:00 2001 From: Pierre-Yves Chibon Date: Jun 12 2018 12:16:33 +0000 Subject: Fix a flake8 error and a test for pagure-admin block-user Signed-off-by: Pierre-Yves Chibon --- diff --git a/pagure/cli/admin.py b/pagure/cli/admin.py index f12da50..f112faf 100644 --- a/pagure/cli/admin.py +++ b/pagure/cli/admin.py @@ -795,7 +795,7 @@ def do_block_user(args): user = pagure.lib.get_user(session, args.username) print('The user `%s` will be blocked from all interaction with this ' - 'pagure instance until: %s.', user.username, date.isoformat()) + 'pagure instance until: %s.', user.username, date.isoformat()) if not _ask_confirmation(): return diff --git a/tests/test_pagure_admin.py b/tests/test_pagure_admin.py index d554344..0638cf6 100644 --- a/tests/test_pagure_admin.py +++ b/tests/test_pagure_admin.py @@ -1393,7 +1393,7 @@ class PagureBlockUserTests(tests.Modeltests): pagure.cli.admin.do_block_user(args) user = pagure.lib.get_user(self.session, 'pingou') - self.assertIsNone(user.refuse_sessions_before) + self.assertIsNotNone(user.refuse_sessions_before) if __name__ == '__main__':