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

Re-activate the remote WMS integration tests #1446

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

tfr42
Copy link
Member

@tfr42 tfr42 commented Dec 30, 2022

Fixes #1246.

This PR re-activates the tests in module deegree-wms-remoteows-tests executed as part of the integration tests.

@tfr42 tfr42 added enhancement enhancement or improvement tests failing unit or Integration tests fail in progress tests unit/integration test labels Dec 30, 2022
@tfr42 tfr42 added this to the 3.5 milestone Dec 30, 2022
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the base64 encoded output to STDOUT was intentional, because by default the LOG is very strongly suppressed on the CI/CD system.
Had you checked whether the results still appear in the CI/CD log after the change in case of an image error ?

@lgoltz lgoltz modified the milestones: 3.5, 3.5.1 May 13, 2023
@lgoltz lgoltz added the needs rebase PR is not up to date and needs rebase label May 22, 2023
@tfr42 tfr42 modified the milestones: 3.5.1, 3.5.2 Aug 9, 2023
@tfr42 tfr42 removed this from the 3.5.2 milestone Aug 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement enhancement or improvement in progress needs rebase PR is not up to date and needs rebase tests failing unit or Integration tests fail tests unit/integration test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix and enable deegree-wms-remotewms-tests
3 participants