add gender data and educational level selections
Why is this change needed?
- To not install
spp_base_demowhenspp_farmer_registry_demois installed. - To fix the problem with the Highest Educational Level.
How was the change implemented?
- Removed
spp_base_demoonspp_farmer_registry_demodepends, and add gender data xml. - Added the same selection values from
res_partnertospp_farmer.
New unit tests
None
Unit tests executed by the author
None
How to test manually
- Install
spp_farmer_registry_demoon new instance or make surespp_base_demois not installed by uninstallingspp_farmer_registry_demofirst.
Test 1
- Check if
spp_base_demois installed (should not be installed automatically).
Test 2
- Go to Configuration > Gender Types.
- Check if Male and Female are existing in the Gender Types.
Test 3
- Go to Group > Select or Open one.
- Check if Farmer Highest Educational Level selections is the same with Individual's Highest Educational Level.
Related links
https://github.com/orgs/OpenSPP/projects/5/views/4?pane=issue&itemId=81324763
Sorry for the message of the latest commit, it was supposed to be "remove user data for program in spp_base_demo".
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 73.23%. Comparing base (
c2d1b67) to head (098063e). Report is 7 commits behind head on 17.0.
Additional details and impacted files
@@ Coverage Diff @@
## 17.0 #617 +/- ##
==========================================
- Coverage 73.50% 73.23% -0.28%
==========================================
Files 560 544 -16
Lines 14084 13623 -461
Branches 1799 1750 -49
==========================================
- Hits 10353 9977 -376
+ Misses 3414 3348 -66
+ Partials 317 298 -19
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
TEST 1: Upon checking spp_base_demo is installed (should not be installed automatically). spp_base_demo was only installed after adding spp_farmer_registry_demo. Regarding this, @reichie020212 @Markpenalosa should this be the expected? In the HOW TO TEST MANUALLY is was specified that:
Install spp_farmer_registry_demo on new instance or make sure spp_base_demo is not installed by uninstalling spp_farmer_registry_demo first.
-
spp_farmer_registry_demo not yet installed in a new instance
-
spp_farmer_registry_demo already installed in the instance
TEST 2:
- Under Configuration >> Gender Types (Male and Female are NOT on the list) .
TEST 3:
-
Individual highest education status:
-
Farmer highest education status:
Quality Gate failed
Failed conditions
21 Security Hotspots
C Reliability Rating on New Code (required ≥ A)
See analysis details on SonarCloud
Catch issues before they fail your Quality Gate with our IDE extension
SonarLint