Hmmmm odd. Hard to find if we cannot reproduce it. Seems like the build 
has passed.

On 17-02-16 09:58, Ben Caradoc-Davies wrote:
> Andrea,
>
> I do not think that this can be ascribed to precision problems unless 
> File.lastModified() on Windows is unstable.
>
> The test compares the File.lastModified() of the resource obtained 
> directly with the File.lastModified() of the same resource, encoded in 
> JSON, HTML, and XML responses. Both timestamps are obtained via the 
> same File.lastModified() code path on the same platform. As the test 
> file is created once and then only read, it is surprising that its 
> lastModified would change on any platform.
>
> Some sort of caching, an automated process, or virtual machine time 
> adjustment, perhaps? I am open to suggestions.
>
> Kind regards,
> Ben.
>
> On 17/02/16 21:25, Andrea Aime wrote:
>> Hi Ben,
>> could it be related to windows often having a more granular "last file
>> modification"
>> time than linux (this really depends on the file system in use, but 
>> still):
>>
>> http://www.coderanch.com/t/384700/java/java/File-lastModified-windows-linux 
>>
>>
>> Long story short, I'm wondering if the test can be modified to ignore 
>> that
>> difference
>>
>> Cheers
>> Andrea
>>
>>
>> On Wed, Feb 17, 2016 at 7:50 AM, Ben Caradoc-Davies <b...@transient.nz>
>> wrote:
>>
>>> Niels,
>>>
>>> probably fixed on master:
>>>
>>> https://github.com/geoserver/geoserver/commit/3f26c66017e665b9ebbfc2b14f8f196371104ac8
>>>  
>>>
>>>
>>> I say probably because the first subsequent build had a different
>>> failure: the response lastModified was 15 milliseconds later than the
>>> resource lastModified. Some sort of intermittent Windows problem? Or a
>>> caching issue? The current build has passed this point, but the failure
>>> may recur. A single failure does not justify a Jira issue, so here 
>>> you go:
>>>
>>> Failed tests:
>>> testDirectoryXML(org.geoserver.rest.resources.ResourceTest):
>>> expected:<...016-02-17 05:45:40.4[43] UTC> but was:<...016-02-17
>>> 05:45:40.4[58] UTC>
>>> testDirectoryJSON(org.geoserver.rest.resources.ResourceTest): objects
>>> differed at key [ResourceDirectory];: objects differed at key
>>> [lastModified]; expected:<...016-02-17 05:45:40.5[52] UTC> but
>>> was:<...016-02-17 05:45:40.5[67] UTC>
>>>
>>> I will remove the -DquietTests=false, returning GeoServer-Master on
>>> winbuild to the state it was.
>>>
>>> Kind regards,
>>> Ben.
>>>
>>> On 17/02/16 18:16, Ben Caradoc-Davies wrote:
>>>> Aha! Revealing logged stack trace included below.
>>>>
>>>> Niels, at ResourceResource:275, instead of java.nio.file.Paths.get, 
>>>> did
>>>> you intend to use org.geoserver.platform.resource.Paths.path? Too many
>>>> Paths!
>>>>
>>>> This passes for me locally so I will push to master to see if it fixes
>>>> Windows.
>>>>
>>>> Kind regards,
>>>> Ben.
>>>>
>>>>
>>>> 17 feb 05:57:27 ERROR [geoserver.rest] - Exception intercepted
>>>> java.nio.file.InvalidPathException: UNC path is missing sharename:
>>> /\mydir
>>>>        at 
>>>> sun.nio.fs.WindowsPathParser.parse(WindowsPathParser.java:118)
>>>>        at 
>>>> sun.nio.fs.WindowsPathParser.parse(WindowsPathParser.java:77)
>>>>        at sun.nio.fs.WindowsPath.parse(WindowsPath.java:94)
>>>>        at 
>>>> sun.nio.fs.WindowsFileSystem.getPath(WindowsFileSystem.java:255)
>>>>        at java.nio.file.Paths.get(Paths.java:84)
>>>>        at
>>>>
>>> org.geoserver.rest.resources.ResourceResource$1$1.wrapInternal(ResourceResource.java:275)
>>>  
>>>
>>>>        at
>>>>
>>> org.geoserver.rest.resources.ResourceResource$1$1.wrapInternal(ResourceResource.java:267)
>>>  
>>>
>>>>        at
>>>>
>>> org.geoserver.rest.format.ReflectiveHTMLFormat$ObjectToMapWrapper.wrap(ReflectiveHTMLFormat.java:335)
>>>  
>>>
>>>>        at
>>>>
>>> org.geoserver.rest.format.ReflectiveHTMLFormat$1.wrap(ReflectiveHTMLFormat.java:156)
>>>  
>>>
>>>>        at
>>>>
>>> freemarker.template.Template.createProcessingEnvironment(Template.java:356) 
>>>
>>>>        at freemarker.template.Template.process(Template.java:237)
>>>>        at
>>>>
>>> org.restlet.ext.freemarker.TemplateRepresentation.write(TemplateRepresentation.java:112)
>>>  
>>>
>>>>        at
>>>>
>>> com.noelios.restlet.http.HttpServerCall.writeResponseBody(HttpServerCall.java:342)
>>>  
>>>
>>>>        at
>>>>
>>> com.noelios.restlet.http.HttpServerCall.sendResponse(HttpServerCall.java:306)
>>>  
>>>
>>>>        at
>>>>
>>> com.noelios.restlet.ext.servlet.ServletCall.sendResponse(ServletCall.java:306)
>>>  
>>>
>>>>        at
>>>>
>>> org.geoserver.rest.GeoServerServletConverter.commit(GeoServerServletConverter.java:54)
>>>  
>>>
>>>>        at
>>>>
>>> com.noelios.restlet.ext.servlet.ServletConverter.service(ServletConverter.java:130)
>>>  
>>>
>>>>        at
>>>>
>>> org.geoserver.rest.RESTDispatcher.handleRequestInternal(RESTDispatcher.java:87)
>>>  
>>>
>>>>        at
>>>>
>>> org.springframework.web.servlet.mvc.AbstractController.handleRequest(AbstractController.java:153)
>>>  
>>>
>>>>        at
>>>>
>>> org.springframework.web.servlet.mvc.SimpleControllerHandlerAdapter.handle(SimpleControllerHandlerAdapter.java:48)
>>>  
>>>
>>>>        at
>>>>
>>> org.springframework.web.servlet.DispatcherServlet.doDispatch(DispatcherServlet.java:933)
>>>  
>>>
>>>>        at
>>>>
>>> org.springframework.web.servlet.DispatcherServlet.doService(DispatcherServlet.java:867)
>>>  
>>>
>>>>        at
>>>>
>>> org.springframework.web.servlet.FrameworkServlet.processRequest(FrameworkServlet.java:953)
>>>  
>>>
>>>>        at
>>>>
>>> org.springframework.web.servlet.FrameworkServlet.doGet(FrameworkServlet.java:844)
>>>  
>>>
>>>>        at javax.servlet.http.HttpServlet.service(HttpServlet.java:689)
>>>>        at
>>>>
>>> org.springframework.web.servlet.FrameworkServlet.service(FrameworkServlet.java:829)
>>>  
>>>
>>>>        at javax.servlet.http.HttpServlet.service(HttpServlet.java:802)
>>>>        at
>>>>
>>> org.geoserver.test.GeoServerSystemTestSupport$1.service(GeoServerSystemTestSupport.java:1466)
>>>  
>>>
>>>>        at javax.servlet.http.HttpServlet.service(HttpServlet.java:802)
>>>>        at
>>>> com.mockrunner.mock.web.MockFilterChain.doFilter(MockFilterChain.java:50) 
>>>>
>>>>        at
>>>>
>>> org.geoserver.test.GeoServerSystemTestSupport.dispatch(GeoServerSystemTestSupport.java:1485)
>>>  
>>>
>>>>        at
>>>>
>>> org.geoserver.test.GeoServerSystemTestSupport.dispatch(GeoServerSystemTestSupport.java:1415)
>>>  
>>>
>>>>        at
>>>>
>>> org.geoserver.test.GeoServerSystemTestSupport.getAsServletResponse(GeoServerSystemTestSupport.java:949)
>>>  
>>>
>>>>        at
>>>>
>>> org.geoserver.test.GeoServerSystemTestSupport.getAsServletResponse(GeoServerSystemTestSupport.java:932)
>>>  
>>>
>>>>        at
>>>>
>>> org.geoserver.test.GeoServerSystemTestSupport.get(GeoServerSystemTestSupport.java:917)
>>>  
>>>
>>>>        at
>>>>
>>> org.geoserver.test.GeoServerSystemTestSupport.getAsDOM(GeoServerSystemTestSupport.java:1264)
>>>  
>>>
>>>>        at
>>>>
>>> org.geoserver.test.GeoServerSystemTestSupport.getAsDOM(GeoServerSystemTestSupport.java:1199)
>>>  
>>>
>>>>        at
>>>>
>>> org.geoserver.rest.resources.ResourceTest.testDirectoryHTML(ResourceTest.java:211)
>>>  
>>>
>>>> [...]
>>>> [Fatal Error] :1:1: Premature end of file.
>>>>
>>>>
>>>
>>> -- 
>>> Ben Caradoc-Davies <b...@transient.nz>
>>> Director
>>> Transient Software Limited <http://transient.nz/>
>>> New Zealand
>>>
>>>
>>> ------------------------------------------------------------------------------
>>>  
>>>
>>> Site24x7 APM Insight: Get Deep Visibility into Application Performance
>>> APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
>>> Monitor end-to-end web transactions and take corrective actions now
>>> Troubleshoot faster and improve end-user experience. Signup Now!
>>> http://pubads.g.doubleclick.net/gampad/clk?id=272487151&iu=/4140
>>> _______________________________________________
>>> Geoserver-devel mailing list
>>> Geoserver-devel@lists.sourceforge.net
>>> https://lists.sourceforge.net/lists/listinfo/geoserver-devel
>>>
>>
>>
>>
>


------------------------------------------------------------------------------
Site24x7 APM Insight: Get Deep Visibility into Application Performance
APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
Monitor end-to-end web transactions and take corrective actions now
Troubleshoot faster and improve end-user experience. Signup Now!
http://pubads.g.doubleclick.net/gampad/clk?id=272487151&iu=/4140
_______________________________________________
Geoserver-devel mailing list
Geoserver-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel

Reply via email to