Diff

--- a/spec/util_promise_spec.lua	Thu Oct 25 14:38:00 2018 +0100
+++ b/spec/util_promise_spec.lua	Thu Oct 25 15:24:52 2018 +0100
@@ -142,7 +142,7 @@
 		local r, result;
 		local test_error = {};
 		local p = promise.new(function (resolve, reject)
-			r = resolve;
+			r = reject;
 		end);
 		local cb = spy.new(function () end);
 		local err_cb = spy.new(function (e) result = e end);
@@ -154,7 +154,8 @@
 		r("oh doh");
 		assert.spy(cb).was_called(0);
 		assert.spy(err_cb).was_called(1);
-		assert.equal(test_error, result);
+		assert.spy(err_cb).was_called_with("oh doh");
+		assert.equal("oh doh", result);
 	end);
 
 	it("propagates values down the chain, even when some handlers are not provided", function ()
@@ -175,6 +176,31 @@
 		assert.spy(err_cb).was_called(0);
 	end);
 
+	it("fulfilled promises do not call error handlers and do propagate value", function ()
+		local p = promise.resolve("foo");
+		local cb = spy.new(function () end);
+		local p2 = p:catch(cb);
+		assert.spy(cb).was_called(0);
+
+		local cb2 = spy.new(function () end);
+		local p3 = p2:catch(cb2);
+		assert.spy(cb2).was_called(0);
+	end);
+
+	it("rejected promises do not call fulfilled handlers and do propagate reason", function ()
+		local p = promise.reject("foo");
+		local cb = spy.new(function () end);
+		local p2 = p:next(cb);
+		assert.spy(cb).was_called(0);
+
+		local cb2 = spy.new(function () end);
+		local cb2_err = spy.new(function () end);
+		local p3 = p2:next(cb2, cb2_err);
+		assert.spy(cb2).was_called(0);
+		assert.spy(cb2_err).was_called(1);
+		assert.spy(cb2_err).was_called_with("foo");
+	end);
+
 	describe("race()", function ()
 		it("works with fulfilled promises", function ()
 			local p1, p2 = promise.resolve("yep"), promise.resolve("nope");