Skip to content

Commit f5ae254

Browse files
committed
Replaced multiple asserts with assertContains().
- Legacy-Id: 16353
1 parent f13887d commit f5ae254

5 files changed

Lines changed: 13 additions & 12 deletions

File tree

ietf/secr/announcement/tests.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
# Copyright The IETF Trust 2013-2019, All Rights Reserved
12
from django.urls import reverse
23

34
from pyquery import PyQuery
@@ -94,8 +95,7 @@ def test_valid_submit(self):
9495
'body':'This is a test.'}
9596
self.client.login(username="secretary", password="secretary+password")
9697
response = self.client.post(url,post_data)
97-
self.assertEqual(response.status_code, 200)
98-
self.assertTrue('Confirm Announcement' in response.content)
98+
self.assertContains(response, 'Confirm Announcement')
9999
response = self.client.post(confirm_url,post_data,follow=True)
100100
self.assertRedirects(response, url)
101101
self.assertEqual(len(outbox),1)

ietf/secr/meetings/tests.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -224,7 +224,7 @@ def test_meetings_times(self):
224224
'name':'Test Morning Session'
225225
}, follow=True)
226226
self.assertRedirects(response, url)
227-
self.assertTrue('Test Morning Session' in response.content)
227+
self.assertContains(response, 'Test Morning Session')
228228

229229
def test_meetings_times_delete(self):
230230
meeting = make_meeting_test_data()
@@ -310,7 +310,7 @@ def test_meetings_nonsession_add_invalid(self):
310310
'group':group.pk,
311311
})
312312
self.assertEqual(response.status_code, 200)
313-
self.assertTrue('invalid format' in response.content)
313+
self.assertContains(response, 'invalid format')
314314

315315
def test_meetings_nonsession_edit(self):
316316
meeting = make_meeting_test_data()

ietf/secr/proceedings/tests.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -87,15 +87,15 @@ def test_post(self):
8787
self.client.login(username="secretary", password="secretary+password")
8888
response = self.client.post(url,data,follow=True)
8989
self.assertEqual(response.status_code, 200)
90-
self.assertTrue(group.acronym in response.content)
90+
self.assertContains(response, group.acronym)
9191

9292
# now test edit
9393
doc = session.materials.filter(type='recording').first()
9494
external_url = 'http://youtube.com/aaa'
9595
url = reverse('ietf.secr.proceedings.views.recording_edit', kwargs={'meeting_num':meeting.number,'name':doc.name})
9696
response = self.client.post(url,dict(external_url=external_url),follow=True)
9797
self.assertEqual(response.status_code, 200)
98-
self.assertTrue(external_url in response.content)
98+
self.assertContains(response, external_url)
9999

100100
def test_import_audio_files(self):
101101
session = SessionFactory(status_id='sched',meeting__type_id='ietf')

ietf/secr/roles/tests.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
# Copyright The IETF Trust 2013-2019, All Rights Reserved
12
# -*- coding: utf-8 -*-
23
from django.urls import reverse
34
from ietf.utils.test_utils import TestCase
@@ -47,7 +48,7 @@ def test_roles_add(self):
4748
self.client.login(username="secretary", password="secretary+password")
4849
response = self.client.post(url,post_data,follow=True)
4950
self.assertRedirects(response, target)
50-
self.assertTrue('added successfully' in response.content)
51+
self.assertContains(response, 'added successfully')
5152

5253
def test_roles_add_no_group(self):
5354
person = Person.objects.get(name='Areað Irector')
@@ -60,4 +61,4 @@ def test_roles_add_no_group(self):
6061
self.client.login(username="secretary", password="secretary+password")
6162
response = self.client.post(url,post_data,follow=True)
6263
self.assertEqual(response.status_code, 200)
63-
self.assertTrue('You must select a group' in response.content)
64+
self.assertContains(response, 'You must select a group')

ietf/secr/telechat/tests.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -76,9 +76,9 @@ def test_doc_detail_draft_with_downref(self):
7676
self.client.login(username="secretary", password="secretary+password")
7777
response = self.client.get(url)
7878
self.assertEqual(response.status_code, 200)
79-
self.assertTrue("Has downref: Yes" in response.content)
80-
self.assertTrue("Add rfc6666" in response.content)
81-
self.assertTrue("to downref registry" in response.content)
79+
self.assertContains(response, "Has downref: Yes")
80+
self.assertContains(response, "Add rfc6666")
81+
self.assertContains(response, "to downref registry")
8282

8383
def test_doc_detail_draft_invalid(self):
8484
'''Test using a document not on telechat agenda'''
@@ -88,7 +88,7 @@ def test_doc_detail_draft_invalid(self):
8888
self.client.login(username="secretary", password="secretary+password")
8989
response = self.client.get(url, follow=True)
9090
self.assertRedirects(response, reverse('ietf.secr.telechat.views.doc', kwargs={'date':date}))
91-
self.assertTrue('not on the Telechat agenda' in response.content)
91+
self.assertContains(response, 'not on the Telechat agenda')
9292

9393
def test_doc_detail_charter(self):
9494
by=Person.objects.get(name="(System)")

0 commit comments

Comments
 (0)