From aeebf0de906e7030282d59734ad30d69548cb521 Mon Sep 17 00:00:00 2001 From: Evan You <yyx990803@gmail.com> Date: Wed, 7 Mar 2018 15:00:40 -0500 Subject: [PATCH] test: add test for observer lazy getter invocation --- test/unit/modules/observer/observer.spec.js | 14 ++++++++++++++ 1 file changed, 14 insertions(+) diff --git a/test/unit/modules/observer/observer.spec.js b/test/unit/modules/observer/observer.spec.js index 0cda76960b1..3ed7a3a48c3 100644 --- a/test/unit/modules/observer/observer.spec.js +++ b/test/unit/modules/observer/observer.spec.js @@ -355,4 +355,18 @@ describe('Observer', () => { expect(obj.__ob__ instanceof Observer).toBe(true) }) }) + + it('should lazy invoke existing getters', () => { + const obj = {} + let called = false + Object.defineProperty(obj, 'getterProp', { + enumerable: true, + get: () => { + called = true + return 'some value' + } + }) + observe(obj) + expect(called).toBe(false) + }) })