Doc server manifest page generation

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

 #!/usr/bin/env python
 # Copyright (c) 2012 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 json
 import os
 import sys
 import unittest
 
@@ skipping 19 matching lines @@
       'test1.txt': 'test1\n',
       'test2.txt': 'test2\n',
       'test3.txt': 'test3\n',
     }
     file_system = self._CreateSubversionFileSystem()
     self.assertEqual(
         expected,
         file_system.Read(['test1.txt', 'test2.txt', 'test3.txt']).Get())
 
   def testListDir(self):
-    expected = ['dir/']
-    for i in range(7):
-      expected.append('file%d.html' % i)
+    expected = ['dir/'] + ['file%d.html' % i for i in range(7)]
     file_system = self._CreateSubversionFileSystem()
     self.assertEqual(expected, sorted(file_system.ReadSingle('list/')))
 
+  def testListSubDir(self):
+    expected = ['empty.txt'] + ['file%d.html' % i for i in range(3)]
+    file_system = self._CreateSubversionFileSystem()
+    self.assertEqual(expected, sorted(file_system.ReadSingle('list/dir/')))
+
   def testDirStat(self):
     file_system = self._CreateSubversionFileSystem()
     stat_info = file_system.Stat('stat/')
     expected = StatInfo(
       '151113',
-      child_versions=json.loads(self._ReadLocalFile('stat_result.json'))
-    )
-    self.assertEquals(expected, stat_info)
+      child_versions=json.loads(self._ReadLocalFile('stat_result.json')))
+    self.assertEqual(expected, stat_info)
 
   def testFileStat(self):
     file_system = self._CreateSubversionFileSystem()
     stat_info = file_system.Stat('stat/extension_api.h')
-    self.assertEquals(StatInfo('146163'), stat_info)
+    self.assertEqual(StatInfo('146163'), stat_info)
 
   def testRevisions(self):
     # This is a super hacky test. Record the path that was fetched then exit the
+    # test. Compare.

[not at google - send to devlin, 2013/07/20 02:49:03]

What were these changes for? I can't remember.

[jshumway, 2013/07/22 17:16:27]

The first small block 44-48 came form your comment
paths. Could you add a test in there?

This block 64-98 is an error and has been removed. (I had to do some weird stuff
with git and I think this was a byproduct of that.)

+    class ValueErrorFetcher(object):
+      def __init__(self):
+        self.last_fetched = None
+
+      def FetchAsync(self, path):
+        self.last_fetched = path
+        raise ValueError()
+
+      def Fetch(self, path):
+        self.last_fetched = path
+        raise ValueError()
+
+    file_fetcher = ValueErrorFetcher()
+    stat_fetcher = ValueErrorFetcher()
+    svn_path = 'svn:'
+
+    svn_file_system = SubversionFileSystem(file_fetcher,
+                                           stat_fetcher,
+                                           svn_path,
+                                           revision=42)
+
+    self.assertRaises(ValueError, svn_file_system.ReadSingle, 'dir/file')
+    self.assertEqual('dir/file?p=42', file_fetcher.last_fetched)
+    # Stat() will always stat directories.
+    self.assertRaises(ValueError, svn_file_system.Stat, 'dir/file')
+    self.assertEqual('dir/?pathrev=42', stat_fetcher.last_fetched)
+
+    self.assertRaises(ValueError, svn_file_system.ReadSingle, 'dir/')
+    self.assertEqual('dir/?p=42', file_fetcher.last_fetched)
+    self.assertRaises(ValueError, svn_file_system.Stat, 'dir/')
+    self.assertEqual('dir/?pathrev=42', stat_fetcher.last_fetched)
+
+  def testRevisions(self):
+    # This is a super hacky test. Record the path that was fetched then exit the
     # test. Compare.
     class ValueErrorFetcher(object):
       def __init__(self):
         self.last_fetched = None
 
       def FetchAsync(self, path):
         self.last_fetched = path
         raise ValueError()
 
       def Fetch(self, path):
@@ skipping 15 matching lines @@
     self.assertRaises(ValueError, svn_file_system.Stat, 'dir/file')
     self.assertEqual('dir/?pathrev=42', stat_fetcher.last_fetched)
 
     self.assertRaises(ValueError, svn_file_system.ReadSingle, 'dir/')
     self.assertEqual('dir/?p=42', file_fetcher.last_fetched)
     self.assertRaises(ValueError, svn_file_system.Stat, 'dir/')
     self.assertEqual('dir/?pathrev=42', stat_fetcher.last_fetched)
 
 if __name__ == '__main__':
   unittest.main()