Supporting more APIs with json_schema_compiler

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 155b78844f49de6b11d21a18919d90eb8c1172bf..32eb60fdffb2b53afc91d5b6cf4b5fa063890329 100644
--- a/tools/json_schema_compiler/cpp_type_generator_test.py
+++ b/tools/json_schema_compiler/cpp_type_generator_test.py
@@ -23,16 +23,22 @@ class CppTypeGeneratorTest(unittest.TestCase):
         'path/to/tabs.json')
     self.tabs = self.model.namespaces.get('tabs')
 
-  def testGenerateCppIncludes(self):
+  def testGenerateIncludesAndForwardDeclarations(self):
     manager = CppTypeGenerator('', self.windows, 'windows_api')
     manager.AddNamespace(self.tabs, 'tabs_api')
     self.assertEquals('#include "path/to/tabs_api.h"',
-        manager.GenerateCppIncludes().Render())
+        manager.GenerateIncludes().Render())
+    self.assertEquals(
+        'namespace tabs {\n'
+        'struct Tab;\n'
+        '}',
+        manager.GenerateForwardDeclarations().Render())
     manager = CppTypeGenerator('', self.permissions, 'permissions_api')
     manager.AddNamespace(self.permissions, 'permissions_api')
-    self.assertEquals('', manager.GenerateCppIncludes().Render())
+    self.assertEquals('', manager.GenerateIncludes().Render())
+    self.assertEquals('', manager.GenerateForwardDeclarations().Render())
 
-  def testGenerateCppIncludesMultipleTypes(self):
+  def testGenerateIncludesAndForwardDeclaratiosnMultipleTypes(self):
     m = model.Model()
     self.tabs_json[0]['types'].append(self.permissions_json[0]['types'][0])
     tabs_namespace = m.AddNamespace(self.tabs_json[0],
@@ -44,7 +50,13 @@ class CppTypeGeneratorTest(unittest.TestCase):
     manager = CppTypeGenerator('', windows, 'windows_api')
     manager.AddNamespace(tabs_namespace, 'tabs_api')
     self.assertEquals('#include "path/to/tabs_api.h"',
-        manager.GenerateCppIncludes().Render())
+        manager.GenerateIncludes().Render())
+    self.assertEquals(
+        'namespace tabs {\n'
+        'struct Tab;\n'
+        'struct Permissions;\n'
+        '}',
+        manager.GenerateForwardDeclarations().Render())
 
   def testGetTypeSimple(self):
     manager = CppTypeGenerator('', self.tabs, 'tabs_api')
@@ -60,9 +72,9 @@ class CppTypeGeneratorTest(unittest.TestCase):
 
   def testGetTypeArray(self):
     manager = CppTypeGenerator('', self.windows, 'windows_api')
-    self.assertEquals('std::vector<Window>',
+    self.assertEquals('std::vector<linked_ptr<Window> >',
         manager.GetType(
-        self.windows.functions['getAll'].callback.param))
+        self.windows.functions['getAll'].callback.params[0]))
     manager = CppTypeGenerator('', self.permissions, 'permissions_api')
     self.assertEquals('std::vector<std::string>',
         manager.GetType(
@@ -72,12 +84,12 @@ class CppTypeGeneratorTest(unittest.TestCase):
     manager = CppTypeGenerator('', self.tabs, 'tabs_api')
     self.assertEquals('Tab',
         manager.GetType(
-        self.tabs.functions['get'].callback.param))
+        self.tabs.functions['get'].callback.params[0]))
 
   def testGetTypeIncludedRef(self):
     manager = CppTypeGenerator('', self.windows, 'windows_api')
     manager.AddNamespace(self.tabs, 'tabs_api')
-    self.assertEquals('std::vector<tabs_api::Tab>',
+    self.assertEquals('std::vector<linked_ptr<tabs_api::Tab> >',
         manager.GetType(
         self.windows.types['Window'].properties['tabs']))
 
@@ -101,25 +113,23 @@ class CppTypeGeneratorTest(unittest.TestCase):
         pad_for_generics=True))
     self.assertEquals('bool',
         manager.GetType(
-        self.permissions.functions['contains'].callback.param,
+        self.permissions.functions['contains'].callback.params[0],
         pad_for_generics=True))
 
   def testNamespaceDeclaration(self):
     manager = CppTypeGenerator('extensions', self.permissions,
                                'permissions_api')
     self.assertEquals(
-        'namespace extensions {\n'
-        'namespace permissions_api {',
-        manager.GetCppNamespaceStart().Render())
+        'namespace extensions {',
+        manager.GetRootNamespaceStart().Render())
 
     manager = CppTypeGenerator('extensions::gen::api', self.permissions,
                                'permissions_api')
     self.assertEquals(
         'namespace extensions {\n'
         'namespace gen {\n'
-        'namespace api {\n'
-        'namespace permissions_api {',
-        manager.GetCppNamespaceStart().Render())
+        'namespace api {',
+        manager.GetRootNamespaceStart().Render())
 
 if __name__ == '__main__':
   unittest.main()