Correctly filter exact tags when getting the sponsorship brochure

Author: marcoacierno

backend/sponsors/tests/test_views.py

@@ -36,6 +36,9 @@ def test_view_brochure_view(client):
     brochure = DocumentFactory()
     brochure.tags.add("sponsorship-brochure", sponsor_lead.conference.code)
 
+    other_brochure = DocumentFactory()
+    other_brochure.tags.add("sponsorship-brochure", "other-code")
+
     view_brochure_url_path = reverse("view-brochure", args=[sponsor_lead.id])
     signer = Signer()
     signature = signer.sign(view_brochure_url_path).split(signer.sep)[-1]

backend/sponsors/views.py

@@ -12,8 +12,10 @@ def view_brochure(request, sponsor_lead_id):
     sponsor_lead.save(update_fields=["brochure_viewed"])
 
     Document = get_document_model()
-    brochure = Document.objects.filter(
-        tags__name__in=["sponsorship-brochure", sponsor_lead.conference.code],
-    ).first()
+    brochure = (
+        Document.objects.filter(tags__name="sponsorship-brochure")
+        .filter(tags__name=sponsor_lead.conference.code)
+        .first()
+    )
 
     return redirect(brochure.url, permanent=False)