From 9746503d3536489e7661f6de0a1315ab2785d820 Mon Sep 17 00:00:00 2001 From: Virginia Dooley Date: Wed, 26 Jun 2024 10:57:48 +0100 Subject: [PATCH] Temp hide prev party affiliations --- .../people/includes/_person_intro_card.html | 4 +- wcivf/apps/people/tests/test_person_views.py | 88 +++++++++---------- 2 files changed, 46 insertions(+), 46 deletions(-) diff --git a/wcivf/apps/people/templates/people/includes/_person_intro_card.html b/wcivf/apps/people/templates/people/includes/_person_intro_card.html index 386e3ba78..97992d01d 100644 --- a/wcivf/apps/people/templates/people/includes/_person_intro_card.html +++ b/wcivf/apps/people/templates/people/includes/_person_intro_card.html @@ -44,9 +44,9 @@
{{ object.name }} {% endfor %} - {% if object.previous_party_count %} + {% comment %} {% if object.previous_party_count %} {% include "elections/includes/_previous_party_affiliations.html" with person=object candidacies=object.current_or_future_candidacies %} - {% endif %} + {% endif %} {% endcomment %}
diff --git a/wcivf/apps/people/tests/test_person_views.py b/wcivf/apps/people/tests/test_person_views.py index ffbe93e2c..4c3a8cebf 100644 --- a/wcivf/apps/people/tests/test_person_views.py +++ b/wcivf/apps/people/tests/test_person_views.py @@ -237,53 +237,53 @@ def test_one_candidacy_intro(self): "was a Conservative and Unionist Party candidate in", ) - def test_previous_party_affiliations_in_current_elections(self): - """This is a test for previous party affiliations in the - person intro""" - - election = ElectionFactory( - name="Welsh Assembly Election", - current=True, - election_date="2022-05-01", - slug="local.welsh.assembly.2022-05-01", - ) - party = PartyFactory( - party_name="Conservative and Unionist Party", party_id="ConUnion" - ) - post = PostFactory(label="Welsh Assembly", territory="WLS") - pe = PostElectionFactory( - election=election, - post=post, - ballot_paper_id="local.welsh.assembly.2022-05-01", - ) - person_post = PersonPostFactory( - person=self.person, - election=election, - party=party, - party_name=party.party_name, - post=post, - post_election=pe, - ) + # def test_previous_party_affiliations_in_current_elections(self): + # """This is a test for previous party affiliations in the + # person intro""" + + # election = ElectionFactory( + # name="Welsh Assembly Election", + # current=True, + # election_date="2022-05-01", + # slug="local.welsh.assembly.2022-05-01", + # ) + # party = PartyFactory( + # party_name="Conservative and Unionist Party", party_id="ConUnion" + # ) + # post = PostFactory(label="Welsh Assembly", territory="WLS") + # pe = PostElectionFactory( + # election=election, + # post=post, + # ballot_paper_id="local.welsh.assembly.2022-05-01", + # ) + # person_post = PersonPostFactory( + # person=self.person, + # election=election, + # party=party, + # party_name=party.party_name, + # post=post, + # post_election=pe, + # ) - previous_party_1 = PartyFactory( - party_name="Conservative", party_id="foo_id" - ) - previous_party_2 = PartyFactory(party_name="Labour", party_id="bar_id") - person_post.previous_party_affiliations.add(previous_party_1) - person_post.previous_party_affiliations.add(previous_party_2) - self.assertEqual(person_post.previous_party_affiliations.count(), 2) - self.assertEqual(self.person.current_or_future_candidacies.count(), 1) - response = self.client.get(self.person_url, follow=True) + # previous_party_1 = PartyFactory( + # party_name="Conservative", party_id="foo_id" + # ) + # previous_party_2 = PartyFactory(party_name="Labour", party_id="bar_id") + # person_post.previous_party_affiliations.add(previous_party_1) + # person_post.previous_party_affiliations.add(previous_party_2) + # self.assertEqual(person_post.previous_party_affiliations.count(), 2) + # self.assertEqual(self.person.current_or_future_candidacies.count(), 1) + # response = self.client.get(self.person_url, follow=True) - self.assertTemplateUsed( - "elections/includes/_previous_party_affiliations.html" - ) - self.assertTemplateUsed("people/includes/_person_intro_card.html") + # self.assertTemplateUsed( + # "elections/includes/_previous_party_affiliations.html" + # ) + # self.assertTemplateUsed("people/includes/_person_intro_card.html") - self.assertContains( - response, - "has declared their affiliation with the following parties in the past 12 months", - ) + # self.assertContains( + # response, + # "has declared their affiliation with the following parties in the past 12 months", + # ) def test_no_previous_party_affiliations(self): election = ElectionFactory(