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

Enhance test to count the features from the ESRI response format. #11

Closed
keshavnangare opened this issue Aug 7, 2018 · 3 comments · Fixed by #12
Closed

Enhance test to count the features from the ESRI response format. #11

keshavnangare opened this issue Aug 7, 2018 · 3 comments · Fixed by #12
Assignees
Labels
Projects
Milestone

Comments

@keshavnangare
Copy link
Collaborator

[1] https://externaltest.dev.geocloud.com/server/services/PuertoRicoRaster/MapServer/WMSServer?request=GetCapabilities&service=WMS

With reference: opengeospatial/ets-wms13-nsg#16

The test "wms Get Feature Info Feature Count With Value Of One" and "wms Get Feature Info Feature Count With Value Of Ten" need to update to count the feature from the ESRI response.

@keshavnangare
Copy link
Collaborator Author

Fixed this issue with commit 76e4381 [PR: #12]

Closes opengeospatial/ets-wms13-nsg#16.

@lgoltz
Copy link
Contributor

lgoltz commented Nov 8, 2018

Unfortunately the service is not available. I've added some tests and made parsing of feature members more robust (allow GML 3.2).

@lgoltz lgoltz removed their assignment Nov 8, 2018
@dstenger dstenger added this to To do in CITE via automation Apr 11, 2019
@dstenger dstenger moved this from To do to Waiting in CITE Apr 11, 2019
@dstenger dstenger moved this from Waiting to To verify in CITE Apr 11, 2019
@dstenger
Copy link
Contributor

dstenger commented Jun 5, 2019

Testing is not possible as no test service available. However, implementation looks good.

CITE automation moved this from To verify to Done Jun 5, 2019
@dstenger dstenger added this to the 0.5 milestone Jun 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
CITE
  
Done
Development

Successfully merging a pull request may close this issue.

3 participants