0001-ovh-retry-on-too-many-requests-error-44730.patch

Nicolas Roche, 02 septembre 2021 16:14

Voir les différences: en ligne côte à côte

Subject: [PATCH] ovh: retry on too many requests error (#44730)

 passerelle/apps/ovh/models.py |  3 +++
 tests/test_sms.py             | 35 +++++++++++++++++++++++++++++++++++
 2 files changed, 38 insertions(+)

     from django.contrib.postgres.fields import ArrayField
     from django.core.mail import send_mail
     from django.db import models
     from django.template.loader import render_to_string
     from django.utils import timezone
     from django.utils.encoding import force_text
     from django.utils.translation import ugettext_lazy as _
     from passerelle.base.models import SkipJob
     from passerelle.sms.models import SMSResource
     from passerelle.utils.jsonresponse import APIError
     class OVHSMSGateway(SMSResource):
         documentation_url = (
             'https://doc-publik.entrouvert.com/admin-fonctionnel/les-tutos/configuration-envoi-sms/'
+        )
-...
                                 self.slug,
                                 credit_left,
                                 self.credit_threshold_alert,
+                            )
                         ret['credit_left'] = credit_left
                         ret['ovh_result'] = result
                         ret['sms_ids'] = result.get('SmsIds', [])
                         return ret
                     elif result['status'] == 429:
                         raise SkipJob(after_timestamp=3)
                     else:
                         raise APIError('OVH error: %r' % result)

                 assert test_vector['result']['err_desc'] in job.status_details['error_summary']
                 nb_failed += 1
             else:
                 assert job.status == 'completed'
         assert Job.objects.filter(method_name='send_job').count() == total
         assert SMSLog.objects.count() == total - nb_failed
     @pytest.mark.parametrize('connector', [OVHSMSGateway], indirect=True)
     def test_sms_legacy_retry_after_error(app, connector, freezer):
         path = '/%s/%s/send/' % (connector.get_connector_slug(), connector.slug)
         assert not SMSLog.objects.filter(appname=connector.get_connector_slug(), slug=connector.slug).exists()
         payload = {
             'message': 'plop',
             'from': '+33699999999',
             'to': ['+33688888888'],
+        }
         freezer.move_to('2019-01-01 00:00:00')
         result = app.post_json(path, params=payload)
         assert result.json['err'] == 0
         assert Job.objects.exists()
         assert Job.objects.get().status == 'registered'
         assert not Job.objects.get().after_timestamp
         with utils.mock_url(
             url=connector.URL,
             response={'status': 429, 'message': 'Too much requests.\nPlease retry in 3 seconds.'},
         ):
             connector.jobs()
         assert Job.objects.get().status == 'registered'
         assert Job.objects.get().after_timestamp
         with utils.mock_url(url=connector.URL, response={'status': 100, 'credit_left': 22}):
             connector.jobs()
         assert Job.objects.get().status == 'registered'
         freezer.move_to('2019-01-01 00:00:04')
         with utils.mock_url(url=connector.URL, response={'status': 100, 'creditLeft': 22}):
             connector.jobs()
         assert Job.objects.get().status == 'completed'
         assert SMSLog.objects.exists()
     def test_manage_views(admin_user, app, connector):
         url = '/%s/%s/' % (connector.get_connector_slug(), connector.slug)
         resp = app.get(url)
         assert 'Endpoints' in resp.text
         assert not 'accessright/add' in resp.text
         app = login(app)
         resp = app.get(url)
         description_fields = [
+    -

Projet

Général

Profil

Produits Entr'ouvert » Passerelle

0001-ovh-retry-on-too-many-requests-error-44730.patch