Skip to content

Commit 4f6123b

Browse files
authored
Merge pull request #3958 from italia/fixsometestswithpostgres
Fix some tests with postgres
2 parents 70ffa2f + 119ec94 commit 4f6123b

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

readthedocs/rtd_tests/tests/test_api.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@
1212
from django.contrib.auth.models import User
1313
from django.core.urlresolvers import reverse
1414
from django.test import TestCase
15+
from django.utils import six
1516
from django_dynamic_fixture import get
1617
from rest_framework import status
1718
from rest_framework.test import APIClient
@@ -247,7 +248,8 @@ def test_project_features(self):
247248
resp = client.get('/api/v2/project/%s/' % (project.pk))
248249
self.assertEqual(resp.status_code, 200)
249250
self.assertIn('features', resp.data)
250-
self.assertEqual(
251+
six.assertCountEqual(
252+
self,
251253
resp.data['features'],
252254
[feature1.feature_id, feature2.feature_id],
253255
)

readthedocs/rtd_tests/tests/test_notifications.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,7 @@ class TestNotification(Notification):
8080
request=mock.ANY,
8181
template='core/email/common.txt',
8282
context={'content': 'Test'},
83-
subject=u'This is 1',
83+
subject=u'This is {}'.format(build.id),
8484
template_html='core/email/common.html',
8585
recipient=user.email,
8686
)

0 commit comments

Comments
 (0)