Handle non-reactive nulls (#231) (#241)

* Fixes #231

* Impl feedback

* Handle undefined as well

* Add unit test for undefined values

* Update setup.spec.js

* Apply suggestions from code review

* Update test/setup.spec.js

Co-authored-by: Eduardo San Martin Morote <[email protected]>
Co-authored-by: Anthony Fu <[email protected]>

Author: 3 people

src/utils.ts

@@ -36,9 +36,8 @@ export function def(obj: Object, key: string, val: any, enumerable?: boolean) {
   });
 }
 
-const hasOwnProperty = Object.prototype.hasOwnProperty;
-export function hasOwn(obj: Object | any[], key: string): boolean {
-  return hasOwnProperty.call(obj, key);
+export function hasOwn(obj: Object, key: string): boolean {
+  return Object.hasOwnProperty.call(obj, key);
 }
 
 export function assert(condition: any, msg: string) {

test/setup.spec.js

@@ -20,6 +20,30 @@ describe('setup', () => {
     expect(vm.a).toBe(1);
   });
 
+  it('should work with non reactive null', () => {
+    const vm = new Vue({
+      setup() {
+        return {
+          a: null,
+        };
+      },
+    }).$mount();
+    expect(vm.a).toBe(null);
+  });
+
+  it('should work with non reactive undefined', () => {
+    const vm = new Vue({
+      setup() {
+        return {
+          a: undefined,
+          b: 'foobar',
+        };
+      },
+    }).$mount();
+    expect(vm.a).toBe(undefined);
+    expect(vm.b).toBe('foobar');
+  });
+
   it('should be overrided by data option of plain object', () => {
     const vm = new Vue({
       setup() {