<div dir="ltr">Hi,<div>running the WMS 1.3 tests from trunk I've noticed that a couple of tests fail to take</div><div>into account the Layer queryable attribute is optional, and defaults to 0.</div><div>Fixing the tests is just a matter of adding a "and @queryable" in the tests, as in</div>
<div>the following patch:</div><div><br></div><div><div>diff --git a/wms/1.3.0/trunk/ctl/getfeatureinfo.xml b/wms/1.3.0/trunk/ctl/getfeatureinfo.xml</div><div>index 3e0b65e..67bc5ec 100644</div><div>--- a/wms/1.3.0/trunk/ctl/getfeatureinfo.xml</div>
<div>+++ b/wms/1.3.0/trunk/ctl/getfeatureinfo.xml</div><div>@@ -32,7 +32,7 @@</div><div>             <xsl:otherwise></div><div>                <ctl:call-test name="getfeatureinfo:exceptions"/></div><div>
                <xsl:choose></div><div>-                  <xsl:when test="not(wms:Capability//wms:Layer[functions:boolean-as-integer(@queryable) = 1]/wms:Name)"></div><div>+                  <xsl:when test="not(wms:Capability//wms:Layer[@queryable and functions:boolean-as-integer(@queryable) = 1]/wms:Name)">^M</div>
<div>                      <ctl:message>No queryable layers in capabilities XML.</ctl:message></div><div>                      <xsl:if test="$queryable='queryable'"></div><div>                         <ctl:fail/></div>
<div>@@ -107,7 +107,7 @@</div><div>          <ctl:call-test name="getfeatureinfo:invalid-query_layers"/></div><div>          <ctl:call-test name="getfeatureinfo:query_layers-not-queryable"/></div>
<div>          <xsl:variable name="capabilities" select="."/></div><div>-         <ctl:for-each select="wms:Capability//wms:Layer[wms:Name and functions:boolean-as-integer(@queryable) = 1]/wms:Name"></div>
<div>+         <ctl:for-each select="wms:Capability//wms:Layer[wms:Name and @queryable and functions:boolean-as-integer(@queryable) = 1]/wms:Name">^M</div><div>             <ctl:call-test name="getfeatureinfo:each-queryable-layer"></div>
<div>                <ctl:with-param name="capabilities" select="$capabilities"/></div><div>                <ctl:with-param name="layer" select="string(.)" label="each queryable layer" label-expr="concat('layer &quot;', ., '&quot;')"/></div>
</div><div><br></div><div><br></div><div>Cheers</div><div>Andrea</div><div><br></div><div>-- <br><div dir="ltr"><div><div>==</div><div>Our support, Your Success! Visit <a href="http://opensdi.geo-solutions.it" target="_blank">http://opensdi.geo-solutions.it</a> for more information.</div>
<div>==</div></div><div><br></div><div>Ing. Andrea Aime <br></div><div>@geowolf</div><div>Technical Lead</div><div><br></div><div>GeoSolutions S.A.S.</div><div>Via Poggio alle Viti 1187</div><div>55054  Massarosa (LU)</div>
<div>Italy</div><div>phone: +39 0584 962313</div><div>fax: +39 0584 1660272</div><div>mob: +39  339 8844549</div><div><br></div><div><a href="http://www.geo-solutions.it" target="_blank">http://www.geo-solutions.it</a></div>
<div><a href="http://twitter.com/geosolutions_it" target="_blank">http://twitter.com/geosolutions_it</a></div><div><br></div><div>-------------------------------------------------------</div></div>
</div></div>