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

Server: Do not display Empty group in Capabilities and project settings #51724

Merged

Conversation

rldhont
Copy link
Contributor

@rldhont rldhont commented Feb 3, 2023

The test_wms_getprojectsettings in test_qgsserver_accesscontrol_wms.py was marked as Expectedfailure, after checking duplicated test methods. This test checks that an empty group because of access control could not be in GetCapabilities document with settings activated. The same test did not exist for simple getCapabilities.

This commit fix it

Funded by 3Liz https://3liz.com

@rldhont rldhont added Bug Either a bug report, or a bug fix. Let's hope for the latter! Server Related to QGIS server testsuite Issue related to testsuite labels Feb 3, 2023
@rldhont rldhont requested review from elpaso and pblottiere February 3, 2023 12:07
@github-actions github-actions bot added this to the 3.30.0 milestone Feb 3, 2023
The `test_wms_getprojectsettings` in `test_qgsserver_accesscontrol_wms.py` was marked as `Expectedfailure`, after checking duplicated test methods.
This test checks that an empty group because fo acces control could not be in GetCapabilities document with settings activated.
The same test did not exist for simple getCapabilities.

This commit fix it

Funded by 3Liz https://3liz.com
@rldhont rldhont force-pushed the fix-server-wms-capabilities-empty-group branch from f64941b to ff0ca50 Compare February 3, 2023 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter! Server Related to QGIS server testsuite Issue related to testsuite
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants