Jody,

the attached fix plugs two hint leaks and makes the build pass with alphabetical, reversealphabetical, and (my) filesystem order.

I am ready to commit now. I will contact you first to ensure you do not have an alternative fix.

Kind regards,
Ben.

On 10/12/12 13:26, Ben Caradoc-Davies wrote:
Jody, I think the working directory is a red herring. The problem is the
axis order hints in the test fixtures. It is likely something is caching
a CRS built with the wrong hints.

--
Ben Caradoc-Davies <ben.caradoc-dav...@csiro.au>
Software Engineer
CSIRO Earth Science and Resource Engineering
Australian Resources Research Centre
diff --git a/modules/library/render/src/test/java/org/geotools/renderer/lite/RenderingTransformationTest.java b/modules/library/render/src/test/java/org/geotools/renderer/lite/RenderingTransformationTest.java
index 8c3415c..7da39b4 100644
--- a/modules/library/render/src/test/java/org/geotools/renderer/lite/RenderingTransformationTest.java
+++ b/modules/library/render/src/test/java/org/geotools/renderer/lite/RenderingTransformationTest.java
@@ -1,6 +1,6 @@
 package org.geotools.renderer.lite;
 
-import static org.junit.Assert.*;
+import static org.junit.Assert.assertEquals;
 
 import java.awt.Color;
 import java.awt.image.BufferedImage;
@@ -16,6 +16,7 @@ import org.geotools.data.FeatureSource;
 import org.geotools.data.Query;
 import org.geotools.data.property.PropertyDataStore;
 import org.geotools.factory.GeoTools;
+import org.geotools.factory.Hints;
 import org.geotools.filter.text.cql2.CQL;
 import org.geotools.filter.text.cql2.CQLException;
 import org.geotools.gce.geotiff.GeoTiffReader;
@@ -27,6 +28,7 @@ import org.geotools.map.MapContent;
 import org.geotools.referencing.CRS;
 import org.geotools.renderer.RenderListener;
 import org.geotools.styling.Style;
+import org.junit.After;
 import org.junit.Before;
 import org.junit.Test;
 import org.opengis.feature.simple.SimpleFeature;
@@ -44,6 +46,11 @@ public class RenderingTransformationTest {
         CRS.reset("all");
     }
 
+    @After
+    public void tearDown() {
+        Hints.removeSystemDefault(Hints.FORCE_LONGITUDE_FIRST_AXIS_ORDER);
+    }
+
     @Test
     public void testTransformReprojectedGridReader() throws Exception {
         Style style = RendererBaseTest.loadStyle(this, "coverageCenter.sld");
diff --git a/modules/library/render/src/test/java/org/geotools/renderer/lite/SpatialFilterTest.java b/modules/library/render/src/test/java/org/geotools/renderer/lite/SpatialFilterTest.java
index c82b997..a435dbd 100644
--- a/modules/library/render/src/test/java/org/geotools/renderer/lite/SpatialFilterTest.java
+++ b/modules/library/render/src/test/java/org/geotools/renderer/lite/SpatialFilterTest.java
@@ -104,7 +104,7 @@ public class SpatialFilterTest {
     
     @After
     public void tearDown() {
-        Hints.putSystemDefault(Hints.FORCE_LONGITUDE_FIRST_AXIS_ORDER, Boolean.FALSE);
+        Hints.removeSystemDefault(Hints.FORCE_LONGITUDE_FIRST_AXIS_ORDER);
     }
 
     @Test
------------------------------------------------------------------------------
LogMeIn Rescue: Anywhere, Anytime Remote support for IT. Free Trial
Remotely access PCs and mobile devices and provide instant support
Improve your efficiency, and focus on delivering more value-add services
Discover what IT Professionals Know. Rescue delivers
http://p.sf.net/sfu/logmein_12329d2d
_______________________________________________
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to