Projet

Général

Profil

0001-lingo-fix-url-signing-for-new-remote-invoice-retriev.patch

Serghei Mihai, 22 mars 2018 18:22

Télécharger (1,98 ko)

Voir les différences: 

Subject: [PATCH] lingo: fix url signing for new remote invoice retrieval
 (#22750)


 combo/apps/lingo/models.py | 2 +-
 tests/test_notification.py | 4 +++-
 2 files changed, 4 insertions(+), 2 deletions(-)
combo/apps/lingo/models.py
214 214 
            return {}
215 215 
        url = self.webservice_url + '/users/with-pending-invoices/'
216 216 
        response = requests.get(url, remote_service='auto', cache_duration=0,
217 
                                log_errors=False)
217 
                                log_errors=False, without_user=True)
218 218 
        if not response.ok:
219 219 
            return {}
220 220 
        return response.json()['data']
tests/test_notification.py
275 275 
    assert result['err'] == 0
276 276 

  		




  277
  277
  
    

  		




  278
  
  
    
@mock.patch('combo.apps.lingo.models.requests.get')

  		




  
  278
  
    
@mock.patch('combo.utils.requests_wrapper.RequestsSession.request')

  		




  279
  279
  
    
def test_notify_remote_items(mock_get, app, user, user2, regie):

  		




  280
  280
  
    

  		




  281
  281
  
    
    datetime_format = '%Y-%m-%dT%H:%M:%S'

  		




  ......




  345
  345
  
    
    regie.webservice_url = 'http://example.org/regie'  # is_remote

  		




  346
  346
  
    
    regie.save()

  		




  347
  347
  
    
    regie.notify_new_remote_invoices()

  		




  
  348
  
    
    assert 'NameID=' not in mock_get.call_args[0][1]

  		




  
  349
  
    
    assert 'email=' not in mock_get.call_args[0][1]

  		




  348
  350
  
    

  		




  349
  351
  
    
    assert Notification.objects.filter(external_id__startswith='invoice-%s:' % regie.slug).visible().new().count() == 2

  		




  350
  352
  
    
    assert Notification.objects.filter(external_id__startswith='invoice-%s:reminder-' % regie.slug).count() == 0

  		




  351
  
  
    
-