-
Notifications
You must be signed in to change notification settings - Fork 2
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 set for release #151
Comments
4/26/2024 Test
Test 4 revealed that the spatial vulnerability question response changed by user in test 3 (overriding the one calculated during spatial data analysis) was not retained when re-loading from saved csv. Instead, the response reverted to the response calculated during spatial data analysis, erasing the user selection. This issue seems to only occur in 127-exposure-results-table, the master branch handles this scenario as expected. |
Can you give more details of your test for tests 3 and 4? What data was used etc? Modifying the spatial results, saving to csv and then reloading seems to work fine in this branch just using the built in data. Or see if you can reproduce this problem again out side of the other parts of the test. I don't know how anything changed in this branch would create this issue. |
This issue outlines 5 tests to be used as a minimum test set that can be manually run in the ccviR app prior to release. These 5 tests have been designed to check that a range of app functions work properly. While testing, complete the test checklist when the described item responds as expected.
When testing the app for release, be sure to simulate usage by new users. Install the library from GitHub in a fresh R session with
remotes::install_github("LandSciTech/ccviR")
.Sample spatial data sourced from the internet to be used for testing can be found this zipped folder. The README contained in the folder outlines what data is included and its intended use.
While testing, be sure to check for errors, warnings, and unexpected results both within the app and in the R console.
Test 1
Check that the app can handle the following:
To test this:
Run the app in demo mode
run_ccvi_app("demo")
and follow the "Using the ccviR app" tutorialTest 2
Check that the app can handle the following:
To test this:
run_ccvi_app("path/to/user/data")
Test 3
Check that the app can handle the following:
To test this:
run_ccvi_app("path/to/user/data")
Test 4
Check that the app can handle the following:
To test this:
run_ccvi_app("path/to/user/data")
Test 5
Check that the app can handle:
To test this:
run_ccvi_app("path/to/user/data")
The text was updated successfully, but these errors were encountered: