Add tests for Project.display_tags

Also, fix the various failures up the tree which prevented these new tests from
passing

cdhweb/pages/tests/conftest.py

@@ -8,6 +8,7 @@
 from django.views.generic.detail import DetailView
 from django.views.generic.list import ListView
 from wagtail.models import Page, Site
+from wagtail_factories import ImageFactory
 
 from cdhweb.pages.models import ContentPage, ExternalAttachment, HomePage, LandingPage
 from cdhweb.pages.views import LastModifiedListMixin, LastModifiedMixin
@@ -34,6 +35,7 @@ def make_homepage(site):
         body=json.dumps(
             [{"type": "paragraph", "value": "<p>content of the home page</p>"}]
         ),
+        hero_image=ImageFactory(),
     )
     root.add_child(instance=home)
     root.save()

cdhweb/projects/models.py

@@ -282,6 +282,16 @@ def get_sitemap_urls(self, request):
         return urls
 
     def display_tags(self):
+        """
+        Get role, method and field values as tag-y objects
+
+        The method/field/role fields are used for filtering
+        the projects in the search, but CDH have also
+        requested that they be shown the same way tags are
+        for many other types of object -- This function
+        delivers them in a (template-equivalent) manner so
+        they can be displayed
+        """
         return sorted(
             str(t)
             for t in itertools.chain(

cdhweb/projects/tests/conftest.py

@@ -4,6 +4,7 @@
 
 import pytest
 from django.utils import timezone
+from wagtail_factories import ImageFactory
 
 from cdhweb.people.models import Person
 from cdhweb.projects.models import (
@@ -34,9 +35,7 @@ def add_project_member(project, role, start_date=None, end_date=None, **person_o
 
 def make_projects_landing_page(homepage):
     """create a test projects landing page underneath the homepage"""
-    plp = ProjectsLandingPageArchived(
-        title="projects", slug="projects", tagline="let's do some stuff"
-    )
+    plp = ProjectsLandingPageArchived(title="projects", slug="projects")
     homepage.add_child(instance=plp)
     homepage.save()
     return plp

cdhweb/projects/tests/test_models.py

@@ -1,7 +1,15 @@
 from datetime import datetime, timedelta
 
 from cdhweb.pages.models import RelatedLinkType
-from cdhweb.projects.models import GrantType, Project, ProjectRelatedLink, Role
+from cdhweb.projects.models import (
+    GrantType,
+    Project,
+    ProjectField,
+    ProjectMethod,
+    ProjectRelatedLink,
+    ProjectRole,
+    Role,
+)
 
 
 class TestGrantType:
@@ -140,3 +148,35 @@ def test_display_url(self, derrida):
         # https
         res.url = "https://%s" % base_url
         assert res.display_url == base_url
+
+
+class TestProjectDisplayTags:
+    def test_no_values(self, derrida):
+        assert derrida.display_tags() == []
+
+    def test_just_role(self, derrida):
+        role = ProjectRole(role="test role")
+        role.save()
+        derrida.role.add(role)
+        assert derrida.display_tags() == ["test role"]
+
+    def test_all_relations(self, derrida):
+        role1 = ProjectRole(role="test role 1")
+        role2 = ProjectRole(role="test role 2")
+        method = ProjectMethod(method="test method")
+        field = ProjectField(field="test field")
+        role1.save()
+        role2.save()
+        method.save()
+        field.save()
+
+        derrida.role.add(role1)
+        derrida.role.add(role2)
+        derrida.method.add(method)
+        derrida.field.add(field)
+        assert derrida.display_tags() == [
+            "test field",
+            "test method",
+            "test role 1",
+            "test role 2",
+        ]

docker/application/Dockerfile

@@ -53,8 +53,7 @@ RUN --mount=type=cache,target=/var/lib/apt/lists \
     netcat
 
 RUN --mount=type=cache,target=/root/.cache/pip \
-    pip3 install -r /app/requirements/dev.txt \
-    npm install
+    pip3 install -r /app/requirements/dev.txt
 
 # No COPY here as the local directory is volume-mounted in docker-compose
 

requirements/test.txt

@@ -9,4 +9,5 @@ wheel
 # fix issue with taggit fields: https://github.com/makecodes/django-dbml/pull/4
 django-dbml
 #git+https://github.com/thatbudakguy/django-dbml@patch-1#egg=django-dbml
-percy-selenium
+percy-selenium
+wagtail-factories~=4.2.1