Reading-eScience-Centre / edal-java

Environmental Data Abstraction Layer libraries
Other
39 stars 30 forks source link

Bug in cdm/src/test/java/uk/ac/rdg/resc/edal/dataset/cdm/CdmGridDatasetFactoryTest.java #43

Closed mvtm-dn closed 8 years ago

mvtm-dn commented 8 years ago

Test did not passed in case of default TZ differs from UTC.

Fix:

--- a/cdm/src/test/java/uk/ac/rdg/resc/edal/dataset/cdm/CdmGridDatasetFactoryTest.java
+++ b/cdm/src/test/java/uk/ac/rdg/resc/edal/dataset/cdm/CdmGridDatasetFactoryTest.java
@@ -41,6 +41,7 @@ import java.util.Set;

 import org.geotoolkit.referencing.crs.DefaultGeographicCRS;
 import org.joda.time.DateTime;
+import org.joda.time.DateTimeZone;
 import org.junit.Before;
 import org.junit.Test;

@@ -119,7 +120,7 @@ public class CdmGridDatasetFactoryTest {
         for (Double zPos = 0.0; zPos <= 100; zPos += 10.0) {
             double expectedDepth = zPos;
             for (int daysFromStart = 0; daysFromStart < 10; daysFromStart++) {
-                DateTime time = new DateTime(2000, 01, 01 + daysFromStart, 00, 00);
+                DateTime time = new DateTime(2000, 01, 01 + daysFromStart, 00, 00,DateTimeZone.UTC);
                 float expectedTime = 100 * daysFromStart / 9.0f;

                 PlottingDomainParams params = new PlottingDomainParams(xSize, ySize,
guygriffiths commented 8 years ago

Thanks. Fixed in develop branch