ampersand-state - npm Package Compare versions

Comparing version 4.4.2 to 4.4.3

ampersand-state.js

@@ -527,3 +527,3 @@ /*$AMPERSAND_VERSION*/
         if (desc.setOnce) def.setOnce = true;
-        if (def.required && _.isUndefined(def.default)) def.default = object._getDefaultForType(type);
+        if (def.required && _.isUndefined(def.default) && !def.setOnce) def.default = object._getDefaultForType(type);
         def.test = desc.test;
@@ -530,0 +530,0 @@ def.values = desc.values;

package.json

 {
   "name": "ampersand-state",
   "description": "An observable, extensible state object with derived watchable properties.",
-  "version": "4.4.2",
+  "version": "4.4.3",
   "author": "Henrik Joreteg <[e-mail]>",
@@ -6,0 +6,0 @@ "browserify": {

test/full.js

@@ -1512,1 +1512,23 @@ var tape = require('tape');
 });
+
+test('#114 setOnce allows values to be set once and only once', function (t) {
+    var Model = State.extend({
+        props: {
+            x: {
+                type: 'string',
+                setOnce: true,
+                required: true,
+            }
+        }
+    });
+
+    var model = new Model({ x: 'foo' });
+
+    t.equal(model.x, 'foo');
+
+    t.throws(function () {
+        model.x = 'bar';
+    }, /can only be set once/);
+
+    t.end();
+});

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

120481

increased by0.39%

Lines of code

2942

increased by0.58%

No dependency changes