Skip to content

Commit

Permalink
Revert "commit transactions manually after task"
Browse files Browse the repository at this point in the history
This reverts commit 0448d0c.
  • Loading branch information
brad committed Dec 18, 2014
1 parent 0448d0c commit ff7fcf7
Showing 1 changed file with 0 additions and 6 deletions.
6 changes: 0 additions & 6 deletions misfitapp/tests/test_tasks.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@
from django.core.cache import cache
from django.core.files.uploadedfile import InMemoryUploadedFile
from django.core.urlresolvers import reverse
from django.db import transaction
from django.test.utils import override_settings
from freezegun import freeze_time
from httmock import HTTMock, urlmatch
Expand Down Expand Up @@ -157,7 +156,6 @@ def setUp(self):
'UnsubscribeURL': 'https://xxxx'
}

@transaction.commit_manually
@patch('misfit.notification.MisfitNotification.verify_signature')
def test_subscription_confirmation(self, verify_signature_mock):
"""
Expand All @@ -168,13 +166,11 @@ def test_subscription_confirmation(self, verify_signature_mock):
with HTTMock(sns_subscribe):
self.client.post(reverse('misfit-notification'), data=content,
content_type='application/json')
transaction.commit()
verify_signature_mock.assert_called_once_with()
self.assertEqual(Goal.objects.count(), 0)
self.assertEqual(Profile.objects.count(), 0)
self.assertEqual(Summary.objects.count(), 0)

@transaction.commit_manually
@patch('misfit.notification.MisfitNotification.verify_signature')
def test_notification(self, verify_signature_mock):
"""
Expand All @@ -186,8 +182,6 @@ def test_notification(self, verify_signature_mock):
JsonMock('summary_detail').summary_http):
self.client.post(reverse('misfit-notification'), data=content,
content_type='application/json')
transaction.commit()
verify_signature_mock.assert_called_once_with()
eq_(Goal.objects.filter(user=self.user).count(), 2)
eq_(Profile.objects.filter(user=self.user).count(), 1)
eq_(Summary.objects.filter(user=self.user).count(), 3)
Expand Down

0 comments on commit ff7fcf7

Please sign in to comment.