Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1014)

Unified Diff: chrome/common/extensions/docs/server2/mock_file_system_test.py

Issue 14247024: Devserver: allow SubversionFileSystem to be pinned to a specific rev on construction (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: comments Created 7 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/common/extensions/docs/server2/mock_file_system_test.py
diff --git a/chrome/common/extensions/docs/server2/mock_file_system_test.py b/chrome/common/extensions/docs/server2/mock_file_system_test.py
index 961128bac730aa608f21c5062d95be286aeb7d13..187462b397d390a906b73c8d901e5843559bf62e 100755
--- a/chrome/common/extensions/docs/server2/mock_file_system_test.py
+++ b/chrome/common/extensions/docs/server2/mock_file_system_test.py
@@ -52,8 +52,9 @@ class MockFileSystemTest(unittest.TestCase):
self.assertTrue(*fs.CheckAndReset())
fs.ReadSingle('404.html')
- fs.Read(['notfound.html', 'apps/'])
+ future = fs.Read(['notfound.html', 'apps/'])
self.assertTrue(*fs.CheckAndReset(read_count=2))
+ self.assertRaises(FileNotFoundError, future.Get)
fs.Stat('404.html')
fs.Stat('404.html')
@@ -68,5 +69,70 @@ class MockFileSystemTest(unittest.TestCase):
self.assertTrue(*fs.CheckAndReset(read_count=1, stat_count=2))
self.assertTrue(*fs.CheckAndReset())
+ def testUpdates(self):
+ fs = MockFileSystem(TestFileSystem(deepcopy(_TEST_DATA)))
+
+ self.assertEqual(StatInfo('0', child_versions={
+ '404.html': '0',
+ 'apps/': '0',
+ 'extensions/': '0'
+ }), fs.Stat('/'))
+ self.assertEqual(StatInfo('0'), fs.Stat('404.html'))
+ self.assertEqual(StatInfo('0', child_versions={
+ 'a11y.html': '0',
+ 'about_apps.html': '0',
+ 'fakedir/': '0',
+ }), fs.Stat('apps/'))
+ self.assertEqual('404.html contents', fs.ReadSingle('404.html'))
+
+ fs.Update({
+ '404.html': 'New version!'
+ })
+
+ self.assertEqual(StatInfo('1', child_versions={
+ '404.html': '1',
+ 'apps/': '0',
+ 'extensions/': '0'
+ }), fs.Stat('/'))
+ self.assertEqual(StatInfo('1'), fs.Stat('404.html'))
+ self.assertEqual(StatInfo('0', child_versions={
+ 'a11y.html': '0',
+ 'about_apps.html': '0',
+ 'fakedir/': '0',
+ }), fs.Stat('apps/'))
+ self.assertEqual('New version!', fs.ReadSingle('404.html'))
+
+ fs.Update({
+ '404.html': 'Newer version!',
+ 'apps': {
+ 'fakedir': {
+ 'file.html': 'yo'
+ }
+ }
+ })
+
+ self.assertEqual(StatInfo('2', child_versions={
+ '404.html': '2',
+ 'apps/': '2',
+ 'extensions/': '0'
+ }), fs.Stat('/'))
+ self.assertEqual(StatInfo('2'), fs.Stat('404.html'))
+ self.assertEqual(StatInfo('2', child_versions={
+ 'a11y.html': '0',
+ 'about_apps.html': '0',
+ 'fakedir/': '2',
+ }), fs.Stat('apps/'))
+ self.assertEqual(StatInfo('0'), fs.Stat('apps/a11y.html'))
+ self.assertEqual(StatInfo('2', child_versions={
+ 'file.html': '2'
+ }), fs.Stat('apps/fakedir/'))
+ self.assertEqual(StatInfo('2'), fs.Stat('apps/fakedir/file.html'))
+ self.assertEqual(StatInfo('0', child_versions={
+ 'activeTab.html': '0',
+ 'alarms.html': '0'
+ }), fs.Stat('extensions/'))
+ self.assertEqual('Newer version!', fs.ReadSingle('404.html'))
+ self.assertEqual('yo', fs.ReadSingle('apps/fakedir/file.html'))
+
if __name__ == '__main__':
unittest.main()

Powered by Google App Engine
This is Rietveld 408576698