Revise chaining of put and del

So chaining put and del messages is more useful. Fixes kriskowal#84.
tebriel · Aug 23, 2012 · b160c43 · b160c43
1 parent 795fd1c
commit b160c43
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/q.js b/q.js
@@ -789,10 +789,12 @@ function resolve(object) {
             return object[name];
         },
         "put": function (name, value) {
-            return object[name] = value;
+            object[name] = value;
+            return object;
         },
         "del": function (name) {
-            return delete object[name];
+            delete object[name];
+            return object;
         },
         "post": function (name, value) {
             return object[name].apply(object, value);

diff --git a/spec/q-spec.js b/spec/q-spec.js
@@ -481,7 +481,7 @@ describe("promises for objects", function () {
             return Q.resolve(object)
             .put('a', 1)
             .then(function (result) {
-                expect(result).toBe(1); // not supported. may change in future version.
+                expect(result).toBe(object);
                 expect(object.a).toBe(1);
             });
         });
@@ -509,7 +509,7 @@ describe("promises for objects", function () {
             .del('a')
             .then(function (result) {
                 expect('a' in object).toBe(false);
-                expect(result).toBe(true); // not supported, may change
+                expect(result).toBe(object);
             }, function (exception) {
                 expect("up").toBe("down");
             });