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

fix tests with language option #519

Merged
merged 1 commit into from
Feb 12, 2020
Merged

fix tests with language option #519

merged 1 commit into from
Feb 12, 2020

Conversation

cehbrecht
Copy link
Collaborator

Overview

This PR fixes the current test failures with language option.

Related Issue / Discussion

See PR #510.

Additional Information

Contribution Agreement

(as per https://github.com/geopython/pywps/blob/master/CONTRIBUTING.rst#contributions-and-licensing)

  • I'd like to contribute [feature X|bugfix Y|docs|something else] to PyWPS. I confirm that my contributions to PyWPS will be compatible with the PyWPS license guidelines at the time of contribution.
  • I have already previously agreed to the PyWPS Contributions and Licensing Guidelines

@cehbrecht cehbrecht added the bug label Feb 12, 2020
@cehbrecht cehbrecht added this to the 5.0.0 milestone Feb 12, 2020
@cehbrecht
Copy link
Collaborator Author

@davidcaron the CONFIG.set overwrite of the config does not work in the tests anymore when using pywps.tests.client_for. To get the tests working again I have added the language=en-US,fr-CA to the test config.

@cehbrecht
Copy link
Collaborator Author

@davidcaron I'm merging this to get the test suite running again. Maybe we can fix the config issue with another PR.

@cehbrecht cehbrecht merged commit a3d1ee9 into geopython:master Feb 12, 2020
@cehbrecht cehbrecht deleted the fix-tests branch February 12, 2020 14:39
@cehbrecht cehbrecht mentioned this pull request Feb 8, 2021
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant