better copying behaviour in remote.CacheFS

fs/remote.py

@@ -117,10 +117,10 @@ def cached(func):
                 raise
             else:
                 self._cache_set(path,func.__name__,args,kwds,(True,res))
-                return res
+                return copy.copy(res)
         else:
             if not success:
-                raise copy.copy(result)
+                raise result
             else:
                 return copy.copy(result)
     return wrapper
@@ -180,26 +180,9 @@ class CacheFS(WrapFS):
                 if cache is None:
                     return 
             # Adjust cached 'listdir' for parent directory.
-            # Currently we only do this for argment-less listdir() calls.
-            # There's probably a better way to manage this...
+            # TODO: account for whether it was added, removed, or unmoved
             cache[""].pop("getinfo",None)
             cache[""].pop("getsize",None)
-            if added:
-                for (key,val) in list(cache[""].get("listdir",{}).items()):
-                    if key == ((),()):
-                        val[1][1].append(basename(path))
-                    else:
-                        cache[""].pop(key,None)
-            elif removed:
-                for (key,val) in list(cache[""].get("listdir",{}).items()):
-                    if key == ((),()):
-                        try:
-                            val[1][1].remove(basename(path))
-                        except ValueError:
-                            pass
-                    else:
-                        cache[""].pop(key,None)
-            elif not unmoved:
             cache[""].pop("listdir",None)
         # Clear all cached info for the path itself.
         cache[names[-1]] = {"":{}}

fs/tests/test_s3fs.py

@@ -17,7 +17,7 @@ from fs import s3fs
 class TestS3FS(unittest.TestCase,FSTestCases,ThreadingTestCases):
 
     #  Disable the tests by default
-    #__test__ = False
+    __test__ = False
 
     bucket = "test-s3fs.rfk.id.au"
 

fs/wrapfs.py

@@ -143,6 +143,7 @@ class WrapFS(FS):
         entries = []
         for e in self.wrapped_fs.listdir(self._encode(path),**kwds):
             if info:
+                e = e.copy()
                 e["name"] = self._decode(e["name"])
                 if wildcard is not None and not fnmatch(e["name"],wildcard):
                     continue