Fix issue regarding piecharts with only single slice

Fixed failing tests. This fixes #4

Author: timeu

.gitignore

@@ -1 +1 @@
-.bowerrc
+bower_components

bower.json

@@ -1,6 +1,6 @@
 {
   "name": "svg-piechart",
-  "version": "1.0.1",
+  "version": "1.0.2",
   "authors": ["Uemit Seren <uemit.seren@gmail.com>"],
   "description": "Element to draw a piechart on a SVG.",
   "keywords": "svg, piechart, polymer, web-components",

svg-piechart.html

@@ -167,15 +167,18 @@
 
         startAngle = endAngle;
         endAngle = startAngle + angle;
-
-        arcs.push({
+        var arc = {
           x1: 180 + 180*Math.cos(Math.PI*startAngle/180),
           y1: 180 + 180*Math.sin(Math.PI*startAngle/180),
           x2: 180 + 180*Math.cos(Math.PI*endAngle/180),
           y2: 180 + 180*Math.sin(Math.PI*endAngle/180),
           largeArcFlag: largeArcFlag,
           color: colors[i]
-        });
+        };
+        if (arc.x2 > 179.9 && arc.y2 === 0 && arc.largeArcFlag === 1) {
+          arc.x2 = 179.99; 
+        }
+        arcs.push(arc);
       }
 
       return arcs;

test/basic-test.html

@@ -38,7 +38,7 @@
 
         test('defaults', function() {
           assert.equal(myEl.colors.length,0);
-          assert.equal(myEl.size,50);
+          assert.equal(myEl.size,150);
           assert.equal(myEl.data.length,0);
         });
 
@@ -49,14 +49,19 @@
           assert.equal(myEl._getColors(customColors),customColors);
         });
 
-        test('get 10 colors when slices <=10',function() {
+        test('get 4 colors when slices <8',function() {
           assert.equal(myEl.colors.length,0); 
-          assert.equal(myEl._getColors([]).length,10);
+          assert.equal(myEl._getColors([]).length,4);
         });
-        test('get 20 colors when slices > 10',function() {
+        test('get 8 colors when slices > 4',function() {
           assert.equal(myEl.colors.length,0);
-          myEl.data = [1,2,3,4,5,6,7,8,9,10,11]; 
-          assert.equal(myEl._getColors().length,20);
+          myEl.data = [1,2,3,4,5]; 
+          assert.equal(myEl._getColors().length,8);
+        });    
+        test('get 16 colors when slices > 8',function() {
+          assert.equal(myEl.colors.length,0);
+          myEl.data = [1,2,3,4,5,6,7,8,9]; 
+          assert.equal(myEl._getColors().length,16);
         });
         test('calculate total',function() {
           var data = [1,2,4,5,6,7,8,9,10];
@@ -71,6 +76,14 @@
           var arcs = myEl._calculateArcs(colors,data);
           assert.equal(arcs.length,3);
           assert.deepEqual(arcs,arcsToCheck);
+        });
+         test('compute arcs for full circle', function() {
+          var arcsToCheck = [{color:'green',x1:180,x2:179.99,largeArcFlag:1,y1:0,y2:0}]
+          var data = [0,20,0];
+          var colors = ['red','green','blue'];
+          var arcs = myEl._calculateArcs(colors,data);
+          assert.equal(arcs.length,3);
+          assert.deepEqual(arcs[1],arcsToCheck[0])
         });
         test('test DOM', function(done) {
           myEl.data = [20,40,40];