strict structs with additional methods should not throw on updating, fix

 gcanti#267 (gcanti#268)

CHANGELOG.md

@@ -12,6 +12,11 @@
 **Note**: Gaps between patch versions are faulty/broken releases.
 **Note**: A feature tagged as Experimental is in a high state of flux, you're at risk of it changing without notice.
 
+# v3.2.16
+
+- **Bug Fix**
+  - strict structs with additional methods should not throw on updating, fix #267 (@gcanti)
+
 # v3.2.15
 
 - **New Feature**

lib/assign.js

@@ -1,8 +1,10 @@
 function assign(x, y) {
   for (var k in y) {
-    x[k] = y[k];
+    if (y.hasOwnProperty(k)) {
+      x[k] = y[k];
+    }
   }
   return x;
 }
 
-module.exports = assign;
+module.exports = assign;

package.json

@@ -1,6 +1,6 @@
 {
   "name": "tcomb",
-  "version": "3.2.15",
+  "version": "3.2.16",
   "description": "Type checking and DDD for JavaScript",
   "main": "index.js",
   "typings": "index.d.ts",

test/update.js

@@ -14,6 +14,20 @@ describe('t.update(instance, patch)', function () {
     y: t.Number
   });
 
+  it('strict structs with additional methods should not throw on updating', function () {
+    var Rectangle = t.struct({
+      width: t.Number,
+      height: t.Number
+    }, {strict: true});
+
+    Rectangle.prototype.getArea = function () {
+      return this.width * this.height;
+    };
+
+    var r = Rectangle({width: 10, height: 10});
+    assert.equal(Rectangle.update(r, {width: {$set: 20}}).width, 20);
+  });
+
   it('should throw if patch is invalid', function () {
     throwsWithMessage(function () {
       t.update({});