swarming: Add server-side implementation for supplemental bot_config

appengine/swarming/server/bot_groups_config_test.py

 #!/usr/bin/env python
 # Copyright 2016 The LUCI Authors. All rights reserved.
 # Use of this source code is governed under the Apache License, Version 2.0
 # that can be found in the LICENSE file.
 
 import logging
 import sys
 import unittest
 
 import test_env
@@ skipping 28 matching lines @@
       dimensions=['pool:default']),
   ],
 )
 
 
 EXPECTED_GROUP_1 = bot_groups_config._make_bot_group_config(
     require_luci_machine_token=True,
     require_service_account=u'',
     ip_whitelist=u'',
     owners=(u'owner@example.com',),
-    dimensions={u'pool': [u'A', u'B'], u'other': [u'D']})
+    dimensions={u'pool': [u'A', u'B'], u'other': [u'D']},
+    bot_config_script='',
+    bot_config_script_content='')
 
 EXPECTED_GROUP_2 = bot_groups_config._make_bot_group_config(
     require_luci_machine_token=False,
     require_service_account=u'a@example.com',
     ip_whitelist=u'',
     owners=(),
-    dimensions={u'pool': []})
+    dimensions={u'pool': []},
+    bot_config_script='',
+    bot_config_script_content='')
 
 EXPECTED_GROUP_3 = bot_groups_config._make_bot_group_config(
     require_luci_machine_token=False,
     require_service_account=u'',
     ip_whitelist=u'bots',
     owners=(),
-    dimensions={u'pool': [u'default']})
+    dimensions={u'pool': [u'default']},
+    bot_config_script='',
+    bot_config_script_content='')
 
 
 DEFAULT_AUTH_CFG = bots_pb2.BotAuth(ip_whitelist='bots')
 
 
 class BotGroupsConfigTest(test_case.TestCase):
   def validator_test(self, cfg, messages):
     ctx = validation.Context()
     bot_groups_config.validate_settings(cfg, ctx)
     self.assertEquals(ctx.result().messages, [
       validation.Message(severity=logging.ERROR, text=m)
       for m in messages
     ])
 
   def mock_config(self, cfg):
     def get_self_config_mock(path, cls, **_kwargs):
       self.assertEquals('bots.cfg', path)
       self.assertEquals(cls, bots_pb2.BotsCfg)
       return None, cfg
     self.mock(config, 'get_self_config', get_self_config_mock)
     utils.clear_cache(bot_groups_config._fetch_bot_groups)
 
   def test_version(self):
-    self.assertEqual('hash:4564ed4cc34544', EXPECTED_GROUP_1.version)
-    self.assertEqual('hash:18206c33fffaa8', EXPECTED_GROUP_2.version)
+    self.assertEqual('hash:be689c69320bcf', EXPECTED_GROUP_1.version)
+    self.assertEqual('hash:ebaa32cc494672', EXPECTED_GROUP_2.version)
 
   def test_expand_bot_id_expr_success(self):
     def check(expected, expr):
       self.assertEquals(
         expected, list(bot_groups_config._expand_bot_id_expr(expr)))
     check(['abc'], 'abc')
     check(['abc1def', 'abc2def'], 'abc{1,2}def')
     check(['abc1def', 'abc2def', 'abc3def'], 'abc{1..3}def')
 
   def test_expand_bot_id_expr_fail(self):
@@ skipping 333 matching lines @@
       'bot_group #0: machine_type #0: target_size must be positive'
     ])
 
 
 if __name__ == '__main__':
   if '-v' in sys.argv:
     unittest.TestCase.maxDiff = None
   logging.basicConfig(
       level=logging.DEBUG if '-v' in sys.argv else logging.CRITICAL)
   unittest.main()