Login | Register For Free | Help
Search for: (Advanced)

Mailing List Archive: Zope: CMF

Re: SVN: Products.GenericSetup/branches/1.3/Products/GenericSetup/ Merge 84270 from trunk: During object manager imports do not throw an

 

 

Zope cmf RSS feed   Index | Next | Previous | View Threaded


tseaver at palladion

Feb 27, 2008, 11:39 AM

Post #1 of 4 (363 views)
Permalink
Re: SVN: Products.GenericSetup/branches/1.3/Products/GenericSetup/ Merge 84270 from trunk: During object manager imports do not throw an

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Maurits van Rees wrote:
> Log message for revision 84271:
> Merge 84270 from trunk: During object manager imports do not throw an
> error when trying to remove an object that was already removed.

<snip>

> +class ObjectManagerHelpersTests(unittest.TestCase):
> +
> + def _getTargetClass(self):
> + from Products.GenericSetup.utils import ObjectManagerHelpers
> +
> + return ObjectManagerHelpers
> +
> + def _makeOne(self, *args, **kw):
> + from Products.GenericSetup.utils import NodeAdapterBase
> +
> + class Foo(self._getTargetClass(), NodeAdapterBase):
> +
> + pass
> +
> + return Foo(*args, **kw)
> +
> + def setUp(self):
> + from OFS.ObjectManager import ObjectManager
> +
> + obj = ObjectManager('obj')
> + self.helpers = self._makeOne(obj, DummySetupEnviron())
> +
> + def test__initObjects(self):
> + obj = self.helpers.context
> + self.failIf('history' in obj.objectIds())
> +
> + # Add object
> + node = parseString(_ADD_IMPORT).documentElement
> + self.helpers._initObjects(node)
> + self.failUnless('history' in obj.objectIds())
> +
> + # Remove it again
> + node = parseString(_REMOVE_IMPORT).documentElement
> + self.helpers._initObjects(node)
> + self.failIf('history' in obj.objectIds())
> +
> + # Removing it a second time should not throw an
> + # AttributeError.
> + node = parseString(_REMOVE_IMPORT).documentElement
> + self.helpers._initObjects(node)
> + self.failIf('history' in obj.objectIds())
> +
> +

Marits, the new test you wrote here is failing when run on the Zope 2.11
branch or the trunk[1]: can you please diagnose why, and remediate?

[1] http://mail.zope.org/pipermail/cmf-tests/2008-February/008099.html



Tres.
- --
===================================================================
Tres Seaver +1 540-429-0999 tseaver[at]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

iD8DBQFHxbxm+gerLs4ltQ4RAiWmAKDLO1x0Ei1bwQ0XgNBd6lQ4i63SJACgsiim
vMtCQ80/yVt+hP47iwtIQAU=
=rOoi
-----END PGP SIGNATURE-----

_______________________________________________
Zope-CMF maillist - Zope-CMF[at]lists.zope.org
http://mail.zope.org/mailman/listinfo/zope-cmf

See http://collector.zope.org/CMF for bug reports and feature requests


y.2008 at wcm-solutions

Feb 27, 2008, 12:26 PM

Post #2 of 4 (340 views)
Permalink
Re: SVN: Products.GenericSetup/branches/1.3/Products/GenericSetup/ Merge 84270 from trunk: During object manager imports do not throw an [In reply to]

Tres Seaver wrote:
> Marits, the new test you wrote here is failing when run on the Zope 2.11
> branch or the trunk[1]: can you please diagnose why, and remediate?
>
> [1] http://mail.zope.org/pipermail/cmf-tests/2008-February/008099.html

Fixed: http://svn.zope.org/?rev=84353&view=rev

Cheers, Yuppie

_______________________________________________
Zope-CMF maillist - Zope-CMF[at]lists.zope.org
http://mail.zope.org/mailman/listinfo/zope-cmf

See http://collector.zope.org/CMF for bug reports and feature requests


tseaver at palladion

Feb 27, 2008, 12:38 PM

Post #3 of 4 (340 views)
Permalink
Re: SVN: Products.GenericSetup/branches/1.3/Products/GenericSetup/ Merge 84270 from trunk: During object manager imports do not throw an [In reply to]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

yuppie wrote:
> Tres Seaver wrote:
>> Marits, the new test you wrote here is failing when run on the Zope 2.11
>> branch or the trunk[1]: can you please diagnose why, and remediate?
>>
>> [1] http://mail.zope.org/pipermail/cmf-tests/2008-February/008099.html
>
> Fixed: http://svn.zope.org/?rev=84353&view=rev
>
> Cheers, Yuppie

Thanks for the fix. I guess I'm worried at seeing more
"integration-style" tests folded into the product in place of unit
tests, but perhaps that is inevitable here.


Tres.
- --
===================================================================
Tres Seaver +1 540-429-0999 tseaver[at]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

iD8DBQFHxcpS+gerLs4ltQ4RAuoNAKCsu+JNEjFo8A8gxwl4Pdm7f74uOACfYBdS
KWX3aEJOY/qCZE3Cfse1lHU=
=pbCl
-----END PGP SIGNATURE-----

_______________________________________________
Zope-CMF maillist - Zope-CMF[at]lists.zope.org
http://mail.zope.org/mailman/listinfo/zope-cmf

See http://collector.zope.org/CMF for bug reports and feature requests


m.van.rees at zestsoftware

Feb 27, 2008, 1:49 PM

Post #4 of 4 (340 views)
Permalink
Re: SVN: Products.GenericSetup/branches/1.3/Products/GenericSetup/ Merge 84270 from trunk: During object manager imports do not throw an [In reply to]

yuppie, on 2008-02-27:
> Tres Seaver wrote:
>> Marits, the new test you wrote here is failing when run on the Zope 2.11
>> branch or the trunk[1]: can you please diagnose why, and remediate?
>>
>> [1] http://mail.zope.org/pipermail/cmf-tests/2008-February/008099.html
>
> Fixed: http://svn.zope.org/?rev=84353&view=rev
>
> Cheers, Yuppie

Ah, thanks!

I only ran the tests in a plone buildout:

http://svn.plone.org/svn/plone/ploneout/trunk
and
http://svn.plone.org/svn/plone/ploneout/branches/3.0

and they passed there.

I have now managed to setup a buildout with CMF trunk and Zope 2.11
and there it indeed fails with my code and passes with yuppies code.

--
Maurits van Rees | http://maurits.vanrees.org/
Work | http://zestsoftware.nl/
"This is your day, don't let them take it away." [Barlow Girl]

_______________________________________________
Zope-CMF maillist - Zope-CMF[at]lists.zope.org
http://mail.zope.org/mailman/listinfo/zope-cmf

See http://collector.zope.org/CMF for bug reports and feature requests

Zope cmf RSS feed   Index | Next | Previous | View Threaded
 
 


Interested in having your list archived? Contact lists@gossamer-threads.com
 
  Web Applications & Managed Hosting Powered by Gossamer Threads Inc.