Projet

Général

Profil

0001-grenoble_gru-do-not-send-empty-dem_pav-if-empty-4804.patch

Serghei Mihai, 27 octobre 2020 09:51

Télécharger (1,89 ko)

Voir les différences: 

Subject: [PATCH] grenoble_gru: do not send empty dem_pav if empty (#48046)


 passerelle/contrib/grenoble_gru/models.py |  2 +-
 tests/test_grenoble_gru.py                | 12 ++++++++++++
 2 files changed, 13 insertions(+), 1 deletion(-)
passerelle/contrib/grenoble_gru/models.py
133 133 
        if 'applicant_free_address' in data:
134 134 
            payload['dem_adresse_manuelle'] = data['applicant_free_address']
135 135 

  		




  136
  
  
    
        if 'dem_pav' in data:

  		




  
  136
  
    
        if 'dem_pav' in data and data['dem_pav']:

  		




  137
  137
  
    
            payload['dem_pav'] = data['dem_pav']

  		




  138
  138
  
    

  		




  139
  139
  
    
        return payload

  		












  

    
      tests/test_grenoble_gru.py
    
  





  269
  269
  
    
        assert request_post.call_args[1]['data']['obs_demande_urgente'] == 1

  		




  270
  270
  
    
        assert request_post.call_args[1]['data']['dem_reponse'] == 1

  		




  271
  271
  
    

  		




  
  272
  
    
        # make sure empty dem_pav value is not sent

  		




  
  273
  
    
        payload['dem_pav'] = ''

  		




  
  274
  
    
        app.post_json(

  		




  
  275
  
    
            reverse('generic-endpoint',

  		




  
  276
  
    
                    kwargs={

  		




  
  277
  
    
                        'connector': 'grenoble-gru',

  		




  
  278
  
    
                        'slug': setup.slug,

  		




  
  279
  
    
                        'endpoint': 'create-demand'

  		




  
  280
  
    
                    }),

  		




  
  281
  
    
            params=payload)

  		




  
  282
  
    
        assert 'dem_pav' not in request_post.call_args[1]['data']

  		




  
  283
  
    

  		




  272
  284
  
    

  		




  273
  285
  
    
def test_add_attachment_to_demand(app, setup):

  		




  274
  286
  
    
    with mock.patch('passerelle.utils.Request.post') as request_post:

  		




  275
  
  
    
-