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

Why using a not found URL when passing input by reference #33

Closed
gfenoy opened this issue Feb 8, 2023 · 1 comment · Fixed by #36
Closed

Why using a not found URL when passing input by reference #33

gfenoy opened this issue Feb 8, 2023 · 1 comment · Fixed by #36
Assignees
Projects
Milestone

Comments

@gfenoy
Copy link

gfenoy commented Feb 8, 2023

Describe the bug
To pass the ValidInputDataTranmissionByReference test, it is required that IUT supports 404 as an acceptable status code when downloading a reference. Indeed, the file used as a template (https://github.com/opengeospatial/ets-wps20/blob/main/src/main/resources/org/opengis/cite/wps20/examples/ValidInputReference.xml) to provide an input by reference uses an URL which return 404.

To Reproduce
Run the ETS.

Expected behavior
Using a found URL (200) as reference for testing execution properly.
In case the input reference returns a 404, I think that the IUT should return an exception with a 400 status code.

@gfenoy gfenoy changed the title Why using a not found URL when passing a reference Why using a not found URL when passing input by reference Feb 8, 2023
@ghobona ghobona added this to To do in CITE via automation Mar 8, 2023
@dstenger dstenger moved this from To do to In progress in CITE Mar 15, 2023
@ghobona
Copy link
Contributor

ghobona commented Mar 15, 2023

@gfenoy We will change the URL in the example to reference an XML document that exists.

@dstenger dstenger moved this from In progress to To verify in CITE Apr 12, 2023
CITE automation moved this from To verify to Done Apr 12, 2023
@arieslai arieslai added this to the 0.4 milestone Aug 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
CITE
  
Done
Development

Successfully merging a pull request may close this issue.

3 participants