should call superclass's initialize for aralejs/base#6

aralejs · May 22, 2013 · f0a55bf · f0a55bf
1 parent b63f9b5
commit f0a55bf
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 1 deletion.
diff --git a/src/widget.js b/src/widget.js
@@ -60,7 +60,7 @@ define(function(require, exports, module) {
 
       // 初始化 attrs
       var dataAttrsConfig = this._parseDataAttrsConfig(config)
-      this.initAttrs(config ? $.extend(dataAttrsConfig, config) : dataAttrsConfig)
+      Widget.superclass.initialize.call(this, config ? $.extend(dataAttrsConfig, config) : dataAttrsConfig)
 
       // 初始化 props
       this.parseElement()

diff --git a/tests/widget-spec.js b/tests/widget-spec.js
@@ -791,6 +791,20 @@ define(function(require) {
 
       dom.remove()
     })
+
+    it('attr change callback', function() {
+      var spy = sinon.spy();
+      var Test = Widget.extend({
+        attrs: {
+          a: 1
+        },
+        _onChangeA: spy
+      })
+
+      var test = new Test();
+      test.set('a', 2)
+      expect(spy.calledOnce).to.be.ok()
+    })
   })