We don't want spaces in the serialized value of style attributes.

Add a unittest that should catch this and other issues with parsing/serializing the `style` attribute

scour/scour.py

@@ -3105,7 +3105,7 @@ def serializeXML(element, options, ind = 0, preserveWhitespace = False):
       attrValue = makeWellFormed( attr.nodeValue )
       if attr.nodeName == 'style':
           # sort declarations
-          attrValue = '; '.join([p for p in sorted(attrValue.split(';'))])
+          attrValue = ';'.join([p for p in sorted(attrValue.split(';'))])
 
       outParts.append(' ')
       # preserve xmlns: if it is a namespace prefix declaration

testscour.py

@@ -1589,6 +1589,13 @@ class DocWithNoFlowtext(unittest.TestCase):
         except Exception as e:
             self.fail("exception '{}' was raised, and we didn't expect that!".format(e))
 
+
+class ParseStyleAttribute(unittest.TestCase):
+	def runTest(self):
+		doc = scour.scourXmlFile('unittests/style.svg')
+		self.assertEqual(doc.documentElement.getAttribute('style'), 'property1:value1;property2:value2;property3:value3',
+			'Style attribute not properly parsed and/or serialized')
+
 # TODO: write tests for --enable-viewboxing
 # TODO; write a test for embedding rasters
 # TODO: write a test for --disable-embed-rasters

unittests/style.svg

@@ -0,0 +1,2 @@
+<?xml version="1.0"?>
+<svg xmlns="http://www.w3.org/2000/svg" style="  ; ;;property1:   value1 ;   ;; property2   :value2; property3 : value3 ;;; ;"></svg>