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

Unified Diff: tests/filesystem_mock.py

Issue 6612011: add tests for owners database module in depot_tools (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/depot_tools
Patch Set: make raise_not_found a file-level fn Created 9 years, 10 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
« no previous file with comments | « owners.py ('k') | tests/owners_unittest.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/filesystem_mock.py
diff --git a/tests/filesystem_mock.py b/tests/filesystem_mock.py
new file mode 100644
index 0000000000000000000000000000000000000000..6ce656110942e3027b7237021532a9108f73c348
--- /dev/null
+++ b/tests/filesystem_mock.py
@@ -0,0 +1,70 @@
+#!/usr/bin/python
+# Copyright (c) 2011 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 errno
+import os
+import re
+import StringIO
+
+def _RaiseNotFound(path):
+ raise IOError(errno.ENOENT, path, os.strerror(errno.ENOENT))
+
+
+class MockFileSystem(object):
+ """Stripped-down version of WebKit's webkitpy.common.system.filesystem_mock
+
+ Implements a filesystem-like interface on top of a dict of filenames ->
+ file contents. A file content value of None indicates that the file should
+ not exist (IOError will be raised if it is opened;
+ reading from a missing key raises a KeyError, not an IOError."""
+
+ def __init__(self, files=None):
+ self.files = files or {}
+ self.written_files = {}
+ self._sep = '/'
+
+ @property
+ def sep(self):
+ return self._sep
+
+ def _split(self, path):
+ return path.rsplit(self.sep, 1)
+
+ def dirname(self, path):
+ if not self.sep in path:
+ return ''
+ return self._split(path)[0]
+
+ def exists(self, path):
+ return self.isfile(path) or self.isdir(path)
+
+ def isfile(self, path):
+ return path in self.files and self.files[path] is not None
+
+ def isdir(self, path):
+ if path in self.files:
+ return False
+ if not path.endswith(self.sep):
+ path += self.sep
+
+ # We need to use a copy of the keys here in order to avoid switching
+ # to a different thread and potentially modifying the dict in
+ # mid-iteration.
+ files = self.files.keys()[:]
+ return any(f.startswith(path) for f in files)
+
+ def join(self, *comps):
+ # FIXME: might want tests for this and/or a better comment about how
+ # it works.
+ return re.sub(re.escape(os.path.sep), self.sep, os.path.join(*comps))
+
+ def open_for_reading(self, path):
+ return StringIO.StringIO(self.read_binary_file(path))
+
+ def read_binary_file(self, path):
+ # Intentionally raises KeyError if we don't recognize the path.
+ if self.files[path] is None:
+ _RaiseNotFound(path)
+ return self.files[path]
« no previous file with comments | « owners.py ('k') | tests/owners_unittest.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698