Handle blocklisting on update_status (bug 800086)

Author: Rob Hudson

mkt/webapps/models.py

@@ -373,7 +373,8 @@ def mark_done(self):
         self.update(status=amo.WEBAPPS_UNREVIEWED_STATUS)
 
     def update_status(self, using=None):
-        if self.is_deleted or self.is_disabled:
+        if (self.is_deleted or self.is_disabled or
+            self.status == amo.STATUS_BLOCKED):
             return
 
         def _log(reason, old=self.status):

mkt/webapps/tests/test_models.py

@@ -990,3 +990,9 @@ def test_one_version_public(self):
                                   file_kw=dict(status=amo.STATUS_DISABLED))
         app.update_status()
         eq_(app.status, amo.STATUS_PUBLIC)
+
+    def test_blocklisted(self):
+        app = amo.tests.app_factory(status=amo.STATUS_BLOCKED)
+        app.current_version.delete()
+        app.update_status()
+        eq_(app.status, amo.STATUS_BLOCKED)