Updated testing suite and fixed compatibility with ember 2.4-LTS and ember 2.8-LTS (broken in last commit)

Author: Levis Pepin

addon/components/ember-chart.js

@@ -11,7 +11,7 @@ export default Ember.Component.extend({
     let data    = this.get('data');
     let type    = this.get('type');
     let options = this.get('options');
-
+		
     let chart = new Chart(context, {
       type: type,
       data: data,
@@ -27,17 +27,20 @@ export default Ember.Component.extend({
 
   didUpdateAttrs() {
     this._super(...arguments);
+
     let chart   = this.get('chart');
     let data    = this.get('data');
     let options = this.get('options');
     let animate = this.get('animate');
-    
-    chart.config.data = data;
-    chart.config.options = options;
-    if (animate) {
-      chart.update();
-    } else {
-      chart.update(0);
-    }
+		
+		if (chart) {
+			chart.config.data = data;
+			chart.config.options = options;
+			if (animate) {
+				chart.update();
+			} else {
+				chart.update(0);
+			}
+		}
   }
 });

tests/unit/components/ember-chart-test.js

@@ -9,7 +9,7 @@ moduleForComponent('ember-chart', 'EmberChartComponent', {
 });
 
 // Test Data
-var ChartTestData = Ember.Object.extend({
+let ChartTestData = Ember.Object.extend({
   pieValue1: 300,
   pieValue2: 50,
   pieValue3: 100,
@@ -166,113 +166,118 @@ var ChartTestData = Ember.Object.extend({
   }),
 });
 
-var testData = ChartTestData.create();
+let testData = ChartTestData.create();
 // Test Data
 
 
 test('it can be a pie chart', function(assert) {
-  var component = this.subject({
+  let component = this.subject({
     type: 'pie',
     data: testData.get('pieData')
   });
 
   this.render();
-  var chart = component.get('chart');
-
+  let chart = component.get('chart');
   assert.equal(chart.config.type, 'pie');
   assert.equal(chart.data.datasets[0].data.length, 3);
 });
 
 test('it can be a line chart', function(assert) {
-  var component = this.subject({
+  let component = this.subject({
     type: 'line',
     data: testData.get('lineData')
   });
 
   this.render();
-  var chart = component.get('chart');
+  let chart = component.get('chart');
 
   assert.equal(chart.config.type, 'line');
   assert.equal(chart.data.datasets.length, 2);
 });
 
 test('it can be a bar chart', function(assert) {
-  var component = this.subject({
+  let component = this.subject({
     type: 'bar',
     data: testData.get('lineData')
   });
 
   this.render();
-  var chart = component.get('chart');
+  let chart = component.get('chart');
 
   assert.equal(chart.config.type, 'bar');
   assert.equal(chart.data.datasets.length, 2);
 });
 
 test('it can be a Radar chart', function(assert) {
-  var component = this.subject({
+  let component = this.subject({
     type: 'radar',
     data: testData.get('lineData')
   });
 
   this.render();
-  var chart = component.get('chart');
+  let chart = component.get('chart');
 
   assert.equal(chart.config.type, 'radar');
   assert.equal(chart.data.datasets.length, 2);
 });
 
 test('it can be a Polar Area chart', function(assert) {
-  var component = this.subject({
+  let component = this.subject({
     type: 'polarArea',
     data: testData.get('pieData')
   });
 
   this.render();
-  var chart = component.get('chart');
+  let chart = component.get('chart');
 
   assert.equal(chart.config.type, 'polarArea');
   assert.equal(chart.data.datasets[0].data.length, 3);
 });
 
 test('it should update pie charts dynamically', function(assert) {
-  var component = this.subject({
+  assert.expect(2);
+  let component = this.subject({
     type: 'pie',
     data: testData.get('pieData')
   });
 
   this.render();
-  var chart = component.get('chart');
+  let chart = component.get('chart');
   assert.equal(chart.data.datasets[0].data[0], 300);
 
   // Update Data
   testData.set('pieValue1', 600);
   component.set('data', testData.get('pieData'));
+	component.didUpdateAttrs();
 
   chart = component.get('chart');
   assert.equal(chart.data.datasets[0].data[0], 600);
 });
 
 test('it should update charts dynamically', function(assert) {
-  var component = this.subject({
+  assert.expect(3);
+  let component = this.subject({
     type: 'line',
     data: testData.get('lineData')
-  });
+	});
 
-  this.render();
-  var chart = component.get('chart');
+	this.render();
+  let chart = component.get('chart');
   assert.equal(chart.data.datasets[0].data[0], 65);
 
   // Update Data
   testData.set('lineValue1', 105);
   component.set('data', testData.get('lineData'));
-
+	component.didUpdateAttrs();
+	
   chart = component.get('chart');
   assert.equal(chart.data.datasets[0].data[0], 105);
 
+
   // Update Labels
   testData.set('labelValue1', 'December');
   component.set('data', testData.get('lineData'));
+	component.didUpdateAttrs();
 
   chart = component.get('chart');
   assert.equal(chart.data.labels[0], 'December');