From 19cb27e73976c5e39a74b32842dd4f02fafc619a Mon Sep 17 00:00:00 2001 From: Serghei Mihai Date: Mon, 15 Jan 2018 14:08:21 +0100 Subject: [PATCH] sms: fix mobile number retrieval from mellon session (#22059) --- corbo/views.py | 2 +- tests/test_manager.py | 10 ++++++++++ 2 files changed, 11 insertions(+), 1 deletion(-) diff --git a/corbo/views.py b/corbo/views.py index 2182835..3f15171 100644 --- a/corbo/views.py +++ b/corbo/views.py @@ -273,7 +273,7 @@ view_announce = AnnounceView.as_view() class SendAnnounceView(FormView): def get_initial(self): return {'email': self.request.user.email, - 'mobile': self.request.session.get('mellon_session', {}).get('mobile', '')} + 'mobile': self.request.session.get('mellon_session', {}).get('mobile', [''])[0]} def get_success_url(self, *args, **kwargs): return reverse('view_announce', kwargs={'pk': self.kwargs['pk']}) diff --git a/tests/test_manager.py b/tests/test_manager.py index 524997f..85a2996 100644 --- a/tests/test_manager.py +++ b/tests/test_manager.py @@ -288,6 +288,16 @@ def test_sms_announce(mocked_post, app, admin_user, settings): resp = send_form.submit() assert resp.status_int == 200 + # add mellon attribute to web session + session = app.session + session['mellon_session'] = {'mobile': ['00000000']} + session.save() + app.set_cookie(settings.SESSION_COOKIE_NAME, session.session_key) + resp = resp.click('First announce') + resp = resp.click('Send test SMS') + send_form = resp.forms[0] + assert send_form.fields['mobile'][0].value == '00000000' + form = resp.forms[0] form['mobile'] = '0607080900' # simulate response from passerelle