Index: chrome/common/extensions/docs/server2/api_models_test.py |
diff --git a/chrome/common/extensions/docs/server2/api_models_test.py b/chrome/common/extensions/docs/server2/api_models_test.py |
index d2353c5e6601e222f06f36356ea7ad771376e0aa..1d2a4c140d14051c1365bda05aa1cd0bd2df8c94 100755 |
--- a/chrome/common/extensions/docs/server2/api_models_test.py |
+++ b/chrome/common/extensions/docs/server2/api_models_test.py |
@@ -8,7 +8,7 @@ import unittest |
from api_models import APIModels |
from compiled_file_system import CompiledFileSystem |
-from extensions_paths import API, EXTENSIONS |
+from extensions_paths import CHROME_API, CHROME_EXTENSIONS |
from features_bundle import FeaturesBundle |
from file_system import FileNotFoundError |
from mock_file_system import MockFileSystem |
@@ -21,7 +21,7 @@ _TEST_DATA = { |
'api': { |
'devtools': { |
'inspected_window.json': ReadFile( |
- API, 'devtools', 'inspected_window.json'), |
+ CHROME_API, 'devtools', 'inspected_window.json'), |
}, |
'_api_features.json': json.dumps({ |
'alarms': {}, |
@@ -36,12 +36,12 @@ _TEST_DATA = { |
}), |
'_manifest_features.json': '{}', |
'_permission_features.json': '{}', |
- 'alarms.idl': ReadFile(API, 'alarms.idl'), |
+ 'alarms.idl': ReadFile(CHROME_API, 'alarms.idl'), |
'declarative_web_request.json': ReadFile( |
- API, 'declarative_web_request.json'), |
+ CHROME_API, 'declarative_web_request.json'), |
'experimental_accessibility.json': ReadFile( |
- API, 'experimental_accessibility.json'), |
- 'page_action.json': ReadFile(API, 'page_action.json'), |
+ CHROME_API, 'experimental_accessibility.json'), |
+ 'page_action.json': ReadFile(CHROME_API, 'page_action.json'), |
}, |
'docs': { |
'templates': { |
@@ -59,7 +59,7 @@ class APIModelsTest(unittest.TestCase): |
object_store_creator = ObjectStoreCreator.ForTest() |
compiled_fs_factory = CompiledFileSystem.Factory(object_store_creator) |
self._mock_file_system = MockFileSystem( |
- TestFileSystem(_TEST_DATA, relative_to=EXTENSIONS)) |
+ TestFileSystem(_TEST_DATA, relative_to=CHROME_EXTENSIONS)) |
features_bundle = FeaturesBundle( |
self._mock_file_system, compiled_fs_factory, object_store_creator) |
self._api_models = APIModels( |
@@ -83,25 +83,29 @@ class APIModelsTest(unittest.TestCase): |
self.assertEqual('devtools.inspectedWindow', |
get_model_name('devtools/inspected_window.json')) |
self.assertEqual('devtools.inspectedWindow', |
- get_model_name(API + 'devtools/inspected_window.json')) |
+ get_model_name(CHROME_API + |
+ 'devtools/inspected_window.json')) |
self.assertEqual('alarms', get_model_name('alarms')) |
self.assertEqual('alarms', get_model_name('alarms.idl')) |
- self.assertEqual('alarms', get_model_name(API + 'alarms.idl')) |
+ self.assertEqual('alarms', get_model_name(CHROME_API + 'alarms.idl')) |
self.assertEqual('declarativeWebRequest', |
get_model_name('declarativeWebRequest')) |
self.assertEqual('declarativeWebRequest', |
get_model_name('declarative_web_request.json')) |
self.assertEqual('declarativeWebRequest', |
- get_model_name(API + 'declarative_web_request.json')) |
+ get_model_name(CHROME_API + |
+ 'declarative_web_request.json')) |
self.assertEqual('experimental.accessibility', |
get_model_name('experimental.accessibility')) |
self.assertEqual('experimental.accessibility', |
get_model_name('experimental_accessibility.json')) |
self.assertEqual('experimental.accessibility', |
- get_model_name(API + 'experimental_accessibility.json')) |
+ get_model_name(CHROME_API + |
+ 'experimental_accessibility.json')) |
self.assertEqual('pageAction', get_model_name('pageAction')) |
self.assertEqual('pageAction', get_model_name('page_action.json')) |
- self.assertEqual('pageAction', get_model_name(API + 'page_action.json')) |
+ self.assertEqual('pageAction', get_model_name(CHROME_API + |
+ 'page_action.json')) |
def testGetNonexistentModel(self): |
self.assertRaises(FileNotFoundError, |
@@ -109,21 +113,26 @@ class APIModelsTest(unittest.TestCase): |
self.assertRaises(FileNotFoundError, |
self._api_models.GetModel('notfound.json').Get) |
self.assertRaises(FileNotFoundError, |
- self._api_models.GetModel(API + 'notfound.json').Get) |
+ self._api_models.GetModel(CHROME_API + |
+ 'notfound.json').Get) |
self.assertRaises(FileNotFoundError, |
- self._api_models.GetModel(API + 'alarms.json').Get) |
+ self._api_models.GetModel(CHROME_API + |
+ 'alarms.json').Get) |
self.assertRaises(FileNotFoundError, |
self._api_models.GetModel('storage').Get) |
self.assertRaises(FileNotFoundError, |
- self._api_models.GetModel(API + 'storage.json').Get) |
+ self._api_models.GetModel(CHROME_API + |
+ 'storage.json').Get) |
self.assertRaises(FileNotFoundError, |
- self._api_models.GetModel(API + 'storage.idl').Get) |
+ self._api_models.GetModel(CHROME_API + |
+ 'storage.idl').Get) |
def testSingleFile(self): |
- # 2 stats (1 for JSON and 1 for IDL), 1 read (for IDL file which existed). |
+ # 4 stats (1 for JSON and 1 for IDL, in both possible API path), |
+ # 1 read (for IDL file which existed). |
future = self._api_models.GetModel('alarms') |
self.assertTrue(*self._mock_file_system.CheckAndReset( |
- read_count=1, stat_count=2)) |
+ read_count=1, stat_count=4)) |
# 1 read-resolve (for the IDL file). |
# |
@@ -134,9 +143,10 @@ class APIModelsTest(unittest.TestCase): |
self.assertTrue(*self._mock_file_system.CheckAndReset( |
read_resolve_count=1)) |
- # 2 stats (1 for JSON and 1 for IDL), no reads (still cached). |
+ # 4 stats (1 for JSON and 1 for IDL, in both possible API paths) |
+ # no reads (still cached). |
future = self._api_models.GetModel('alarms') |
- self.assertTrue(*self._mock_file_system.CheckAndReset(stat_count=2)) |
+ self.assertTrue(*self._mock_file_system.CheckAndReset(stat_count=4)) |
future.Get() |
self.assertTrue(*self._mock_file_system.CheckAndReset()) |