Add unit tests for removing fill attributes when fill=none

scour.py

@@ -44,6 +44,9 @@
 #  * Process quadratic Bezier curves
 #  * Collapse all group based transformations
 
+# Suggestion from Richard Hutch:
+#  * Put id attributes first in the serialization (or make the d attribute last)
+
 # Next Up:
 # + fix bug when removing stroke styles
 # - Convert all colors to #RRGGBB format

testscour.py

@@ -340,6 +340,18 @@ class RemoveStrokeDashoffsetWhenStrokeNone(unittest.TestCase):
 		self.assertEquals(doc.getElementsByTagNameNS(SVGNS, 'path')[0].getAttribute('stroke-dashoffset'), '',
 			"stroke-dashoffset attribute not emptied when no stroke" )
 
+class RemoveFillRuleWhenFillNone(unittest.TestCase):
+	def runTest(self):
+		doc = scour.scourXmlFile('unittests/fill-none.svg')
+		self.assertEquals(doc.getElementsByTagNameNS(SVGNS, 'path')[0].getAttribute('fill-rule'), '',
+			"fill-rule attribute not emptied when no fill" )
+			
+class RemoveFillOpacityWhenFillNone(unittest.TestCase):
+	def runTest(self):
+		doc = scour.scourXmlFile('unittests/fill-none.svg')
+		self.assertEquals(doc.getElementsByTagNameNS(SVGNS, 'path')[0].getAttribute('fill-opacity'), '',
+			"fill-opacity attribute not emptied when no fill" )
+			
 #class RemoveUnreferencedFonts(unittest.TestCase):
 #	def runTest(self):
 #		doc = scour.scourXmlFile('unittests/unreferenced-font.svg')

unittests/fill-none.svg

@@ -0,0 +1,3 @@
+<svg xmlns="http://www.w3.org/2000/svg">
+    <path id="p" style="fill: none; fill-rule: evenodd; fill-opacity: 0.5;" d="M 7.7592046,36.982095 C 7.8831049,40.873696 7.8339808,45.305308 7.8339808,49.436888  Z" />
+</svg>