From 6fc7cb7ca09422521f8fd4cdc854ff8b27434c32 Mon Sep 17 00:00:00 2001
From: Kevin Ngo <me@ngokevin.com>
Date: Thu, 12 Nov 2015 12:20:08 -0800
Subject: [PATCH] remove old content tool perms (bug 1220623)

---
 mkt/account/serializers.py      |  2 --
 mkt/account/tests/test_views.py | 18 +-----------------
 2 files changed, 1 insertion(+), 19 deletions(-)

diff --git a/mkt/account/serializers.py b/mkt/account/serializers.py
index bc11a831502..3fcaad12da8 100644
--- a/mkt/account/serializers.py
+++ b/mkt/account/serializers.py
@@ -129,8 +129,6 @@ def get_permissions(self, obj):
             'website_submitter': allowed('Websites', 'Submit'),
             'stats': allowed('Stats', 'View'),
             'revenue_stats': allowed('RevenueStats', 'View'),
-            'content_tools_login': True,
-            'content_tools_addon_submit': True,
             'content_tools_addon_review': allowed('ContentTools',
                                                   'AddonReview'),
         }
diff --git a/mkt/account/tests/test_views.py b/mkt/account/tests/test_views.py
index 1fc0230b523..ceefbaa223b 100644
--- a/mkt/account/tests/test_views.py
+++ b/mkt/account/tests/test_views.py
@@ -89,8 +89,7 @@ def test_no_permissions(self):
         self.assertSetEqual(
             ['admin', 'developer', 'localizer', 'lookup', 'curator',
              'reviewer', 'webpay', 'website_submitter', 'stats',
-             'revenue_stats', 'content_tools_login',
-             'content_tools_addon_submit', 'content_tools_addon_review'],
+             'revenue_stats', 'content_tools_addon_review'],
             res.json['permissions'].keys()
         )
         ok_(not all(res.json['permissions'].values()))
@@ -434,12 +433,6 @@ def _test_login(self, http_request):
 
     def test_login_new_user_success(self):
         data = self._test_login()
-
-        # TODO: we want to remove these permissions entirely, but for now we
-        # just grant them to everybody.
-        del data['permissions']['content_tools_login']
-        del data['permissions']['content_tools_addon_submit']
-
         ok_(not any(data['permissions'].values()))
 
     def test_login_existing_user_success(self):
@@ -462,8 +455,6 @@ def test_login_existing_user_success(self):
              'website_submitter': False,
              'stats': False,
              'revenue_stats': False,
-             'content_tools_login': True,
-             'content_tools_addon_submit': True,
              'content_tools_addon_review': False})
         eq_(data['apps']['installed'], [])
         eq_(data['apps']['purchased'], [])
@@ -549,11 +540,6 @@ def test_login_new_user_success(self):
         eq_(UserProfile.objects.count(), 0)
         data = self._test_login()
 
-        # TODO: we want to remove these permissions entirely, but for now we
-        # just grant them to everybody.
-        del data['permissions']['content_tools_login']
-        del data['permissions']['content_tools_addon_submit']
-
         ok_(not any(data['permissions'].values()))
         profile = UserProfile.objects.get()
         eq_(profile.email, 'cvan@mozilla.com')
@@ -582,8 +568,6 @@ def test_login_existing_user_uid_success(self):
              'website_submitter': False,
              'stats': False,
              'revenue_stats': False,
-             'content_tools_login': True,
-             'content_tools_addon_submit': True,
              'content_tools_addon_review': False})
         eq_(data['apps']['installed'], [])
         eq_(data['apps']['purchased'], [])