Docserver: Generate a table of extension/app API owners

chrome/common/extensions/docs/server2/cron_servlet_test.py

 #!/usr/bin/env python
 # Copyright 2013 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import unittest
 
 from app_yaml_helper import AppYamlHelper
 from content_providers import IgnoreMissingContentProviders
 from cron_servlet import CronServlet
@@ skipping 69 matching lines @@
     # When re-running, all file systems should be Stat()d the same number of
     # times, but the second round shouldn't have been re-Read() since the
     # Stats haven't changed.
     response = CronServlet(Request.ForTest('trunk'),
                            delegate_for_test=delegate).Get()
     self.assertEqual(200, response.status)
 
     self.assertEqual(len(first_run_file_systems), len(delegate.file_systems))
     for i, second_run_file_system in enumerate(delegate.file_systems):
       self.assertTrue(*second_run_file_system.CheckAndReset(
           read_count=0,

[ahernandez, 2014/08/13 23:24:28]

I'm not sure what to do about this test. One of the file systems has a bunch of
read counts because of OwnersDataSource. I'm passing all other tests besides
this.

[not at google - send to devlin, 2014/08/14 00:05:05]

This test is supposed to catch anything that doesn't cache the file system data
in its Cron run. At least, with respect to the next Cron run. What's weird is
that I'm staring at it and OwnersDataSource looks fine from that perspective.

           stat_count=first_run_file_systems[i].GetStatCount()))
 
   @IgnoreMissingContentProviders
   def testSafeRevision(self):
     test_data = {
-      'api': {
-        '_api_features.json': '{}',
-        '_manifest_features.json': '{}',
-        '_permission_features.json': '{}',
+      'extensions': {
+        'browser': {
+          'api': {}
+        }
       },
-      'docs': {
-        'examples': {
-          'examples.txt': 'examples.txt contents'
+      'chrome': {
+        'browser': {
+          'extensions': {
+            'OWNERS': '',
+            'api': {}
+          }
         },
-        'server2': {
-          'app.yaml': AppYamlHelper.GenerateAppYaml('2-0-8')
-        },
-        'static': {
-          'static.txt': 'static.txt contents'
-        },
-        'templates': {
-          'articles': {
-            'activeTab.html': 'activeTab.html contents'
-          },
-          'intros': {
-            'browserAction.html': 'activeTab.html contents'
-          },
-          'private': {
-            'table_of_contents.html': 'table_of_contents.html contents',
-          },
-          'public': {
-            'apps': {
-              'storage.html': '<h1>storage.html</h1> contents'
+        'common': {
+          'extensions': {
+            'api': {
+              '_api_features.json': '{}',
+              '_manifest_features.json': '{}',
+              '_permission_features.json': '{}',
             },
-            'extensions': {
-              'storage.html': '<h1>storage.html</h1> contents'
-            },
-          },
-          'json': {
-            'chrome_sidenav.json': '{}',
-            'content_providers.json': ReadFile(CONTENT_PROVIDERS),
-            'manifest.json': '{}',
-            'permissions.json': '{}',
-            'strings.json': '{}',
-            'whats_new.json': '{}',
-          },
+            'docs': {
+              'examples': {
+                'examples.txt': 'examples.txt contents'
+              },
+              'server2': {
+                'app.yaml': AppYamlHelper.GenerateAppYaml('2-0-8')
+              },
+              'static': {
+                'static.txt': 'static.txt contents'
+              },
+              'templates': {
+                'articles': {
+                  'activeTab.html': 'activeTab.html contents'
+                },
+                'intros': {
+                  'browserAction.html': 'activeTab.html contents'
+                },
+                'private': {
+                  'table_of_contents.html': 'table_of_contents.html contents',
+                },
+                'public': {
+                  'apps': {
+                    'storage.html': '<h1>storage.html</h1> contents'
+                  },
+                  'extensions': {
+                    'storage.html': '<h1>storage.html</h1> contents'
+                  },
+                },
+                'json': {
+                  'chrome_sidenav.json': '{}',
+                  'content_providers.json': ReadFile(CONTENT_PROVIDERS),
+                  'manifest.json': '{}',
+                  'permissions.json': '{}',
+                  'strings.json': '{}',
+                  'whats_new.json': '{}',
+                },
+              }
+            }
+          }
         }
       }
     }
 
     updates = []
 
     def app_yaml_update(version):
       return MoveTo(SERVER2, {
         'app.yaml': AppYamlHelper.GenerateAppYaml(version)
       })
     def storage_html_update(update):
       return MoveTo(PUBLIC_TEMPLATES, {
         'apps': {'storage.html': update}
       })
     def static_txt_update(update):
       return MoveTo(STATIC_DOCS, {
         'static.txt': update
       })
 
     storage_html_path = PUBLIC_TEMPLATES + 'apps/storage.html'
     static_txt_path = STATIC_DOCS + 'static.txt'
 
     def create_file_system(revision=None):
       '''Creates a MockFileSystem at |revision| by applying that many |updates|
       to it.
       '''
-      mock_file_system = MockFileSystem(
-          TestFileSystem(test_data, relative_to=CHROME_EXTENSIONS))
+      mock_file_system = MockFileSystem(TestFileSystem(test_data))
       updates_for_revision = (
           updates if revision is None else updates[:int(revision)])
       for update in updates_for_revision:
         mock_file_system.Update(update)
       return mock_file_system
 
     delegate = _TestDelegate(create_file_system)
     delegate.SetAppVersion('2-0-8')
 
     file_systems = delegate.file_systems
@@ skipping 67 matching lines @@
     self.assertEqual(AppYamlHelper.GenerateAppYaml('2-1-0'),
                      file_systems[-1].ReadSingle(APP_YAML).Get())
     self.assertEqual('<h1>y</h1> u not update!',
                      file_systems[-1].ReadSingle(storage_html_path).Get())
     self.assertEqual('important content!',
                      file_systems[-1].ReadSingle(static_txt_path).Get())
 
 
 if __name__ == '__main__':
   unittest.main()