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

wcs 2.0.1 te2 Test wcs2:req41 has problem #124

Closed
greatzzk opened this issue Aug 2, 2023 · 8 comments · Fixed by #129
Closed

wcs 2.0.1 te2 Test wcs2:req41 has problem #124

greatzzk opened this issue Aug 2, 2023 · 8 comments · Fixed by #129
Assignees
Labels
Projects
Milestone

Comments

@greatzzk
Copy link

greatzzk commented Aug 2, 2023

error msg from test:
image
image

but i think my server has the correct response containing an "invalidAxisLabel" exception.
http://54.90.187.123:6080/arcgis/services/mynetcdf/ImageServer/WCSServer?&request=GetCoverage&coverageid=mynetcdf&service=WCS&VERSION=2.0.1&subset=fakeaxislabel(10)
image

could you help investigate?

@dstenger dstenger added this to To do in CITE via automation Aug 3, 2023
@bpross-52n
Copy link
Contributor

Hi @greatzzk, we noticed that the server responds a HTTP 404 error with no body, if we execute the request in your comment above. Could you have a look into that?

@dstenger dstenger moved this from To do to In progress in CITE Aug 3, 2023
@greatzzk
Copy link
Author

greatzzk commented Aug 3, 2023

@bpross-52n i am not sure if i understand your comment. isn't the response displaying in screenshot above showing it is inside "body" of the response?

@bpross-52n
Copy link
Contributor

@greatzzk You are right, sorry. I only opened the URL in the browser and there no body was shown, only a browser error page. With more advanced tools the body is shown. Sorry for the confusion.

@bpross-52n
Copy link
Contributor

@greatzzk The problem is that the test suite is only looking for Envelope-Elements in the CoverageDescription and your service has EnvelopWithTimePeriod, which is fine, of course. So I think this is a bug in the test suite. I will work on a solution.

@bpross-52n
Copy link
Contributor

My current findings are:
The failing test (see the code here) sends the following request:
http://54.90.187.123:6080/arcgis/services/mynetcdf/ImageServer/WCSServer?service=WCS&VERSION=2.0.1&request=GetCoverage&coverageid=mynetcdf&subset=x(-1.67048810872E7)&subset=x(-1.67048810872E7,-1.67048810872E7)

Response:

<ows:ExceptionReport
	xmlns:ows="http://www.opengis.net/ows/2.0"
	xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" version="2.0.0"
	xsi:schemaLocation="http://www.opengis.net/ows/2.0 http://schemas.opengis.net/ows/2.0/owsExceptionReport.xsd">
	<ows:Exception exceptionCode="InvalidSubsetting"
		locator="(-1.67048810872E7,-1.67048810872E7)" />
</ows:ExceptionReport>

Other services return an InvalidAxisLabel exception code here, because of the duplicate x-axis, thus pasing the test. We will confirm that this is the correct behavior.

@dstenger dstenger moved this from In progress to Waiting in CITE Aug 10, 2023
@dstenger dstenger moved this from Waiting to In progress in CITE Aug 10, 2023
@dstenger
Copy link
Contributor

Requirement 41 is specified in this document https://portal.opengeospatial.org/files/?artifact_id=48428.
@bpross-52n Can you please check whether InvalidAxisLabel and InvalidSubsetting are correct in this case and adjust the test accordingly?

@bpross-52n bpross-52n mentioned this issue Aug 17, 2023
@dstenger dstenger assigned dstenger and unassigned bpross-52n Aug 18, 2023
@dstenger dstenger moved this from In progress to To verify in CITE Aug 18, 2023
@bpross-52n
Copy link
Contributor

The test was modified now so that a valid subsetting request is created (first axis with correct values, second axis with correct values) and then a duplicate of the first axis with correct values is appended to the request. This leads now to the expexted exception code InvalidAxisLabel. An InvalidSubsetting exception should not be returned anymore, because the subsetting is correct. I tested with the service referenced here.

@dstenger dstenger added the bug label Aug 18, 2023
@dstenger dstenger added this to the 1.20 milestone Aug 22, 2023
CITE automation moved this from To verify to Done Aug 22, 2023
dstenger added a commit that referenced this issue Aug 22, 2023
@greatzzk
Copy link
Author

greatzzk commented Oct 3, 2023

i verified this issue on te2 against my server. cheers!!

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