Added an option to leave the base storage open when a demo storage is

closed.

src/ZODB/DemoStorage.py

@@ -36,7 +36,9 @@ class DemoStorage(object):
         ZODB.interfaces.IStorageIteration,
         )
 
-    def __init__(self, name=None, base=None, changes=None):
+    def __init__(self, name=None, base=None, changes=None,
+                 keep_base_open=False):
+        self._keep_base_open = keep_base_open
         if base is None:
             base = ZODB.MappingStorage.MappingStorage()
         self.base = base
@@ -71,6 +73,7 @@ class DemoStorage(object):
         self.changes.cleanup()
 
     def close(self):
+        if not self._keep_base_open:
             self.base.close()
         self.changes.close()
         if getattr(self, '_blob_dir', ''):

src/ZODB/DemoStorage.test

@@ -2,10 +2,8 @@
 DemoStorage demo (doctest)
 ==========================
 
-Note that most people will configure the storage through ZConfig.  If
-you are one of those people, you may want to stop here. :)  The
-examples below show you how to use the storage from Python, but they
-also exercise lots of details you might not be interested in.
+DemoStorages provide a way to provide incremental updates to an
+existing, base, storage without updating the storage.
 
 To see how this works, we'll start by creating a base storage and
 puting an object (in addition to the root object) in it:
@@ -122,7 +120,37 @@ Undo methods are simply copied from the changes storage:
     ...  ]
     [True, True, True, True]
 
+
+Normally, when we close a demo storage, the changes and base storages
+are closed:
+
     >>> db.close()
+    >>> base._file.closed
+    True
+    >>> changes._file.closed
+    True
+
+A common use case is to stack multiple DemoStorages, returning to a
+previous state by popping a DemoStorage off the stack.  In this case,
+we want to leave the base storage open:
+
+    >>> base = FileStorage('base.fs', read_only=True)
+    >>> storage = DemoStorage(base=base, keep_base_open=True)
+
+Here, we didn't specify a changes storage.  A MappingStorage was
+automatically created:
+
+    >>> type(storage.changes).__name__
+    'MappingStorage'
+
+Because we specified the keep_base_open option, the base storage is
+left open when we close the DemoStorage:
+
+    >>> storage.close()
+    >>> base._file.closed
+    False
+    >>> storage.changes.opened()
+    False
 
 Blob Support
 ============