Return non-zero status when errors occur.

Clean all was not returning non-zero when an error occur (like when
failing to remove a directory). This patch fix the problem and add a test
for it.

1 files changed, 6 insertions, 3 deletions

diff --git a/src/test.cc b/src/test.cc
index 4dc2f47..719cec3 100644
--- a/src/test.cc
+++ b/src/test.cc
@@ -59,13 +59,16 @@ string VirtualFileSystem::ReadFile(const string& path, string* err) {
   return "";
 }
 
-bool VirtualFileSystem::RemoveFile(const string& path) {
+int VirtualFileSystem::RemoveFile(const string& path) {
+  if (find(directories_made_.begin(), directories_made_.end(), path)
+      != directories_made_.end())
+    return -1;
   FileMap::iterator i = files_.find(path);
   if (i != files_.end()) {
     files_.erase(i);
     files_removed_.insert(path);
-    return true;
+    return 0;
   } else {
-    return false;
+    return 1;
   }
 }