Add tests for color.fill and color.stroke

Author: mdtusz

src/snapshot/tosvg.js

@@ -105,7 +105,7 @@ module.exports = function toSVG(gd, format) {
         }
     }
 
-    // remove draglayer (it's invisible anyways!)
+    // remove draglayer for Adobe Illustrator compatibility
     if(fullLayout._draggers) {
         fullLayout._draggers.node().remove();
     }

test/jasmine/tests/color_test.js

@@ -145,4 +145,34 @@ describe('Test color:', function() {
             expect(container2).toEqual(expectedContainer2);
         });
     });
+
+    describe('fill', function() {
+
+        it('should call style with both fill and fill-opacity', function() {
+            var mockElement = {
+                style: function(object) {
+                    expect(object.fill).toBe('rgb(255, 255, 0)');
+                    expect(object['fill-opacity']).toBe(0.5);
+                }
+            };
+
+            Color.fill(mockElement, 'rgba(255,255,0,0.5');
+        });
+
+    });
+
+    describe('stroke', function() {
+
+        it('should call style with both fill and fill-opacity', function() {
+            var mockElement = {
+                style: function(object) {
+                    expect(object.stroke).toBe('rgb(255, 255, 0)');
+                    expect(object['stroke-opacity']).toBe(0.5);
+                }
+            };
+
+            Color.stroke(mockElement, 'rgba(255,255,0,0.5');
+        });
+
+    });
 });