Add support for disabled Ember prototype extensions

Author: soulim

addon/components/datepicker-support.js

@@ -3,7 +3,7 @@ import Ember from 'ember';
 export default Ember.Mixin.create({
   value: null,
 
-  setupBootstrapDatepicker: function() {
+  setupBootstrapDatepicker: Ember.on('didInsertElement', function() {
     var self = this;
 
     this.$().
@@ -29,18 +29,22 @@ export default Ember.Mixin.create({
       }).
       on('changeDate', function(event) {
         Ember.run(function() {
-          self.didChangeDate(event);
+          self._didChangeDate(event);
         });
       });
 
     this._updateDatepicker();
-  }.on('didInsertElement'),
+  }),
 
-  teardownBootstrapDatepicker: function() {
+  teardownBootstrapDatepicker: Ember.on('willDestroyElement', function() {
     this.$().datepicker('remove');
-  }.on('willDestroyElement'),
+  }),
 
-  didChangeDate: function(event) {
+  didChangeValue: Ember.observer('value', function() {
+    this._updateDatepicker();
+  }),
+
+  _didChangeDate: function(event) {
     var value = null;
 
     if (event.date) {
@@ -54,10 +58,6 @@ export default Ember.Mixin.create({
     this.set('value', value);
   },
 
-  didChangeValue: function() {
-    this._updateDatepicker();
-  }.observes('value'),
-
   _updateDatepicker: function() {
     var self = this,
         element = this.$(),

tests/integration/bootstrap-datepicker-test.js

@@ -13,6 +13,6 @@ moduleForComponent('bootstrap-datepicker', 'bootstrap-datepicker component', {
   }
 });
 
-test('...', function() {
-  expect(0);
+test('...', function(assert) {
+  assert.expect(0);
 });

tests/unit/components/bootstrap-datepicker-test.js

@@ -3,65 +3,65 @@ import Ember from 'ember';
 
 moduleForComponent('bootstrap-datepicker', 'bootstrap-datepicker component');
 
-test('should be an input tag', function() {
-  expect(1);
+test('should be an input tag', function(assert) {
+  assert.expect(1);
 
-  equal('INPUT', this.$().prop('tagName'));
+  assert.equal('INPUT', this.$().prop('tagName'));
 });
 
-test('should display empty input field when no date is set', function(){
-  expect(1);
+test('should display empty input field when no date is set', function(assert) {
+  assert.expect(1);
 
   var component = this.subject({
     value: null
   });
 
-  equal(this.$().val(), '');
+  assert.equal(this.$().val(), '');
 });
 
 
-test('should display date with default format when no format is set', function(){
-  expect(1);
+test('should display date with default format when no format is set', function(assert) {
+  assert.expect(1);
 
   var component = this.subject({
     value: new Date(2014, 11, 31)
   });
 
-  equal(this.$().val(), '12/31/2014');
+  assert.equal(this.$().val(), '12/31/2014');
 });
 
-test('should display date with custom format when format is set', function(){
-  expect(1);
+test('should display date with custom format when format is set', function(assert) {
+  assert.expect(1);
 
   var component = this.subject({
     value: new Date(2014, 11, 31),
     format: 'dd.M.yy'
   });
 
-  equal(this.$().val(), '31.Dec.14');
+  assert.equal(this.$().val(), '31.Dec.14');
 });
 
-test('should set dates provided by value (multidate, default multidateSeparator)', function(){
-  expect(2);
+test('should set dates provided by value (multidate, default multidateSeparator)', function(assert) {
+  assert.expect(2);
 
   var component = this.subject({
     value: [new Date(2015, 0, 13), new Date(2015, 0, 7), new Date(2015, 0, 15)],
     multidate: true
   });
 
-  equal(this.$().val(), '01/13/2015,01/07/2015,01/15/2015', 'should set value as input field value');
-  equal(this.$().datepicker('getDates').length, 3, 'should set internal datepicker dates by value');
+  assert.equal(this.$().val(), '01/13/2015,01/07/2015,01/15/2015', 'should set value as input field value');
+  assert.equal(this.$().datepicker('getDates').length, 3, 'should set internal datepicker dates by value');
 });
 
-test('should set dates provided by value (multidate, multidateSeparator provided)', function(){
-  expect(2);
+test('should set dates provided by value (multidate, multidateSeparator provided)', function(assert) {
+  assert.expect(2);
 
   var component = this.subject({
     value: [new Date(2015, 0, 13), new Date(2015, 0, 7), new Date(2015, 0, 15)],
     multidate: true,
     multidateSeparator: ';'
   });
 
-  equal(this.$().val(), '01/13/2015;01/07/2015;01/15/2015', 'should set value as input field value using multidate separator');
-  equal(this.$().datepicker('getDates').length, 3, 'should set internal datepicker dates by value');
+  assert.equal(this.$().val(), '01/13/2015;01/07/2015;01/15/2015', 'should set value as input field value using multidate separator');
+  assert.equal(this.$().datepicker('getDates').length, 3, 'should set internal datepicker dates by value');
 });