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

test framework: allow testing using all test data from system under test #104

Merged
merged 1 commit into from
Apr 7, 2022

Conversation

tlvu
Copy link
Contributor

@tlvu tlvu commented Apr 5, 2022

Set TEST_NO_USE_PROD_DATA to use all test data from system under test.

Useful to test future production server that will have all the huge datas.

Fixes #103.

Relate to bird-house/finch#232 and Ouranosinc/pavics-sdi#254.

Set TEST_NO_USE_PROD_DATA to use all test data from system under test.

Useful to test future production server that will have all the huge datas.

Fixes #103.
@tlvu tlvu requested a review from matprov April 5, 2022 03:37
tlvu added a commit to Ouranosinc/pavics-sdi that referenced this pull request Apr 5, 2022
…w-production-server

notebooks: allow to test GeoServer on new production server and fix old .ncml path

Switch from `boreas.ouranos.ca` to `pavics.ouranos.ca` will allow us to
conditionally regex replace `pavics.ouranos.ca` to the server under test
so we can actually test the Geoserver on that new server.

Everywhere under this pavics-sdi repo, we use `pavics.ouranos.ca`.  This
is the only locations that hardcode `boreas.ouranos.ca` because
geoserver data are not replicated to standard test servers so hardcoding
it this way will use the data from the production server.

This patch achieve the same "data from prod" but also allow override if
we need to actually test the future production server.

Related to #183 and Ouranosinc/PAVICS-e2e-workflow-tests#104.

Also fix old `nrcan_v2.ncml` path in `regridding.ipynb`.
tlvu added a commit to bird-house/finch that referenced this pull request Apr 5, 2022
…w-production-server

subset.ipynb: allow to test GeoServer on new production server

Switch from `boreas.ouranos.ca` to `pavics.ouranos.ca` will allow us to
conditionally regex replace `pavics.ouranos.ca` to the server under test
so we can actually test the Geoserver on that new server.

Everywhere under this finch repo, we use `pavics.ouranos.ca`. This
is the only locations that hardcode `boreas.ouranos.ca` because
geoserver data are not replicated to standard test servers so hardcoding
it this way will use the data from the production server.

This patch achieve the same "data from prod" but also allow override if
We need to actually test the future production server.

Related to Ouranosinc/pavics-sdi#183 and Ouranosinc/PAVICS-e2e-workflow-tests#104.
@tlvu tlvu merged commit 8dae868 into master Apr 7, 2022
@tlvu tlvu deleted the allow-to-test-using-all-data-from-test-server branch April 7, 2022 12:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow to run everything from server under test for testing new production server
2 participants