Build failure caused by broken wfs strategy selection
-----------------------------------------------------

                 Key: GEOT-2160
                 URL: http://jira.codehaus.org/browse/GEOT-2160
             Project: GeoTools
          Issue Type: Bug
          Components: data wfs
    Affects Versions: 2.6-M0
            Reporter: Ben Caradoc-Davies
            Assignee: Gabriel Roldán
            Priority: Blocker
         Attachments: broken-wfs-strategy.patch

Build failure on head of trunk when building in a directory whose path contains 
"geoserver". Failed unit test is WFSDataStoreFactoryTest.

WFSDataStoreFactory examines capabilities comments to detect a CubeWerx server. 
This succeeds. However, if the capabilities document has a path with URI 
containing "geoserver", the algorithm will discard the CubeWerx strategy and 
use a GeoServerStrategy instead. This causes the unit test to fail.

To reproduce, checkout source as "geoserver/src/geotools-trunk", and run "mvn 
install".

Attached patch fixes this behaviour by only looking at URI when strategy has 
not yet been set.

As an aside, the algorithm should be improved to try harder to detect 
GeoServer. This is not a good test. Can't we use something like that used to 
detect CubeWerx?


-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

       

-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
Geotools-devel mailing list
Geotools-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to