From 4b87df88ee59bb2ec9e2d1cd8bd9ecc351697cd8 Mon Sep 17 00:00:00 2001 From: Quentin Defenouillere Date: Mon, 4 Feb 2019 14:20:18 +0100 Subject: [PATCH] [cragr] Remove assert False since all error message are handled The assert False at the end of the security check will force a crash if we manage to login after the second try. Since all error messages are now handled, this assert False is now obsolete so I removed it. Closes: 35412@sibi --- modules/cragr/api/browser.py | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/modules/cragr/api/browser.py b/modules/cragr/api/browser.py index 9039ccf666..d636535889 100644 --- a/modules/cragr/api/browser.py +++ b/modules/cragr/api/browser.py @@ -159,9 +159,8 @@ def do_login(self): message = error.get('message', '') if 'Un incident technique' in message: raise BrowserUnavailable(message) - assert False, 'Unhandled Server Error encountered: %s' % error.get('message', '') - # accounts_url may contain '/particulier', '/professionnel' or '/association' + # accounts_url may contain '/particulier', '/professionnel', '/entreprise' or '/association' self.accounts_url = self.page.get_accounts_url() assert self.accounts_url, 'Could not get accounts url from security check' self.location(self.accounts_url) -- GitLab