[Zope3-checkins] CVS: Zope3/src/zope/app/traversing/tests - test_namespacetrversal.py:1.6 test_physicallocationadapters.py:1.9

[Jim Fulton]

Update of /cvs-repository/Zope3/src/zope/app/traversing/tests
In directory cvs.zope.org:/tmp/cvs-serv6583/src/zope/app/traversing/tests

Modified Files:
	test_namespacetrversal.py test_physicallocationadapters.py 
Log Message:
Simplified the strategy for handling "side effect" URL path segments.
Before, we created an extra context wrapper for each segment.  Now we
just keep track of the extra names used in the context wrapper for the
original object.


=== Zope3/src/zope/app/traversing/tests/test_namespacetrversal.py 1.5 => 1.6 ===
--- Zope3/src/zope/app/traversing/tests/test_namespacetrversal.py:1.5	Tue May 27 10:18:27 2003
+++ Zope3/src/zope/app/traversing/tests/test_namespacetrversal.py	Wed May 28 18:15:47 2003
@@ -63,7 +63,7 @@
         traverser = Traverser(c2c1)
         v = traverser.traverse('++skin++ZopeTop', request=TestRequest())
         self.assertEqual(v, c2)
-        self.failUnless(getWrapperContainer(v) is c2c1)
+        self.failUnless(getWrapperContainer(v) is c1)
 
 
 def test_suite():


=== Zope3/src/zope/app/traversing/tests/test_physicallocationadapters.py 1.8 => 1.9 ===
--- Zope3/src/zope/app/traversing/tests/test_physicallocationadapters.py:1.8	Tue May 27 10:18:27 2003
+++ Zope3/src/zope/app/traversing/tests/test_physicallocationadapters.py	Wed May 28 18:15:47 2003
@@ -56,8 +56,9 @@
                        RootPhysicallyLocatable)
 
         root = Root()
-        root = ContextWrapper(root, root, name='.',
-                              side_effect_name="++skin++ZopeTop")
+        root = ContextWrapper(root, None,
+                              side_effect_names=("++skin++ZopeTop", ),
+                              )
         f1 = ContextWrapper(C(), root, name='f1')
         f2 = ContextWrapper(C(),   f1, name='f2')
         f3 = ContextWrapper(C(),   f2, name='f3')
@@ -78,9 +79,10 @@
 
         root = Root()
         c = C()
-        f1 = ContextWrapper(c, root, name='f1')
-        f1 = ContextWrapper(c, f1, name='.',
-                            side_effect_name="++skin++ZopeTop")
+        f1 = ContextWrapper(c, root,
+                            name='f1',
+                            side_effect_names=("++skin++ZopeTop", ),
+                            )
         f2 = ContextWrapper(C(),   f1, name='f2')
         f3 = ContextWrapper(C(),   f2, name='f3')