diff --git a/ietf/ipr/tests.py b/ietf/ipr/tests.py index d72018f10..67be278db 100644 --- a/ietf/ipr/tests.py +++ b/ietf/ipr/tests.py @@ -32,7 +32,8 @@ from ietf.ipr.factories import ( GenericIprDisclosureFactory, IprDisclosureBaseFactory, IprDocRelFactory, - IprEventFactory + IprEventFactory, + ThirdPartyIprDisclosureFactory ) from ietf.ipr.forms import DraftForm, HolderIprDisclosureForm from ietf.ipr.mail import (process_response_email, get_reply_to, get_update_submitter_emails, @@ -104,9 +105,26 @@ class IprTests(TestCase): self.assertTrue(messages[0].startswith('To: %s' % ipr.submitter_email)) def test_showlist(self): - ipr = HolderIprDisclosureFactory() - r = self.client.get(urlreverse("ietf.ipr.views.showlist")) - self.assertContains(r, ipr.title) + for disc_factory_type in (HolderIprDisclosureFactory, GenericIprDisclosureFactory, ThirdPartyIprDisclosureFactory): + ipr = disc_factory_type(state_id="removed") + r = self.client.get(urlreverse("ietf.ipr.views.showlist")) + self.assertContains(r, ipr.title) + self.assertContains(r, "removed at the request of the submitter") + self.assertNotContains(r, "removed as objectively false") + ipr.state_id="posted" + ipr.save() + r = self.client.get(urlreverse("ietf.ipr.views.showlist")) + self.assertContains(r, ipr.title) + self.assertNotContains(r, "removed at the request of the submitter") + self.assertNotContains(r, "removed as objectively false") + ipr.state_id="removed_objfalse" + ipr.save() + r = self.client.get(urlreverse("ietf.ipr.views.showlist")) + self.assertContains(r, ipr.title) + self.assertNotContains(r, "removed at the request of the submitter") + self.assertContains(r, "removed as objectively false") + ipr.delete() + def test_show_posted(self): ipr = HolderIprDisclosureFactory() diff --git a/ietf/templates/ipr/ipr_table.html b/ietf/templates/ipr/ipr_table.html index ce3982c98..d8247ea58 100644 --- a/ietf/templates/ipr/ipr_table.html +++ b/ietf/templates/ipr/ipr_table.html @@ -39,10 +39,13 @@ {% endif %} {% endfor %} - {% else %} + {% elif ipr.state_id == 'removed' %}