Updated getSubscriptions to return all subscriptions by topic

mroderick · Feb 23, 2021 · 969484b · 969484b
1 parent fbf975d
commit 969484b
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 2 deletions.
diff --git a/src/pubsub.js b/src/pubsub.js
@@ -271,13 +271,18 @@
      * @function
      * @public
      * @alias getSubscriptions
+     * @return { Array }
     */
     PubSub.getSubscriptions = function getSubscriptions(topic){
         var m;
+        var token;
         var list = [];
         for (m in messages){
-            if (Object.prototype.hasOwnProperty.call(messages, m) && m.indexOf(topic) === 0){
-                list.push(m);
+            if (Object.prototype.hasOwnProperty.call(messages, m) && m.indexOf(topic) === 0) {
+                for (token in messages[m]) {
+                    list.push(messages[m][token]);
+                }
+                break;
             }
         }
         return list;

diff --git a/test/test-getSubscriptions.js b/test/test-getSubscriptions.js
@@ -17,4 +17,17 @@ describe('getSubscriptions method', function () {
         assert.equals(subscriptions,1);
     });
 
+    it('should return all subscriptions', function() {
+        var topic = TestHelper.getUniqueString(),
+            spy1 = sinon.spy(),
+            spy2 = sinon.spy();
+
+        PubSub.subscribe(topic, spy1);
+        PubSub.subscribe(topic, spy2);
+
+        var subscriptions = PubSub.getSubscriptions(topic);
+        assert.equals(subscriptions[0], spy1);
+        assert.equals(subscriptions[1], spy2);
+    });
+
 });