JSON Schema Compiler: Added event compilation.

tools/json_schema_compiler/cpp_type_generator_test.py

Index: tools/json_schema_compiler/cpp_type_generator_test.py
diff --git a/tools/json_schema_compiler/cpp_type_generator_test.py b/tools/json_schema_compiler/cpp_type_generator_test.py
index b22c430274409daa7a20c93fa5c57af6553249b3..f8cf158e9e4797792c7a15662c43e7e3079bbe5b 100755
--- a/tools/json_schema_compiler/cpp_type_generator_test.py
+++ b/tools/json_schema_compiler/cpp_type_generator_test.py
@@ -210,15 +210,27 @@ class CppTypeGeneratorTest(unittest.TestCase):
         '}  // extensions',
         manager.GetRootNamespaceEnd().Render())
 
-  def testExpandChoicesInParams(self):
+  def testGetAllPossibleParameters(self):
     manager = CppTypeGenerator('extensions', self.tabs,
                                self.tabs.unix_name)
     props = self.tabs.functions['move'].params
     self.assertEquals(2, len(props))
-    self.assertEquals(3, len(manager.GetExpandedChoicesInParams(props)))
+    self.assertEquals(3, len(manager.GetAllPossibleParameters(props)))
     self.assertEquals(['move_properties', 'tab_ids_array', 'tab_ids_integer'],
-        sorted([x.unix_name for x in manager.GetExpandedChoicesInParams(props)])
+        sorted([x.unix_name for x in manager.GetAllPossibleParameters(props)])
     )
 
+  def testGetAllPossibleParameterLists(self):
+    manager = CppTypeGenerator('extensions', self.tabs,
+                               self.tabs.unix_name)

[not at google - send to devlin, 2012/07/11 07:22:06]

nit: parameters should all be on 1 line or vertically aligned

[Matt Tytel, 2012/07/12 03:07:56]

Ah copied this from the function above.
Done.

+    props = self.tabs.functions['move'].params
+    self.assertEquals(2, len(props))
+    self.assertEquals(2, len(manager.GetAllPossibleParameterLists(props)))
+    expected_lists = [['move_properties', 'tab_ids_array'],

[not at google - send to devlin, 2012/07/11 07:22:06]

Not being used?

Looks like this was half transitioned to doing a direct comparison rather than
based on length (which is the right change)?

tabs.move isn't the most interesting choice, though. Ideally we should use a
function with at least 2 choices in it, and at least 2 non-choices, to test the
permutations. The other tests you've added in this change are pretty epic
though...

[Matt Tytel, 2012/07/12 03:07:56]

Dang rushed through this test. Somehow deleted the comparison at the bottom.
Done.

+                      ['move_properties', 'tab_ids_integer']]
+    param_lists = manager.GetAllPossibleParameterLists(props)

[not at google - send to devlin, 2012/07/11 07:22:06]

looks like this is being evaluated twice? move this line up?

[Matt Tytel, 2012/07/12 03:07:56]

Done.

+    result_lists = sorted([sorted([param.unix_name for param in param_list])
+                          for param_list in param_lists])
+
 if __name__ == '__main__':
   unittest.main()