From f1f0aab96589fc22155b33e5279ceb4671bf8288 Mon Sep 17 00:00:00 2001
From: JSCHILL1 <jschill1@Lithium.local>
Date: Sun, 5 Jul 2009 08:14:40 -0500
Subject: [PATCH] Add closepath test

---
 release-notes.html                     |  3 ++-
 testscour.py                           |  8 +++++++-
 unittests/important-groups-in-defs.svg | 11 +++++++++++
 unittests/path-with-closepath.svg      |  3 +++
 4 files changed, 23 insertions(+), 2 deletions(-)
 create mode 100644 unittests/important-groups-in-defs.svg
 create mode 100644 unittests/path-with-closepath.svg

diff --git a/release-notes.html b/release-notes.html
index 6d40887..262af68 100644
--- a/release-notes.html
+++ b/release-notes.html
@@ -13,10 +13,11 @@
 	<header>
 		<h2><a href="#0.15">Version 0.15</a></h2>
 	</header>
-	<p>July 4th, 2009</p>
+	<p>July 5th, 2009</p>
 	<ul>
 		<li>added --keep-editor-data command-line option</li>
 		<li>Fix Bug 395645: Keep all identified children inside a defs (Thanks Frederik!)</li>
+		<li>Fix Bug 395647: Do not remove closepath (Z) path segments</li>
 	</ul>
 </section>
 
diff --git a/testscour.py b/testscour.py
index d12f325..ce9459d 100755
--- a/testscour.py
+++ b/testscour.py
@@ -600,7 +600,13 @@ class DoNotRemoveGroupsWithIDsInDefs(unittest.TestCase):
 	def runTest(self):
 		f = scour.scourXmlFile('unittests/important-groups-in-defs.svg')
 		self.assertEquals(len(f.getElementsByTagNameNS(SVGNS, 'linearGradient')), 1,
-			'Group in defs with id\'ed element removed');
+			'Group in defs with id\'ed element removed')
+
+class AlwaysKeepClosePathSegments(unittest.TestCase):
+	def runTest(self):
+		p= scour.scourXmlFile('unittests/path-with-closepath.svg').getElementsByTagNameNS(SVGNS, 'path')[0]
+		self.assertEquals(p.getAttribute('d'), 'M10,10h100v100h-100z',
+			'Path with closepath not preserved')
 
 # TODO: write tests for --set-precision for path data, for polygon data, for attributes
 # TODO; write a test for embedding rasters
diff --git a/unittests/important-groups-in-defs.svg b/unittests/important-groups-in-defs.svg
new file mode 100644
index 0000000..44b718c
--- /dev/null
+++ b/unittests/important-groups-in-defs.svg
@@ -0,0 +1,11 @@
+<svg xmlns="http://www.w3.org/2000/svg">
+<defs>
+	<g>
+		<linearGradient id="g1">
+			<stop offset="0" stop-color="blue"/>
+			<stop offset="1" stop-color="red"/>
+		</linearGradient>
+	</g>
+</defs>
+<rect fill="url(#g1)" width="100" height="100" />
+</svg>
\ No newline at end of file
diff --git a/unittests/path-with-closepath.svg b/unittests/path-with-closepath.svg
new file mode 100644
index 0000000..c9c771b
--- /dev/null
+++ b/unittests/path-with-closepath.svg
@@ -0,0 +1,3 @@
+<svg xmlns="http://www.w3.org/2000/svg">
+<path fill="none" stroke="#000" d="M10,10h100v100h-100z"/>
+</svg>
\ No newline at end of file