Some tests for public_id/pay setup (bug 834344)

mozilla · Jan 30, 2013 · 3fe78f8 · 3fe78f8
1 parent 7ced0d4
commit 3fe78f8
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/mkt/developers/tests/test_views_payments.py b/mkt/developers/tests/test_views_payments.py
@@ -224,6 +224,12 @@ def test_associate_acct_to_app(self):
         eq_(res.status_code, 200)
         eq_(self.webapp.app_payment_account.payment_account.pk, acct.pk)
 
+        # Make sure the product was created with some properties we care about.
+        kw = self.sol.post_product.call_args[1]['data']
+        assert kw['public_id'], kw
+        assert kw['external_id'], kw
+        assert kw['secret'], kw
+
 
 class TestRegions(amo.tests.TestCase):
     fixtures = ['base/apps', 'base/users', 'webapps/337141-steamcube']

diff --git a/mkt/developers/views_payments.py b/mkt/developers/views_payments.py
@@ -220,7 +220,7 @@ def in_app_config(request, addon_id, addon, webapp=True):
     if request.method == 'POST':
         (client.api.generic
                .product(seller_config['resource_pk'])
-               .patch(data={'secret':generate_key(48)}))
+               .patch(data={'secret': generate_key(48)}))
         messages.success(request, _('Changes successfully saved.'))
         return redirect(reverse('mkt.developers.apps.in_app_config',
                                 args=[addon.app_slug]))