Details

    • Type: Bug
    • Status: Resolved
    • Priority: Medium
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.13-RC1
    • Component/s: None
    • Labels:
      None

      Description

      the TextDecorationTest fails, presumably anywhere behind GMT

      org.junit.ComparisonFailure: 
      Expected :31.10.2008
      Actual   :30.10.2008
       <Click to see difference>
      
      
      	at org.junit.Assert.assertEquals(Assert.java:115)
      	at org.junit.Assert.assertEquals(Assert.java:144)
      	at org.geoserver.wms.decoration.TextDecorationTest.testExpandRequestVariable(TextDecorationTest.java:48)
      	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
      	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
      	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
      	at java.lang.reflect.Method.invoke(Method.java:497)
      	at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:47)
      	at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
      	at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:44)
      	at org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
      	at org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27)
      	at org.junit.runners.ParentRunner.runLeaf(ParentRunner.java:271)
      	at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:70)
      	at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:50)
      	at org.junit.runners.ParentRunner$3.run(ParentRunner.java:238)
      

      I took a brief look at it, but it's not clear how to fix this test case. The date gets converted to local time (which I think is normal for java), but it's not clear how to both format the output date into a consistent time zone, but I'm definitely not a Freemarker expert.

      It's not obvious what the intent of the test is, so I'm not sure what the fix is. If you just leave off all TZ info altogether then the test seems to work.

        Attachments

          Activity

            People

            • Assignee:
              aaime Andrea Aime
              Reporter:
              dtucker Devon Tucker
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: