Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Test GetTileParametersKvp.wmtsGetTileRequestFormatParameters fails cause the RESTful URL is used #43

Open
lgoltz opened this issue Aug 9, 2018 · 5 comments · Fixed by #49
Labels
Projects

Comments

@lgoltz
Copy link
Contributor

lgoltz commented Aug 9, 2018

The test GetTileParametersKvp.wmtsGetTileRequestFormatParameters fails with "The following asserts failed: Expected status code(s) [200] but received 400." if the Capabilities contains URLs for RESTful and GET:

<ows:Operation name="GetTile">
    <ows:DCP>
        <ows:HTTP>
            <ows:Get
                    xlink:href="https://externaltest.dev.geocloud.com/server/rest/services/PuertoRicoRaster/MapServer/WMTS/tile/1.0.0/">
                <ows:Constraint name="GetEncoding">
                    <ows:AllowedValues>
                        <ows:Value>RESTful</ows:Value>
                    </ows:AllowedValues>
                </ows:Constraint>
            </ows:Get>
            <ows:Get
                    xlink:href="https://externaltest.dev.geocloud.com/server/rest/services/PuertoRicoRaster/MapServer/WMTS?">
                <ows:Constraint name="GetEncoding">
                    <ows:AllowedValues>
                        <ows:Value>KVP</ows:Value>
                    </ows:AllowedValues>
                </ows:Constraint>
            </ows:Get>
        </ows:HTTP>
    </ows:DCP>
</ows:Operation>

The following request is send: https://externaltest.dev.geocloud.com/server/rest/services/PuertoRicoRaster/MapServer/WMTS/tile/1.0.0/?TileMatrixSet=default&Format=image%2Fpng&Version=1.0.0&TileRow=73349&Request=GetTile&Service=WMTS&Style=default&TileCol=512055&Layer=PuertoRicoRaster&TileMatrix=20

Instead the base URL https://externaltest.dev.geocloud.com/server/rest/services/PuertoRicoRaster/MapServer/WMTS? should be used.

@lgoltz lgoltz added the bug label Aug 9, 2018
@lgoltz
Copy link
Contributor Author

lgoltz commented Aug 9, 2018

The same applies to

  • GetTileParametersRest.wmtsGetTileRequestParametersSupported
  • GetTileOfferings.wmtsGetCapabilitiiesWithHTML
  • GetTileOfferings.wmtsGetTileOfferingsTestGIF
  • GetTileOfferings.wmtsGetTileOfferingsTestJPG
  • GetTileOfferings.wmtsGetTileOfferingsTestPNG
  • GetTileCachingInfo.wmtsGetTileCachingInformationExists

@keshavnangare
Copy link

@lgoltz

I have add a fix but the service is not available to test these changes. Can you please provide the service URL?

@dstenger dstenger assigned lgoltz and unassigned keshavnangare Jan 9, 2019
@dstenger dstenger added this to In progress in CITE Apr 11, 2019
@dstenger dstenger moved this from In progress to To do in CITE Jun 6, 2019
@marycarome
Copy link

@keshav-nangare
I encountered the same issues. You should be able to test your changes using this service: http://services.arcgisonline.com/arcgis/rest/services/World_Topo_Map/MapServer/WMTS?Service=WMTS

@dstenger dstenger assigned keshavnangare and unassigned lgoltz Oct 23, 2019
@dstenger dstenger moved this from To do to In progress in CITE Oct 23, 2019
@keshavnangare
Copy link

The changes are successfully tested with the above service. Now the test will use the base URL instead of the RESTful URL.

Fixed with this commit: 6a750f8

@keshavnangare keshavnangare moved this from In progress to To verify in CITE Nov 11, 2019
@dstenger dstenger assigned dstenger and lgoltz and unassigned keshavnangare Nov 13, 2019
@dstenger dstenger moved this from To verify to Alliance partner in CITE May 17, 2021
@dstenger dstenger added this to the 0.4 milestone Aug 5, 2022
CITE automation moved this from Alliance partner to Done Aug 5, 2022
@dstenger dstenger reopened this Aug 5, 2022
CITE automation moved this from Done to In progress Aug 5, 2022
@dstenger dstenger removed this from the 0.4 milestone Aug 5, 2022
@dstenger dstenger moved this from In progress to To do in CITE Aug 5, 2022
@dstenger
Copy link
Contributor

dstenger commented Aug 5, 2022

Fix #49 had to be reverted as the pull request just works for the specific case described in the issue (#56).
More common cases do not work anymore which is confirmed by one failing unit test.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
CITE
  
To do
Development

Successfully merging a pull request may close this issue.

4 participants