Skip to content
This repository was archived by the owner on Sep 19, 2018. It is now read-only.

Commit 541d735

Browse files
committed
Merge pull request #275 from gandalfar/fix_pytest
Fix pytest South migrations, removed workaround for ambassadors test
2 parents 22ef953 + 73ea4fc commit 541d735

File tree

5 files changed

+2
-7
lines changed

5 files changed

+2
-7
lines changed
File renamed without changes.
File renamed without changes.

pytest.ini

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
[pytest]
22
addopts = --reuse-db --strict --create-db
3-
python_files=*.py
43

54
markers =
65
client: marks tests that use the django client (and hence run a bit slow)

requirements.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ oauthlib==0.6.1
2121
py==1.4.23
2222
pytest==2.6.0
2323
pytest-cov==1.7.0
24-
pytest-django==2.6.2
24+
-e git+https://github.com/pelme/pytest_django.git@f10e48d4fb7bca3fa2640f6f1c74c4c10655c76d#egg=pytest_django
2525
python-openid==2.2.5
2626
python-social-auth==0.1.21
2727
requests==2.2.1

web/tests/test_user_processors.py

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -34,11 +34,7 @@ def test_get_ambassadors_for_country(self):
3434
self.up1.country = "SI"
3535
self.up1.save()
3636

37-
# temporary workaround because South migrations don't work yet
38-
# fix me in the future
39-
group, created = Group.objects.get_or_create(name="ambassadors")
40-
if created:
41-
group.save()
37+
group = Group.objects.get(name="ambassadors")
4238

4339
group.user_set.add(self.u1)
4440

0 commit comments

Comments
 (0)