API: Patron creation problem

* FIX Fixes error during an non-existing patron creation.

Signed-off-by: Johnny Mariéthoz <[email protected]>

Author: jma

rero_ils/config.py

@@ -180,6 +180,9 @@ def _(x):
 ACCOUNTS_SESSION_REDIS_URL = 'redis://localhost:6379/1'
 # Disable User Profiles
 USERPROFILES = False
+# make security blueprints available to the REST API
+ACCOUNTS_REGISTER_BLUEPRINT = True
+
 
 # Celery configuration
 # ====================

rero_ils/modules/ext.py

@@ -77,8 +77,14 @@ def init_app(self, app):
         """Flask application initialization."""
         self.init_config(app)
         app.extensions['rero-ils'] = self
-        app.context_processor(lambda: dict(
-            admin_root_menu=current_admin.admin.menu()))
+
+        # invenio-admin is not available for the wsgi_api application
+        def get_admin_menu():
+            if app.extensions.get('invenio-admin'):
+                return dict(admin_root_menu=current_admin.admin.menu())
+            return dict()
+
+        app.context_processor(get_admin_menu)
 
     def init_config(self, app):
         """Initialize configuration."""

tests/api/test_patrons_rest.py

@@ -154,14 +154,12 @@ def test_patrons_post_put_delete(client, lib_martigny,
     # Create record / POST
     patron_data['pid'] = '1'
     patron_data['email'] = '[email protected]'
-    with mock.patch('rero_ils.modules.patrons.api.'
-                    'send_reset_password_instructions'):
-
-        res = client.post(
-            post_url,
-            data=json.dumps(patron_data),
-            headers=json_header
-        )
+
+    res = client.post(
+        post_url,
+        data=json.dumps(patron_data),
+        headers=json_header
+    )
 
     assert res.status_code == 201
     assert len(Patron.get_all_pids()) == pids + 1

tests/api/test_patrons_views.py

@@ -143,7 +143,7 @@ def test_patron_listener(client, librarian_martigny_no_email,
                          lib_fully, loc_public_martigny,
                          patron_martigny_no_email,
                          patron_martigny,
-                         loan_pending):
+                         loan_pending, mailbox):
     """Test patron listener."""
     login_user_via_session(client, librarian_martigny_no_email.user)
     requests = item_lib_fully.number_of_requests()
@@ -152,12 +152,8 @@ def test_patron_listener(client, librarian_martigny_no_email,
         item_lib_fully.validate_request(**request)
         item_lib_fully.receive(**loan_pending)
 
-    with mock.patch(
-        'rero_ils.modules.ext.current_admin',
-        {}
-    ):
-        sender = {}
-        data = {'item': item_lib_fully}
-        with mock.patch('rero_ils.utils.send_mail'):
-            with pytest.raises(AttributeError):
-                assert listener_item_at_desk(sender, **data)
+    sender = {}
+    data = {'item': item_lib_fully}
+    n_msg = len(mailbox)
+    listener_item_at_desk(sender, **data)
+    assert len(mailbox) == n_msg + 1