[z3-five] Re: r17834 - in z3/Five/branch/Five-1.1: . browser browser/tests

[whit]

sorry...will fix and do the right way.

-w

Philipp von Weitershausen wrote:
> Whit,
> 
> 
>>Modified: z3/Five/branch/Five-1.1/browser/tests/test_pages.py
>>==============================================================================
>>--- z3/Five/branch/Five-1.1/browser/tests/test_pages.py	(original)
>>+++ z3/Five/branch/Five-1.1/browser/tests/test_pages.py	Sat Sep 24 18:26:37 2005
>>@@ -19,17 +19,56 @@
>> if __name__ == '__main__':
>>     execfile(os.path.join(sys.path[0], 'framework.py'))
>> 
>>+from Testing import ZopeTestCase
>>+from zope.app.tests.placelesssetup import setUp, tearDown
>>+
>>+import Products.Five.browser.tests
>>+from Products.Five import zcml
>>+from Products.Five.browser.tests.pages import SimpleView
>>+from Products.Five.testing.simplecontent import manage_addSimpleContent
>>+
>>+from Acquisition import aq_parent, aq_base
>>+
>>+ZopeTestCase.installProduct('Five')
>>+
> 
> 
> installProduct('Five') is a BIG no-no in Five 1.1 and higher. It totally
> messes everything up. The way to do sane test setup in Five is to load
> the ZCML files you want and clean everything up with
> zope.app.placelesssetup.tearDown afterwards.
> 
> I'm CC'ing the Five-list for so that everyone else will know about this too.
> 
> Also, this revision wasn't merged to the trunk... Done now.
> 
> Philipp