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

Assert that features in response are in requested BoundingBox #18

Closed
lgoltz opened this issue Jul 2, 2018 · 5 comments · Fixed by #220
Closed

Assert that features in response are in requested BoundingBox #18

lgoltz opened this issue Jul 2, 2018 · 5 comments · Fixed by #220
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@lgoltz
Copy link
Contributor

lgoltz commented Jul 2, 2018

Test: org.opengis.cite.wfs30.collections.GetFeaturesOperation#validateBboxParameter_requests()

Abstract Test 22, Method 3

@lgoltz lgoltz added the enhancement New feature or request label Jul 2, 2018
@ghobona
Copy link
Contributor

ghobona commented Feb 12, 2021

@ghobona
Copy link
Contributor

ghobona commented Mar 10, 2021

@keshav-nangare

Verified using OpenAPITools Generator that bbox is supposed to be structured like:

            "extent": {
                "spatial": {
                    "bbox": [
                        [
                            11.0,
                            21.0,
                            31.0,
                            41.0
                        ]
                    ],
                    "crs": "http://www.opengis.net/def/crs/OGC/1.3/CRS84"
                }
            }

So the ETS should be enabled to handle this.

@keshavnangare
Copy link

Thanks, @ghobona, I will update ETS for the same.

@keshavnangare
Copy link

The parseSpatialExtent method updated to trigger the unimplemented test. Please find the changes in #149 PR.

@dstenger
Copy link
Contributor

dstenger commented Jun 1, 2023

Method was moved to

public void validateFeaturesWithBoundingBoxResponse( Map<String, Object> collection, BBox bbox ) {

@bpross-52n bpross-52n mentioned this issue Jul 6, 2023
@dstenger dstenger assigned dstenger and unassigned bpross-52n Jul 6, 2023
@dstenger dstenger added this to the 1.5 milestone Jul 7, 2023
dstenger added a commit that referenced this issue Jul 19, 2023
dstenger added a commit that referenced this issue Jul 19, 2023
dstenger added a commit that referenced this issue Jul 19, 2023
@dstenger dstenger added this to CITE Aug 1, 2024
@dstenger dstenger moved this to Done in CITE Aug 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Archived in project
5 participants