Separate blacklist check in different conditions. (#309)

mozilla · Oct 2, 2014 · a233608 · a233608
1 parent 0d7a47a
commit a233608
Showing 1 changed file with 7 additions and 5 deletions.
diff --git a/pulse.py b/pulse.py
@@ -201,11 +201,13 @@ def on_build(self, data):
         target_branch = self.config['testrun']['by_branch'][data['tree']]
         target_platform = self.get_platform_identifier(data['platform'])
 
-        # Check the locale against the locales blacklist if one is present and
-        # stop processing it if listed there
-        if 'blacklist' in target_branch and 'locales' in target_branch['blacklist'] and \
-                data['locale'] in target_branch['blacklist']['locales']:
-            return
+        # Process the blacklist if one is present
+        if 'blacklist' in target_branch:
+            # Check the locale against the locales blacklist if one is present and
+            # stop processing it if listed there
+            if 'locales' in target_branch['blacklist'] and \
+                    data['locale'] in target_branch['blacklist']['locales']:
+                return
 
         # Process the whitelist if one is present
         if 'locales' in target_branch and \