Remove bogus master/builder/slave/force parameters from bot_update.

recipe_modules/bot_update/test_api.py

Index: recipe_modules/bot_update/test_api.py
diff --git a/recipe_modules/bot_update/test_api.py b/recipe_modules/bot_update/test_api.py
index c14d98adb73def226024187dca139e4b3340137b..7294052b1c4d7d50b5e32270471de98ea703c912 100644
--- a/recipe_modules/bot_update/test_api.py
+++ b/recipe_modules/bot_update/test_api.py
@@ -3,69 +3,58 @@
 # found in the LICENSE file.
 
 import hashlib
-import os
 import struct
-import sys
 from recipe_engine import recipe_test_api
 
-# TODO(phajdan.jr): Clean up this somewhat ugly import.
-sys.path.insert(0, os.path.join(os.path.dirname(__file__), 'resources'))
-import bot_update
-
 
 class BotUpdateTestApi(recipe_test_api.RecipeTestApi):
-  def output_json(self, master, builder, slave, root, first_sln,
-                  revision_mapping, force=False, fail_patch=False,
+  def output_json(self, root, first_sln, revision_mapping, fail_patch=False,
                   output_manifest=False, fixed_revisions=None):
     """Deterministically synthesize json.output test data for gclient's
     --output-json option.
     """
-    active = bot_update.check_valid_host(master, builder, slave) or force
-
     output = {
-        'did_run': active,
+        'did_run': True,
         'patch_failure': False
     }
 
-    # Add in extra json output if active.
-    if active:
-      properties = {
-          property_name: self.gen_revision(project_name)
-          for project_name, property_name in revision_mapping.iteritems()
-      }
-      properties.update({
-          '%s_cp' % property_name: ('refs/heads/master@{#%s}' %
-                                    self.gen_commit_position(project_name))
-          for project_name, property_name in revision_mapping.iteritems()
-      })
+    properties = {
+        property_name: self.gen_revision(project_name)
+        for project_name, property_name in revision_mapping.iteritems()
+    }
+    properties.update({
+        '%s_cp' % property_name: ('refs/heads/master@{#%s}' %
+                                  self.gen_commit_position(project_name))
+        for project_name, property_name in revision_mapping.iteritems()
+    })
 
-      output.update({
-          'patch_root': root or first_sln,
-          'root': first_sln,
-          'properties': properties,
-          'step_text': 'Some step text'
-      })
+    output.update({
+        'patch_root': root or first_sln,
+        'root': first_sln,
+        'properties': properties,
+        'step_text': 'Some step text'
+    })
 
-      if output_manifest:
-        output.update({
-          'manifest': {
-            project_name: {
-              'repository': 'https://fake.org/%s.git' % project_name,
-              'revision': self.gen_revision(project_name),
-            }
-            for project_name in revision_mapping
+    if output_manifest:
+      output.update({
+        'manifest': {
+          project_name: {
+            'repository': 'https://fake.org/%s.git' % project_name,
+            'revision': self.gen_revision(project_name),
           }
-        })
+          for project_name in revision_mapping
+        }
+      })
 
-      if fixed_revisions:
-        output['fixed_revisions'] = fixed_revisions
+    if fixed_revisions:
+      output['fixed_revisions'] = fixed_revisions
 
-      if fail_patch:
-        output['log_lines'] = [('patch error', 'Patch failed to apply'),]
-        output['patch_failure'] = True
-        output['patch_apply_return_code'] = 1
-        if fail_patch == 'download':
-          output['patch_apply_return_code'] = 3
+    if fail_patch:
+      output['log_lines'] = [('patch error', 'Patch failed to apply'),]
+      output['patch_failure'] = True
+      output['patch_apply_return_code'] = 1
+      if fail_patch == 'download':
+        output['patch_apply_return_code'] = 3
     return self.m.json.output(output)
 
   @staticmethod