Changeset 56:01740c205934 for wokkel

Timestamp:

Apr 22, 2009, 4:53:43 PM (13 years ago)

Author:

Ralph Meijer <ralphm@…>

Branch:

default

Convert:

svn:b33ecbfc-034c-dc11-8662-000475d9059e/trunk@168

Message:

Don't do type interpretation on parsing Data Forms fields.

Location:

wokkel

Files:

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

wokkel/data_form.py

@@ -296 +296 @@
     def _parse_value(field, element):
         value = unicode(element)
-        if field.fieldType == 'boolean':
-            value = value.lower() in ('1', 'true')
-        elif field.fieldType in ('jid-multi', 'jid-single'):
-            value = JID(value)
         field.values.append(value)
 

wokkel/test/test_data_form.py

@@ -230 +230 @@
 
 
-    def test_fromElementValue(self):
-        element = domish.Element((NS_X_DATA, 'field'))
-        element.addElement("value", content="text")
+    def test_fromElementValueTextSingle(self):
+        """
+        Parsed text-single field values should be of type C{unicode}.
+        """
+        element = domish.Element((NS_X_DATA, 'field'))
+        element['type'] = 'text-single'
+        element.addElement('value', content=u'text')
         field = data_form.Field.fromElement(element)
         self.assertEquals('text', field.value)
+
+
+    def test_fromElementValueJID(self):
+        """
+        Parsed jid-single field values should be of type C{unicode}.
+        """
+        element = domish.Element((NS_X_DATA, 'field'))
+        element['type'] = 'jid-single'
+        element.addElement('value', content=u'user@example.org')
+        field = data_form.Field.fromElement(element)
+        self.assertEquals(u'user@example.org', field.value)
+
+    def test_fromElementValueJIDMalformed(self):
+        """
+        Parsed jid-single field values should be of type C{unicode}.
+
+        No validation should be done at this point, so invalid JIDs should
+        also be passed as-is.
+        """
+        element = domish.Element((NS_X_DATA, 'field'))
+        element['type'] = 'jid-single'
+        element.addElement('value', content=u'@@')
+        field = data_form.Field.fromElement(element)
+        self.assertEquals(u'@@', field.value)
+
+
+    def test_fromElementValueBoolean(self):
+        """
+        Parsed boolean field values should be of type C{unicode}.
+        """
+        element = domish.Element((NS_X_DATA, 'field'))
+        element['type'] = 'boolean'
+        element.addElement('value', content=u'false')
+        field = data_form.Field.fromElement(element)
+        self.assertEquals(u'false', field.value)
 
 

wokkel/test/test_pubsub.py

@@ -754 +754 @@
                 "pubsub#deliver_payloads":
                     {"type": "boolean",
-                     "label": "Deliver payloads with event notifications"}
+                     "label": "Deliver payloads with event notifications"},
+                "pubsub#owner":
+                    {"type": "jid-single",
+                     "label": "Owner of the node"}
                 }
 
@@ -763 +766 @@
 
             return defer.succeed({'pubsub#deliver_payloads': '0',
-                                  'pubsub#persist_items': '1'})
+                                  'pubsub#persist_items': '1',
+                                  'pubsub#owner': JID('user@example.org')})
 
         def cb(element):
@@ -776 +780 @@
             field = fields['pubsub#deliver_payloads']
             self.assertEqual('boolean', field.fieldType)
+            field.typeCheck()
             self.assertEqual(False, field.value)
 
@@ -781 +786 @@
             field = fields['pubsub#persist_items']
             self.assertEqual('boolean', field.fieldType)
+            field.typeCheck()
             self.assertEqual(True, field.value)
+
+            self.assertIn('pubsub#owner', fields)
+            field = fields['pubsub#owner']
+            self.assertEqual('jid-single', field.fieldType)
+            field.typeCheck()
+            self.assertEqual(JID('user@example.org'), field.value)
 
         self.service.getConfigurationOptions = getConfigurationOptions