source: ralphm-patches/pubsub-disco-info-42.patch @ 7:0fbac5c2e97d

wokkel/pubsub.py

@@ -567 +567 @@
                 for metaDatum in metaData:
                     form.addField(data_form.Field.fromDict(metaDatum))
 
-                info.append(form.toElement())
+                info.append(form)
 
         d = self.getNodeInfo(requestor, target, nodeIdentifier or '')
         d.addCallback(toInfo)

wokkel/test/test_pubsub.py

@@ -13 +13 @@
 from twisted.words.protocols.jabber import error
 from twisted.words.protocols.jabber.jid import JID
 
-from wokkel import data_form, iwokkel, pubsub, shim
+from wokkel import data_form, disco, iwokkel, pubsub, shim
 from wokkel.test.helpers import TestableRequestHandlerMixin, XmlStreamStub
 
 try:
@@ -26 +26 @@
 NS_PUBSUB_ERRORS = 'http://jabber.org/protocol/pubsub#errors'
 NS_PUBSUB_EVENT = 'http://jabber.org/protocol/pubsub#event'
 NS_PUBSUB_OWNER = 'http://jabber.org/protocol/pubsub#owner'
+NS_PUBSUB_META_DATA = 'http://jabber.org/protocol/pubsub#meta-data'
 
 def calledAsync(fn):
     """
@@ -524 +525 @@
         return d
 
 
+    def test_getDiscoInfoNodeType(self):
+        """
+        Test getDiscoInfo with node type.
+        """
+        def cb(info):
+            discoInfo = disco.DiscoInfo()
+            for item in info:
+                discoInfo.append(item)
+            self.assertIn(('pubsub', 'collection'), discoInfo.identities)
+
+        def getNodeInfo(requestor, target, nodeIdentifier):
+            return defer.succeed({'type': 'collection',
+                                  'meta-data': {}})
+
+        self.service.getNodeInfo = getNodeInfo
+        d = self.service.getDiscoInfo(JID('user@example.org/home'),
+                                      JID('pubsub.example.org'), '')
+        d.addCallback(cb)
+        return d
+
+
+    def test_getDiscoInfoMetaData(self):
+        """
+        Test getDiscoInfo with returned meta data.
+        """
+        def cb(info):
+            discoInfo = disco.DiscoInfo()
+            for item in info:
+                discoInfo.append(item)
+
+            self.assertIn(('pubsub', 'leaf'), discoInfo.identities)
+            self.assertIn(NS_PUBSUB_META_DATA, discoInfo.extensions)
+            form = discoInfo.extensions[NS_PUBSUB_META_DATA]
+            self.assertIn('pubsub#node_type', form.fields)
+
+        def getNodeInfo(requestor, target, nodeIdentifier):
+            metaData = [{'var': 'pubsub#persist_items',
+                         'label': 'Persist items to storage',
+                         'value': True}]
+            return defer.succeed({'type': 'leaf', 'meta-data': metaData})
+
+        self.service.getNodeInfo = getNodeInfo
+        d = self.service.getDiscoInfo(JID('user@example.org/home'),
+                                      JID('pubsub.example.org'), '')
+        d.addCallback(cb)
+        return d
+
+
     def test_onPublishNoNode(self):
         """
         The root node is always a collection, publishing is a bad request.