fix(setup): Allow retuning frozen objects on the setup (#366)

vuejs · Jun 10, 2020 · bca3a69 · bca3a69
1 parent 34c0fad
commit bca3a69
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/src/reactivity/reactive.ts b/src/reactivity/reactive.ts
@@ -238,7 +238,7 @@ export function reactive<T extends object>(obj: T): UnwrapRef<T> {
  * Make sure obj can't be a reactive
  */
 export function markRaw<T extends object>(obj: T): T {
-  if (!isPlainObject(obj)) {
+  if (!isPlainObject(obj) || !Object.isExtensible(obj)) {
     return obj;
   }
 

diff --git a/test/setup.spec.js b/test/setup.spec.js
@@ -308,6 +308,19 @@ describe('setup', () => {
     expect(warn).not.toHaveBeenCalled();
   });
 
+  it('Should allow to return Object.freeze', () => {
+    const vm = new Vue({
+      template: `<div>{{foo.bar}}</div>`,
+      setup() {
+        const foo = Object.freeze({ bar: 'baz' });
+        return {
+          foo,
+        };
+      },
+    }).$mount();
+    expect(vm.$el.textContent).toBe('baz');
+  });
+
   it('this should be undefined', () => {
     const vm = new Vue({
       template: '<div></div>',