Comparison

util.pubsub tests: Extend publishing tests to check for correct notification behaviour on subscribe/unsubscribe

comparison

equal deleted inserted replaced

-:822e9c5ff4a4
+:c53663e13b51
 			end);
 		end);
 	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;
 		});
 		it("creates a node", function ()
 			assert.truthy(service:add_subscription("node", true, "someone"));
 		end);
 		it("publishes an item", function ()
 			assert.truthy(service:publish("node", true, "1", "item 1"));
+			assert.truthy(notified["someone"]);
 		end);
 		it("called the broadcaster", function ()
 			assert.spy(broadcaster).was_called();
 		end);
 			local ok, ret = service:get_items("node", true);
 			assert.truthy(ok);
 			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 ()
 		local service = pubsub.new();