zopefoundation / ZODB / 23421885197

84%

master: 84%

Pull #421

github

Add class_factory parameter to DB and ZConfig class-factory option

DB.__init__() opens a connection to verify the root object exists,
and that connection is returned to the pool for future reuse.
Connections capture the classFactory reference at creation time
(in Connection.__init__ via ObjectReader), so setting db.classFactory
after DB.__init__() does not apply to connections already in the pool.

This is a problem for Zope, which overrides DB.classFactory after
construction to provide graceful handling of broken/uninstalled objects.
The first connection handed out after startup may silently use the
wrong class factory.

Add a class_factory parameter to DB.__init__() that is set before any
connection is opened, and a corresponding class-factory option to the
ZConfig database configuration schema. This allows frameworks like
Zope to pass the class factory at construction time.

2444 of 3546 branches covered (68.92%)

14 of 17 new or added lines in 4 files covered. (82.35%)

13367 of 15955 relevant lines covered (83.78%)

0.84 hits per line