change test order

zopefoundation · Jan 21, 2014 · 42e460c · 42e460c
1 parent 5d13129
commit 42e460c
Showing 1 changed file with 28 additions and 25 deletions.
diff --git a/src/zope/schema/tests/test_fieldproperty.py b/src/zope/schema/tests/test_fieldproperty.py
@@ -558,31 +558,6 @@ class Foo(object):
         foo.__dict__['testing'] = '789'
         self.assertRaises(ValueError, setattr, foo, 'testing', '123')
 
-    def test_field_event(self):
-        from zope.schema import Text
-        from zope.schema._compat import u
-        from zope.event import subscribers
-        from zope.schema.fieldproperty import FieldUpdatedEvent
-        log = []
-        subscribers.append(log.append)
-        self.assertEqual(log, [])
-        field = Text(
-            __name__='testing',
-            description=u('DESCRIPTION'),
-            default=u('DEFAULT'),
-            readonly=True,
-            required=True,
-        )
-        self.assertEqual(len(log), 6)
-        event = log[0]
-        self.assertTrue(isinstance(event, FieldUpdatedEvent))
-        self.assertEqual(event.inst, field)
-        self.assertEqual(event.old_value, 0)
-        self.assertEqual(event.new_value, 0)
-        self.assertEqual(
-            [ev.field.__name__ for ev in log],
-            ['min_length', 'max_length', 'title', 'description', 'required', 'readonly'])
-
     def test_field_event_update(self):
         from zope.schema import Text
         from zope.schema._compat import u
@@ -612,6 +587,34 @@ class Foo(object):
         self.assertEqual(event.old_value, u'Bar')
         self.assertEqual(event.new_value, u'Foo')
 
+    def test_field_event(self):
+        # fieldproperties are everywhere including in field themselfs
+        # so event are triggered
+        from zope.schema import Text
+        from zope.schema._compat import u
+        from zope.event import subscribers
+        from zope.schema.fieldproperty import FieldUpdatedEvent
+        log = []
+        subscribers.append(log.append)
+        self.assertEqual(log, [])
+        field = Text(
+            __name__='testing',
+            description=u('DESCRIPTION'),
+            default=u('DEFAULT'),
+            readonly=True,
+            required=True,
+        )
+        self.assertEqual(len(log), 6)
+        # these are fieldproperties in the field
+        self.assertEqual(
+            [ev.field.__name__ for ev in log],
+            ['min_length', 'max_length', 'title', 'description', 'required', 'readonly'])
+        event = log[0]
+        self.assertTrue(isinstance(event, FieldUpdatedEvent))
+        self.assertEqual(event.inst, field)
+        self.assertEqual(event.old_value, 0)
+        self.assertEqual(event.new_value, 0)
+
 
 def _getSchema():
     from zope.interface import Interface