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

The WMS Client test does not put "Perform GetMap request for all the layers" on passed when all child tests have been passed #68

Closed
robinhoutmeyers opened this issue Nov 14, 2023 · 7 comments · Fixed by #72
Assignees
Labels
Projects
Milestone

Comments

@robinhoutmeyers
Copy link

I executed the WMS Client test in combination with our desktop WMS client software based on LuciadLightspeed. It works and it seems all child tests have been passed, but the "Perform GetMap request for all the layers" parent node seems to remain on the failed state.

Here is a screenshot of the test result:

image

As you can see, all tests below "Perform GetMap request for all the layers" are passed, but the parent node remains failed. I can consistently reproduce this.

@dstenger
Copy link
Contributor

Thank you for reporting.
I am also able to reproduce the behavior when testing with QGIS.
However, when finishing the test session and viewing the report, I cannot see any tracked failure. Thus, this just seems to be a display error.
Can you please check and confirm if this also applies to your use case?

@dstenger dstenger added this to To do in CITE via automation Nov 15, 2023
@dstenger dstenger self-assigned this Nov 15, 2023
@dstenger dstenger moved this from To do to Waiting in CITE Nov 15, 2023
@robinhoutmeyers
Copy link
Author

Indeed! The summary of results doesn't shown any failure after finishing the test session and viewing the report; only the test overview this shows it as shown in my screenshot, but I guess this is not a problem to request compliancy.

@dstenger
Copy link
Contributor

I agree, it should be no problem to request compliance.
However, I leave the issue open to further investigate why the display is not correct.

@dstenger dstenger removed their assignment Nov 15, 2023
@dstenger dstenger moved this from Waiting to To do in CITE Nov 15, 2023
@robinhoutmeyers
Copy link
Author

I also encountered a similar issue during a beta test for #66. Steps to reproduce:

=> although the test session console and the test results (after stopping the test) will recognize the fact that a GetCapabilities and GetMap requests were received, the display during the test is never updated.

@dstenger dstenger added the bug label Jan 8, 2024
@ghobona
Copy link
Contributor

ghobona commented Jan 8, 2024

@robinhoutmeyers Please note that since the issue is acknowledged as a bug in the ETS, you can proceed with submitting your product for compliance certification.

@robinhoutmeyers
Copy link
Author

Noted, thanks!

@dstenger dstenger moved this from To do to In progress in CITE Jan 18, 2024
This was referenced Jan 31, 2024
@dstenger
Copy link
Contributor

dstenger commented Feb 1, 2024

@robinhoutmeyers We created a pull request for the problem mentioned in the description of the issue.
However, the second issue reported here seems not to be related to the first one. Thus, we moved it to a separate issue: #73. We are already working on the new issue.

@dstenger dstenger moved this from In progress to To verify in CITE Feb 1, 2024
@dstenger dstenger assigned dstenger and unassigned bpross-52n Feb 1, 2024
@dstenger dstenger moved this from To verify to In progress in CITE Mar 8, 2024
@dstenger dstenger assigned bpross-52n and unassigned dstenger Mar 8, 2024
@dstenger dstenger moved this from In progress to To verify in CITE Apr 23, 2024
@dstenger dstenger assigned dstenger and unassigned bpross-52n Apr 23, 2024
@dstenger dstenger added this to the 1.7 milestone Apr 30, 2024
dstenger added a commit that referenced this issue Apr 30, 2024
CITE automation moved this from To verify to Done Apr 30, 2024
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.

4 participants