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

Hardcoded BBox add a requirement to test data? #243

Open
coim32 opened this issue Jul 1, 2024 · 1 comment
Open

Hardcoded BBox add a requirement to test data? #243

coim32 opened this issue Jul 1, 2024 · 1 comment
Assignees
Labels
question Further information is requested

Comments

@coim32
Copy link

coim32 commented Jul 1, 2024

collectionsWithBboxes.add( new Object[] { collection, new BBox( -1.5, 50.0, 1.5, 53.0 ) } );

There are a few BBox hard coded, does this add a requirement for the test data?
If there are no features inside this bbox some test will fail?

@dstenger dstenger added this to CITE Nov 7, 2024
@github-project-automation github-project-automation bot moved this to To do in CITE Nov 7, 2024
@dstenger dstenger moved this from To do to In progress in CITE Nov 7, 2024
@dstenger
Copy link
Contributor

dstenger commented Nov 7, 2024

Thank you for reporting.
No, this does not add an additional requirement for the test data as the test is skipped if there is no feature inside the bounding box.
Can you please close the issue if this answers your question?

@dstenger dstenger self-assigned this Nov 7, 2024
@dstenger dstenger moved this from In progress to Waiting in CITE Nov 7, 2024
@dstenger dstenger added the question Further information is requested label Nov 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
Status: Waiting
Development

No branches or pull requests

2 participants