Replace ElementsAPICall model with a celery task to patch update an E… (

#173)

Replace ElementsAPICall model with a celery task to patch update an Elements record. Delete elements models and make database migration. Update tests.

Pipfile

@@ -9,6 +9,7 @@ python_version = "3.7"
 [dev-packages]
 coverage = ">=4.5.1"
 freezegun = "*"
+requests-mock = "*"
 
 [packages]
 # Only needed for Heroku
@@ -32,3 +33,5 @@ newrelic = "*"
 whitenoise = "*"
 Pillow = "*"
 sentry-sdk = "*"
+celery = "*"
+redis = "*"

solenoid/__init__.py

@@ -0,0 +1,3 @@
+from .celery import app as celery_app
+
+__all__ = ('celery_app',)

solenoid/celery.py

@@ -0,0 +1,14 @@
+import os
+
+from celery import Celery
+
+os.environ.setdefault('DJANGO_SETTINGS_MODULE', 'solenoid.settings.base')
+
+app = Celery('solenoid')
+app.config_from_object('django.conf:settings', namespace='CELERY')
+app.autodiscover_tasks()
+
+
+@app.task(bind=True)
+def debug_task(self):
+    print('Request: {0!r}'.format(self.request))

solenoid/elements/errors.py

@@ -0,0 +1,10 @@
+class Error(Exception):
+    """Base class for exceptions in this module."""
+    pass
+
+
+class RetryError(Error):
+    """Exception raised for HTTP status codes that indicate a Symplectic
+    Elements API call should be retried (409, 500, 504).
+    """
+    pass

solenoid/elements/helpers.py

@@ -0,0 +1,24 @@
+from xml.etree.ElementTree import Element, SubElement
+
+from django.utils import timezone
+
+
+def make_xml(username):
+    top = Element('update-object')
+    top.set('xmlns', 'http://www.symplectic.co.uk/publications/api')
+    oa_field = SubElement(top, 'oa')
+
+    # Update library status field
+    status_field = SubElement(oa_field, 'library-status')
+    status_field.set('status', 'full-text-requested')
+    date_field = SubElement(status_field, 'last-requested-when')
+    date_field.text = timezone.now().isoformat()
+    note_field = SubElement(status_field, 'note-field')
+    note_field.set('clear-existing-note', 'true')
+    note = SubElement(note_field, 'note')
+    note.text = "Library status changed to Full text requested on " \
+        "{date} by {username}.".format(
+            date=timezone.now().strftime('%-d %B %Y'),
+            username=username)
+
+    return top

solenoid/elements/migrations/0003_delete_elementsapicall.py

@@ -0,0 +1,16 @@
+# Generated by Django 2.2.7 on 2019-11-21 19:56
+
+from django.db import migrations
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('elements', '0002_auto_20180308_1812'),
+    ]
+
+    operations = [
+        migrations.DeleteModel(
+            name='ElementsAPICall',
+        ),
+    ]