[recipes.py] move run arg parsing to its module.

recipe_engine/unittests/run_test.py

 #!/usr/bin/env python
 # Copyright 2014 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 json
 import os
 import re
 import subprocess
 import unittest
 import tempfile
 import time
 
 import test_env
 from test_env import BASE_DIR
 
 import recipe_engine.run
 import recipe_engine.step_runner
+from recipe_engine import arguments_pb2
+from google.protobuf import json_format as jsonpb
 from recipe_engine import requests_ssl
 
 class RunTest(unittest.TestCase):
   def _run_cmd(self, recipe, properties=None, engine_args=None):
     script_path = os.path.join(BASE_DIR, 'recipes.py')
 
     if properties:
       proplist = [ '%s=%s' % (k, json.dumps(v))
                    for k,v in properties.iteritems() ]
     else:
@@ skipping 164 matching lines @@
         stderr=subprocess.PIPE)
     stdout, _ = proc.communicate()
     self.assertRegexpMatches(stdout, r'(?m)^!@@@BUILD_STEP@steppy@@@$')
     self.assertRegexpMatches(stdout, r'(?m)^@@@BUILD_STEP@pippy@@@$')
     # Before 'Subannotate me' we expect an extra STEP_CURSOR to reset the
     # state.
     self.assertRegexpMatches(stdout,
         r'(?m)^@@@STEP_CURSOR@Subannotate me@@@\n@@@STEP_CLOSED@@@$')
 
 
+class TestOperationalArgs(unittest.TestCase):
+  def test_operational_arg_parsing(self):
+    # For convenience, we'll define the JSONPB data as a Python dict that we
+    # will then dump into JSON.
+    op_args = jsonpb.Parse(json.dumps({
+      'properties': {'property': {
+        'a': {'s': 'Hello'},
+        'b': {'int': -12345},
+        'c': {'uint': 12345},
+        'd': {'d': 3.14159},
+        'e': {'b': True},
+        'f': {'data': '\x60\x0d\xd0\x65'.encode('base64')},
+        'g': {'map': {
+          'property': {
+            'foo': {'s': 'FOO!'},
+            'bar': {'map': {
+              'property': {
+                'baz': {'s': 'BAZ!'},
+              },
+            }},
+          }},
+        },
+        'h': {'list': {
+          'property': [
+            {'s': 'foo'},
+            {'s': 'bar'},
+            {'s': 'baz'},
+          ],
+        }},
+      }},
+      'annotationFlags': {
+        'emitTimestamp': True,
+      },
+    }), arguments_pb2.Arguments())
+
+    self.assertEqual(
+        recipe_engine.run._op_properties_to_dict(op_args.properties.property),
+        {
+          u'a': u'Hello',
+          u'b': -12345L,
+          u'c': 12345L,
+          u'd': 3.14159,
+          u'e': True,
+          u'f': '\x60\x0d\xd0\x65',
+          u'g': {
+            u'foo': u'FOO!',
+            u'bar': {
+              u'baz': u'BAZ!',
+            },
+          },
+          u'h': [
+            u'foo',
+            u'bar',
+            u'baz',
+          ],
+        })
+
+
 if __name__ == '__main__':
   unittest.TestCase.maxDiff = None
   unittest.main()