-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 yuppie wrote:
>> - The TraversingEventZCMLLayer now also loads implements.zcml, as the >> correct behaviour of both the catalog and the adapter class is required >> for the folder tests to pass. >> >> - The CatalogTool tests set up the adapter at the moment, as a lot of >> the catalog tests require the adapter to work properly. This is done in >> the _makeContent method as it applied to most tests that used the dummy >> content. However, I think it belongs somewhere else, but I wasn't sure >> whether that place was a layer, a setup method or somewhere else. Any >> suggestions? > > I agree it belongs somewhere else. Maybe a registerWrapper method. But > can't we make the adapter lookup in catalog_object optional and wouldn't > that make test setups simpler? Agreed. I had expected that the catalog would do a queryAdapter, and default to the existing wrapper class if not found. Tres. - -- =================================================================== Tres Seaver +1 540-429-0999 tsea...@palladion.com Palladion Software "Excellence by Design" http://palladion.com -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.6 (GNU/Linux) Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org iD8DBQFJv+To+gerLs4ltQ4RAvt/AJ9D75eOsZh7FINca9MJsnTU26998gCghu8C TaiqRzwLe2WDoHIqaD7qGhs= =kd1W -----END PGP SIGNATURE----- _______________________________________________ Zope-CMF maillist - Zope-CMF@lists.zope.org http://mail.zope.org/mailman/listinfo/zope-cmf See https://bugs.launchpad.net/zope-cmf/ for bug reports and feature requests
