Changeset

8615:e77b37de482e

util.async: Behaviour change: continue to process queued items after errors
author Matthew Wild <mwild1@gmail.com>
date Sat, 17 Mar 2018 14:54:48 +0000
parents 8614:bfbafeced0c4
children 8616:a15c891c6232
files spec/util_async_spec.lua util/async.lua
diffstat 2 files changed, 27 insertions(+), 0 deletions(-) [+]
line wrap: on
line diff
--- a/spec/util_async_spec.lua	Sat Mar 17 11:47:07 2018 +0000
+++ b/spec/util_async_spec.lua	Sat Mar 17 14:54:48 2018 +0000
@@ -156,6 +156,30 @@
 				assert.equal(r.state, "ready");
 				assert.equal(last_processed_item, "hello again");
 			end);
+
+			it("should continue to process work items", function ()
+				local wait, done, last_item;
+				local runner_func = spy.new(function (item)
+					if item == "error" then
+						error("test error");
+					elseif item == "wait-error" then
+						wait, done = async.waiter();
+						wait();
+						error("test error");
+					end
+					last_item = item;
+				end);
+				local runner = async.runner(runner_func, { error = spy.new(function () end) });
+				runner:enqueue("one");
+				runner:enqueue("error");
+				runner:enqueue("two");
+				runner:run();
+				assert.equal(r.state, "ready");
+				assert.equal(r.state, r.notified_state);
+				assert.spy(runner_func).was.called(3);
+				assert.spy(runner.watchers.error).was.called(1);
+				assert.equal(last_item, "two");
+			end);
 		end);
 	end);
 	describe("#waiter", function()
--- a/util/async.lua	Sat Mar 17 11:47:07 2018 +0000
+++ b/util/async.lua	Sat Mar 17 14:54:48 2018 +0000
@@ -180,6 +180,9 @@
 		local handler = self.watchers[state];
 		if handler then handler(self, err); end
 	end
+	if n > 0 then
+		return self:run();
+	end
 	return true, state, n;
 end