From d24b7ceb6d771aefc89d827005f5441d3cd2ca6c Mon Sep 17 00:00:00 2001 From: Matthew Wild Date: Sun, 12 Aug 2018 11:34:05 +0100 Subject: util.pubsub tests: Extend publishing tests to check for correct notification behaviour on subscribe/unsubscribe --- spec/util_pubsub_spec.lua | 14 +++++++++++++- 1 file changed, 13 insertions(+), 1 deletion(-) diff --git a/spec/util_pubsub_spec.lua b/spec/util_pubsub_spec.lua index 0715298c..5ee5a14b 100644 --- a/spec/util_pubsub_spec.lua +++ b/spec/util_pubsub_spec.lua @@ -30,7 +30,10 @@ describe("util.pubsub", function () end); describe("simple publishing", function () - local broadcaster = spy.new(function () end); + local notified; + local broadcaster = spy.new(function (notif_type, node_name, subscribers, item) + notified = subscribers; + end); local service = pubsub.new({ broadcaster = broadcaster; }); @@ -45,6 +48,7 @@ describe("util.pubsub", function () it("publishes an item", function () assert.truthy(service:publish("node", true, "1", "item 1")); + assert.truthy(notified["someone"]); end); it("called the broadcaster", function () @@ -57,6 +61,14 @@ describe("util.pubsub", function () assert.same({ "1", ["1"] = "item 1" }, ret); end); + it("lets someone unsubscribe", function () + assert.truthy(service:remove_subscription("node", true, "someone")); + end); + + it("does not send notifications after subscription is removed", function () + assert.truthy(service:publish("node", true, "1", "item 1")); + assert.is_nil(notified["someone"]); + end); end); describe("#issue1082", function () -- cgit v1.2.3