Merge branch 'kerl/fix-wagtail-deps' into 'master'

Règle un problème de dépendances de wagtailmenus avec python 3.8

See merge request klub-dev-ens/gestioCOF!381

.gitlab-ci.yml

@@ -27,7 +27,7 @@ test:
         - sed -i.bak -E 's;^REDIS_PASSWD = .*$;REDIS_PASSWD = "";' cof/settings/secret.py
         # Remove the old test database if it has not been done yet
         - psql --username=$POSTGRES_USER --host=$DBHOST -c "DROP DATABASE IF EXISTS test_$POSTGRES_DB"
-        - pip install --upgrade -r requirements.txt coverage tblib
+        - pip install --upgrade -r requirements-prod.txt coverage tblib
         - python --version
     script:
         - coverage run manage.py test --parallel
@@ -67,7 +67,7 @@ migration_checks:
         - mkdir -p vendor/{pip,apt}
         - apt-get update -q && apt-get -o dir::cache::archives="vendor/apt" install -yqq postgresql-client
         - cp cof/settings/secret_example.py cof/settings/secret.py
-        - pip install --upgrade -r requirements.txt
+        - pip install --upgrade -r requirements-prod.txt
         - python --version
     script: python manage.py makemigrations --dry-run --check
     services:

gestioncof/autocomplete.py

@@ -3,11 +3,17 @@ from django.conf import settings
 from django.contrib.auth.models import User
 from django.db.models import Q
 from django.http import Http404
-from ldap3 import Connection
 
 from gestioncof.decorators import buro_required
 from gestioncof.models import CofProfile
 
+if getattr(settings, "LDAP_SERVER_URL", None):
+    from ldap3 import Connection
+else:
+    # shared.tests.testcases.TestCaseMixin.mockLDAP needs
+    # Connection to be defined in order to mock it.
+    Connection = None
+
 
 class Clipper(object):
     def __init__(self, clipper, fullname):

kfet/autocomplete.py

@@ -2,12 +2,18 @@ from django.conf import settings
 from django.db.models import Q
 from django.http import Http404
 from django.shortcuts import render
-from ldap3 import Connection
 
 from gestioncof.models import User
 from kfet.decorators import teamkfet_required
 from kfet.models import Account
 
+if getattr(settings, "LDAP_SERVER_URL", None):
+    from ldap3 import Connection
+else:
+    # shared.tests.testcases.TestCaseMixin.mockLDAP needs
+    # Connection to be defined in order to mock it.
+    Connection = None
+
 
 class Clipper(object):
     def __init__(self, clipper, fullname):

kfet/cms/fixtures/kfet_wagtail_02_19.json

@@ -1598,8 +1598,7 @@
             "localhost",
             8000
         ],
-        "max_levels": 2,
-        "use_specific": 1
+        "max_levels": 2
     }
 },
 {
@@ -1613,8 +1612,7 @@
         "title": "K-F\u00eat - Navigation",
         "handle": "kfet-nav",
         "heading": "",
-        "max_levels": 1,
-        "use_specific": 1
+        "max_levels": 1
     }
 }
 ]

requirements-prod.txt

@@ -0,0 +1,14 @@
+-r requirements.txt
+
+# Postgresql bindings
+psycopg2<2.8
+
+# Redis
+django-redis-cache==2.1.*
+
+# ASGI protocol and HTTP server
+asgiref==1.1.1
+daphne==1.3.0
+
+# ldap bindings
+ldap3

requirements.txt

@@ -4,21 +4,16 @@ django-autocomplete-light==3.3.*
 django-cas-ng==3.6.*
 django-djconfig==0.8.0
 django-recaptcha==1.4.0
-django-redis-cache==1.8.1
 icalendar
-psycopg2<2.8
 Pillow
 django-bootstrap-form==3.3
-asgiref==1.1.1
-daphne==1.3.0
 asgi-redis==1.3.0
 statistics==1.0.3.5
 django-widget-tweaks==1.4.1
 git+https://git.eleves.ens.fr/cof-geek/django_custommail.git#egg=django_custommail
-ldap3
 channels==1.1.5
 python-dateutil
-wagtail==2.4.*
-wagtailmenus<3
+wagtail==2.7.*
+wagtailmenus==3.*
 wagtail-modeltranslation==0.10.*
 django-cors-headers==2.2.0