Merge pull request #221 from davidbgk/833445-removing-issuer-notice

 Removing Issuer and Notice models (bug 833445)

Author: davidbgk

lib/solitude/api.py

@@ -9,7 +9,6 @@
 from ..utils import SlumberWrapper
 from .constants import ACCESS_PURCHASE
 from .errors import ERROR_STRINGS
-from webpay.pay.models import Issuer
 
 
 log = logging.getLogger('w.solitude')
@@ -273,10 +272,6 @@ def get_transaction(self, uuid):
         notes = transaction['notes']
         if notes:
             transaction['notes'] = json.loads(notes)
-            issuer = transaction['notes'].get('issuer')
-            if issuer:
-                # If there's an issuer there, get it.
-                transaction['notes']['issuer'] = Issuer.objects.get(pk=issuer)
         return transaction
 
 

lib/solitude/tests.py

@@ -10,7 +10,6 @@
 
 from lib.solitude.api import client, SellerNotConfigured
 from lib.solitude.errors import ERROR_STRINGS
-from webpay.pay.models import Issuer
 
 
 class SolitudeAPITest(TestCase):
@@ -234,11 +233,3 @@ def test_notes_transactions(self, slumber):
         }
         trans = client.get_transaction('x')
         eq_(trans['notes'], {'foo': 'bar'})
-
-    def test_notes_issuer_transactions(self, slumber):
-        iss = Issuer.objects.create()
-        slumber.generic.transaction.get_object.return_value = {
-                'notes': json.dumps({'issuer': iss.pk})
-        }
-        trans = client.get_transaction('x')
-        eq_(trans['notes']['issuer'], iss)

migrations/012-drop-notices.sql

@@ -0,0 +1 @@
+DROP TABLE `notices`;

migrations/013-drop-issuers.sql

@@ -0,0 +1 @@
+DROP TABLE `issuers`;

webpay/pay/constants.py

@@ -0,0 +1,3 @@
+NOT_SIMULATED = 0
+SIMULATED_POSTBACK = 1
+SIMULATED_CHARGEBACK = 2

webpay/pay/fields.py

@@ -0,0 +1,11 @@
+from django.db import models
+
+
+class BlobField(models.Field):
+    """
+    MySQL blob column.
+    """
+    description = "blob"
+
+    def db_type(self, **kw):
+        return 'blob'

webpay/pay/models.py

@@ -20,8 +20,7 @@
 
 from webpay.base.helpers import absolutify
 from webpay.constants import TYP_CHARGEBACK, TYP_POSTBACK
-from .models import (Notice, NOT_SIMULATED, SIMULATED_POSTBACK,
-                     SIMULATED_CHARGEBACK)
+from .constants import NOT_SIMULATED, SIMULATED_POSTBACK, SIMULATED_CHARGEBACK
 from .utils import send_pay_notice, trans_id
 
 log = logging.getLogger('w.pay.tasks')
@@ -361,13 +360,6 @@ def _notify(notifier_task, trans, extra_response=None, simulated=NOT_SIMULATED,
     success, last_error = send_pay_notice(url, trans['type'], signed_notice,
                                           trans['uuid'], notifier_task,
                                           task_args, simulated=simulated)
-    s = Notice._meta.get_field_by_name('last_error')[0].max_length
-    last_error = last_error[:s]  # truncate to fit
-    Notice.objects.create(transaction_uuid=trans['uuid'],
-                          success=success,
-                          url=url,
-                          simulated=simulated,
-                          last_error=last_error)
 
 
 def _prepare_notice(trans):

webpay/pay/tasks.py

@@ -6,7 +6,6 @@
 from lib.solitude import constants
 
 from webpay.base.tests import BasicSessionCase
-from webpay.pay.models import Issuer, ISSUER_ACTIVE
 from webpay.pay.samples import JWTtester
 
 sample = os.path.join(os.path.dirname(__file__), 'sample.key')
@@ -19,7 +18,6 @@ def setUp(self):
         self.url = reverse('pay.lobby')
         self.key = 'public.key'
         self.secret = 'private.secret'
-        self.create()
 
     def get(self, payload):
         return self.client.get('%s?req=%s' % (self.url, payload))
@@ -35,14 +33,6 @@ def request(self, **kw):
         kw.setdefault('app_secret', settings.SECRET)
         return super(Base, self).request(**kw)
 
-    def create(self, key=None, secret=None):
-        key = key or self.key
-        secret = secret or self.secret
-        self.iss = Issuer.objects.create(issuer_key=key,
-                                         status=ISSUER_ACTIVE)
-        with self.settings(INAPP_KEY_PATHS={None: sample}, DEBUG=True):
-            self.iss.set_private_key(secret)
-
     def set_secret(self, get_active_product):
         get_active_product.return_value = {
             'secret': self.secret,

webpay/pay/tests/__init__.py

@@ -9,7 +9,6 @@
 from nose.tools import eq_
 
 from webpay.pay.forms import VerifyForm
-from webpay.pay.models import ISSUER_INACTIVE
 
 from . import Base, sample
 
@@ -89,8 +88,6 @@ def test_not_public(self):
         # an active status in solitude.
         raise SkipTest
 
-        self.iss.status = ISSUER_INACTIVE
-        self.iss.save()
         payload = self.request(iss=self.key, app_secret=self.secret)
         with self.settings(INAPP_KEY_PATHS={None: sample}, DEBUG=True):
             form = VerifyForm({'req': payload})