Created attachment 149243 [details] [review] Bug 33435: Add sample ERM data
Created attachment 149244 [details] [review] Bug 33435: Add sample ERM data Apply patch run reset_all Check ERM module and verify it has data in it
Was not able to set erm_user_roles because by the time sample_erm_data.yml is imported, no patrons exist in the database yet. Any quick way around this?
Created attachment 149248 [details] [review] Bug 33435: Add sample ERM data Apply patch run reset_all Check ERM module and verify it has data in it Signed-off-by: David Nind <david@davidnind.com>
(In reply to Pedro Amorim from comment #3) > Was not able to set erm_user_roles because by the time sample_erm_data.yml > is imported, no patrons exist in the database yet. Any quick way around this? For other bugs I have tested, if specific permissions are required then the test plan will normally say to set the specific permissions. Also, for most testing I use the koha user, and it has the super librarian permissions. Also, a great idea to add some sample ERM data! I was thinking the other day that having some sample data would make it much easier to test any ERM bugs.
Sounds like a great idea but perhaps it should go in the koha-misc4dev repo?
I agree, this is probably better placed in misc4dev. Perhaps Jonathan could guild you in migrating this patch to a merge-request for that?
Thank you David and Martin. Understood. However, should we decide on a standard way/place where dev sample data should be? (if not already) At the moment it seems it's split between core and misc4dev, and honestly I wasn't sure where to place it to begin with!
You are not wrong, traditionally we had sample libraries, sample users etc. that could be installed via the web installer. And I think they are really helpful if libraries just want to have a first look at Koha for initial testing. The lable examples are also very helpful if you are just starting out using the module. The ERM data is maybe a grey area? We haven't added new sample data (not configuration) to the installer in a while. The web installer makes it more generally available, the misc4dev only to devs. If we want to add to web installer, we should make some of the fields (the notes) translatable in the yaml file :)