Changeset 80:3ee3e922549d

Timestamp:

Jan 5, 2010, 2:55:40 PM (12 years ago)

Author:

Ralph Meijer <ralphm@…>

Branch:

default

Message:

Move findForm to wokkel.data_form

Location:

wokkel

Files:

• data_form.py (modified) (1 diff)
• pubsub.py (modified) (4 diffs)
• test/test_data_form.py (modified) (1 diff)

wokkel/data_form.py

@@ -654 +654 @@
         for field in filtered:
             self.removeField(field)
+
+
+
+def findForm(element, formNamespace):
+    """
+    Find a Data Form.
+
+    Look for an element that represents a Data Form with the specified
+    form namespace as a child element of the given element.
+    """
+    if not element:
+        return None
+
+    for child in element.elements():
+        if (child.uri, child.name) == ((NS_X_DATA, 'x')):
+            form = Form.fromElement(child)
+
+            if (form.formNamespace == formNamespace or
+                not form.formNamespace and form.formType=='cancel'):
+                return form
+
+    return None

wokkel/pubsub.py

@@ -253 +253 @@
 
 
-    @staticmethod
-    def _findForm(element, formNamespace):
-        """
-        Find a Data Form.
-
-        Look for an element that represents a Data Form with the specified
-        form namespace as a child element of the given element.
-        """
-        if not element:
-            return None
-
-        form = None
-        for child in element.elements():
-            try:
-                form = data_form.Form.fromElement(child)
-            except data_form.Error:
-                continue
-
-            if form.formNamespace != NS_PUBSUB_NODE_CONFIG:
-                continue
-
-        return form
-
-
     def _parse_node(self, verbElement):
         """
@@ -367 +343 @@
         Parse node type out of a request for the default node configuration.
         """
-        form = PubSubRequest._findForm(verbElement, NS_PUBSUB_NODE_CONFIG)
+        form = data_form.findForm(verbElement, NS_PUBSUB_NODE_CONFIG)
         if form and form.formType == 'submit':
             values = form.getValues()
@@ -379 +355 @@
         Parse options out of a request for setting the node configuration.
         """
-        form = PubSubRequest._findForm(verbElement, NS_PUBSUB_NODE_CONFIG)
+        form = data_form.findForm(verbElement, NS_PUBSUB_NODE_CONFIG)
         if form:
             if form.formType == 'submit':
@@ -438 +414 @@
 
     def _parse_options(self, verbElement):
-        form = PubSubRequest._findForm(verbElement, NS_PUBSUB_SUBSCRIBE_OPTIONS)
+        form = data_form.findForm(verbElement, NS_PUBSUB_SUBSCRIBE_OPTIONS)
         if form:
             if form.formType == 'submit':

wokkel/test/test_data_form.py

@@ -1152 +1152 @@
 
 
+
+class FindFormTest(unittest.TestCase):
+    """
+    Tests for L{data_form.findForm}.
+    """
+
+    def test_findForm(self):
+        element = domish.Element((None, 'test'))
+        theForm = data_form.Form('submit', formNamespace='myns')
+        element.addChild(theForm.toElement())
+        form = data_form.findForm(element, 'myns')
+        self.assertEqual('myns', form.formNamespace)
+
+
+    def test_noFormType(self):
+        element = domish.Element((None, 'test'))
+        otherForm = data_form.Form('submit')
+        element.addChild(otherForm.toElement())
+        form = data_form.findForm(element, 'myns')
+        self.assertIdentical(None, form)
+
+
+    def test_noFormTypeCancel(self):
+        """
+        Cancelled forms don't have a FORM_TYPE field, the first is returned.
+        """
+        element = domish.Element((None, 'test'))
+        cancelledForm = data_form.Form('cancel')
+        element.addChild(cancelledForm.toElement())
+        form = data_form.findForm(element, 'myns')
+        self.assertEqual('cancel', form.formType)
+
+
+    def test_otherFormType(self):
+        """
+        Forms with other FORM_TYPEs are ignored.
+        """
+        element = domish.Element((None, 'test'))
+        otherForm = data_form.Form('submit', formNamespace='otherns')
+        element.addChild(otherForm.toElement())
+        form = data_form.findForm(element, 'myns')
+        self.assertIdentical(None, form)
+
+
+    def test_otherFormTypeCancel(self):
+        """
+        Cancelled forms with another FORM_TYPE are ignored.
+        """
+        element = domish.Element((None, 'test'))
+        cancelledForm = data_form.Form('cancel', formNamespace='otherns')
+        element.addChild(cancelledForm.toElement())
+        form = data_form.findForm(element, 'myns')
+        self.assertIdentical(None, form)
+
+
+    def test_noElement(self):
+        """
+        When None is passed as element, None is returned.
+        """
+        element = None
+        form = data_form.findForm(element, 'myns')
+        self.assertIdentical(None, form)
+
+
+    def test_noForm(self):
+        """
+        When no child element is a form, None is returned.
+        """
+        element = domish.Element((None, 'test'))
+        form = data_form.findForm(element, 'myns')
+        self.assertIdentical(None, form)
     def test_typeCheckNoFieldDefs(self):
         """