astregs: return only valid mobile number in linking means (#35641)
This commit is contained in:
parent
ddf5cd9f86
commit
79562129c3
|
@ -399,11 +399,14 @@ class AstreGS(BaseResource):
|
|||
'value': r.AdresseMail,
|
||||
'type': 'email'})
|
||||
if r.TelephoneMobile:
|
||||
data.append({'id': 'mobile',
|
||||
'text': 'par SMS vers %s****%s' % (r.TelephoneMobile[:2], r.TelephoneMobile[-3:]),
|
||||
'value': r.TelephoneMobile,
|
||||
'type': 'mobile'})
|
||||
mobile = ''.join((n for n in r.TelephoneMobile if n.isdigit()))
|
||||
if mobile and len(mobile) == 10 and mobile[:2] in ('06', '07'):
|
||||
data.append({'id': 'mobile',
|
||||
'text': 'par SMS vers %s*****%s' % (mobile[:2], mobile[-3:]),
|
||||
'value': mobile,
|
||||
'type': 'mobile'})
|
||||
response['data'] = data
|
||||
response['raw_data'] = serialize_object(r)
|
||||
return response
|
||||
|
||||
@endpoint(description=_('Create link between user and association'),
|
||||
|
|
|
@ -1 +1 @@
|
|||
<?xml version='1.0' encoding='utf-8'?><soapenv:Envelope xmlns:soapenv="http://schemas.xmlsoap.org/soap/envelope/"><soapenv:Body><ns1:chargementResponse xmlns:ns1="http://gfi.astre.webservices/rf/gf/contact"><ns1:response><ns1:ContactReturn><ns1:idContact>13012</ns1:idContact><ns1:CodeContact>AS173957</ns1:CodeContact><ns1:CodeTitreCivilite>035</ns1:CodeTitreCivilite><ns1:Nom>MARTIN</ns1:Nom><ns1:Prenom>Jean-Marc</ns1:Prenom><ns1:NomDeJeuneFille></ns1:NomDeJeuneFille><ns1:DateDeNaissance></ns1:DateDeNaissance><ns1:FormuleCivilite></ns1:FormuleCivilite><ns1:IntituleTitre2></ns1:IntituleTitre2><ns1:IntituleTitre3></ns1:IntituleTitre3><ns1:IntituleTitre4>Président de l&#39;</ns1:IntituleTitre4><ns1:SituationDeFamille></ns1:SituationDeFamille><ns1:CodeFonction></ns1:CodeFonction><ns1:LibelleFonction></ns1:LibelleFonction><ns1:TelephoneBureau></ns1:TelephoneBureau><ns1:TelephoneMobile>0660909980</ns1:TelephoneMobile><ns1:NumeroDeFax></ns1:NumeroDeFax><ns1:AdresseMail>jeanmarcallan@neuf.fr</ns1:AdresseMail><ns1:PageWeb></ns1:PageWeb><ns1:AdresseDestinataire>Jean-Marc MARTIN</ns1:AdresseDestinataire><ns1:AdresseComplementaire></ns1:AdresseComplementaire><ns1:ComplementGeographique></ns1:ComplementGeographique><ns1:RueVoie>271 chemin de Curnier</ns1:RueVoie><ns1:ComplementVoie></ns1:ComplementVoie><ns1:CodePostal>06750</ns1:CodePostal><ns1:Ville>SERANON</ns1:Ville><ns1:CodePays>FR</ns1:CodePays><ns1:LibellePays>France</ns1:LibellePays><ns1:LibelleAdresse>SIEGE SOCIAL</ns1:LibelleAdresse><ns1:Commentaire></ns1:Commentaire></ns1:ContactReturn></ns1:response></ns1:chargementResponse></soapenv:Body></soapenv:Envelope>
|
||||
<?xml version='1.0' encoding='utf-8'?><soapenv:Envelope xmlns:soapenv="http://schemas.xmlsoap.org/soap/envelope/"><soapenv:Body><ns1:chargementResponse xmlns:ns1="http://gfi.astre.webservices/rf/gf/contact"><ns1:response><ns1:ContactReturn><ns1:idContact>13012</ns1:idContact><ns1:CodeContact>AS173957</ns1:CodeContact><ns1:CodeTitreCivilite>035</ns1:CodeTitreCivilite><ns1:Nom>FOO</ns1:Nom><ns1:Prenom>Foo-Marc</ns1:Prenom><ns1:NomDeJeuneFille></ns1:NomDeJeuneFille><ns1:DateDeNaissance></ns1:DateDeNaissance><ns1:FormuleCivilite></ns1:FormuleCivilite><ns1:IntituleTitre2></ns1:IntituleTitre2><ns1:IntituleTitre3></ns1:IntituleTitre3><ns1:IntituleTitre4>Président de l&#39;</ns1:IntituleTitre4><ns1:SituationDeFamille></ns1:SituationDeFamille><ns1:CodeFonction></ns1:CodeFonction><ns1:LibelleFonction></ns1:LibelleFonction><ns1:TelephoneBureau></ns1:TelephoneBureau><ns1:TelephoneMobile>06 67 78 89 90</ns1:TelephoneMobile><ns1:NumeroDeFax></ns1:NumeroDeFax><ns1:AdresseMail>foo@example.com</ns1:AdresseMail><ns1:PageWeb></ns1:PageWeb><ns1:AdresseDestinataire>Foo FOO</ns1:AdresseDestinataire><ns1:AdresseComplementaire></ns1:AdresseComplementaire><ns1:ComplementGeographique></ns1:ComplementGeographique><ns1:RueVoie>271 chemin de Curnier</ns1:RueVoie><ns1:ComplementVoie></ns1:ComplementVoie><ns1:CodePostal>06750</ns1:CodePostal><ns1:Ville>SERANON</ns1:Ville><ns1:CodePays>FR</ns1:CodePays><ns1:LibellePays>France</ns1:LibellePays><ns1:LibelleAdresse>SIEGE SOCIAL</ns1:LibelleAdresse><ns1:Commentaire></ns1:Commentaire></ns1:ContactReturn></ns1:response></ns1:chargementResponse></soapenv:Body></soapenv:Envelope>
|
||||
|
|
|
@ -128,19 +128,37 @@ def test_check_association_presence(mocked_post, mocked_get, connector, app):
|
|||
@mock.patch('passerelle.utils.Request.post', side_effect=contact_search_side_effect)
|
||||
def test_association_linking_means(mocked_post, mocked_get, client, connector, app):
|
||||
resp = app.get('/astregs/test/get-association-link-means', params={'association_id': '42'})
|
||||
assert len(resp.json['data']) == 2
|
||||
assert resp.json['already_paired'] == False
|
||||
for result in resp.json['data']:
|
||||
assert 'id' in result
|
||||
assert 'text' in result
|
||||
assert 'type' in result
|
||||
assert 'value' in result
|
||||
assert resp.json['data'] == [
|
||||
{'id': 'email', 'text': 'par courriel vers fo***@***com',
|
||||
'value': 'foo@example.com', 'type': 'email'},
|
||||
{'id': 'mobile', 'text': 'par SMS vers 06*****990',
|
||||
'value': '0667788990', 'type': 'mobile'}
|
||||
]
|
||||
assert resp.json['raw_data']['AdresseMail'] == 'foo@example.com'
|
||||
assert resp.json['raw_data']['TelephoneMobile'] == '06 67 78 89 90'
|
||||
|
||||
Link.objects.create(name_id='user_name_id', association_id='42', resource=connector)
|
||||
resp = app.get('/astregs/test/get-association-link-means',
|
||||
params={'association_id': '42', 'NameID': 'user_name_id'})
|
||||
assert resp.json['already_paired'] == True
|
||||
|
||||
for bad_mobile_number in ('', '01 43 35 01 35', '00 33 7 01 02 03 04', 'letters', '06 01 02'):
|
||||
mocked_post.side_effect = [
|
||||
mock.Mock(content=get_xml_file('Tiers.xml'), status_code=200,
|
||||
headers={'Content-Type': 'text/xml'}),
|
||||
mock.Mock(content=get_xml_file('Contact.xml').replace(
|
||||
'<ns1:TelephoneMobile>06 67 78 89 90<',
|
||||
'<ns1:TelephoneMobile>%s<' % bad_mobile_number),
|
||||
status_code=200, headers={'Content-Type': 'text/xml'})
|
||||
]
|
||||
resp = app.get('/astregs/test/get-association-link-means', params={'association_id': '42'})
|
||||
assert resp.json['data'] == [
|
||||
{'id': 'email', 'text': 'par courriel vers fo***@***com',
|
||||
'value': 'foo@example.com', 'type': 'email'}
|
||||
]
|
||||
assert resp.json['raw_data']['AdresseMail'] == 'foo@example.com'
|
||||
assert resp.json['raw_data']['TelephoneMobile'] == (bad_mobile_number or None)
|
||||
|
||||
|
||||
@mock.patch('passerelle.utils.Request.get', side_effect=search_wsdl_side_effect)
|
||||
|
|
Loading…
Reference in New Issue