fixed issue expired events

app/common/enums.py

@@ -43,6 +43,7 @@ class Groups(ChoiceEnum):
     JUBKOM = "JubKom"
     REDAKSJONEN = "Redaksjonen"
     FONDET = "Forvaltningsgruppen"
+    PLASK = "Plask"
 
 
 class AppModel(ChoiceEnum):

app/content/views/event.py

@@ -69,14 +69,16 @@ def _list_queryset(self):
         if activity and category:
             return self._list_activity_queryset(category, expired, time)
 
+        if (
+            expired and 
+            category and
+            not activity
+        ):
+            return self.queryset.filter(end_date__lt=time).filter(~Q(category=category))
+
         if expired:
             return self.queryset.filter(end_date__lt=time).order_by("-start_date")
 
-        if category:
-            return self.queryset.filter(end_date__gte=time).filter(
-                ~Q(category=category)
-            )
-
         return self.queryset.filter(end_date__gte=time)
 
     def _list_activity_queryset(self, category, expired, time):

app/tests/conftest.py

@@ -112,6 +112,12 @@ def jubkom_member(member):
     return member
 
 
+@pytest.fixture()
+def plask_member(member):
+    add_user_to_group_with_name(member, Groups.PLASK)
+    return member
+
+
 @pytest.fixture()
 def member_client(member):
     return get_api_client(user=member)

app/tests/content/test_event_integration.py

@@ -174,10 +174,7 @@ def test_list_as_anonymous_user(default_client, event):
     """An anonymous user should be able to list all events that are not activities."""
 
     category = Category.objects.create(text="Aktivitet")
-    activity = EventFactory(category=category)
-
-    activity.category = category
-    activity.save()
+    EventFactory(category=category)
 
     event.category = None
     event.save()
@@ -192,10 +189,7 @@ def test_list_activities_as_anonymous_user(default_client, event):
     """An anonymous user should be able to list all activities."""
 
     category = Category.objects.create(text="Aktivitet")
-    activity = EventFactory(category=category)
-
-    activity.category = category
-    activity.save()
+    EventFactory(category=category)
 
     event.category = None
     event.save()
@@ -206,6 +200,26 @@ def test_list_activities_as_anonymous_user(default_client, event):
     assert response.json().get("count") == 1
 
 
+@pytest.mark.django_db
+def test_list_expired_events_as_anonymous_user(default_client, event):
+    """
+    An anonymous user should be able to list all expired events.
+    Activites should not be included.
+    """
+
+    two_days_ago = now() - timedelta(days=1)
+    event.end_date = two_days_ago
+    event.save()
+
+    category = Category.objects.create(text="Aktivitet")
+    EventFactory(end_date=two_days_ago, category=category)
+
+    response = default_client.get(f"{API_EVENTS_BASE_URL}?expired=true")
+
+    assert response.status_code == 200
+    assert response.json().get("count") == 1
+
+
 @pytest.mark.django_db
 def test_retrieve_as_anonymous_user(default_client, event):
     """An anonymous user should be able to retrieve an event."""