Tests were failing on upgrade route because they tried to create EDIFACT_SI authorised values that already existed from the atomicupdate script, causing unique constraint violations. This patch: - Removes test code that created EDIFACT_SI values (tests should assume database defaults exist) - Adds EDIFACT_SI category to auth_val_cat.sql for fresh installs - Adds all 32 EDItEUR List 3B servicing instruction codes to auth_values.yml for fresh installs Now the authorised values exist in both installation paths: - Upgrade route: Created by atomicupdate script - Fresh install: Created by installer SQL/YAML files Test plan: 1. prove t/db_dependent/Koha/EDI.t 2. prove t/db_dependent/Koha/Edifact/Order.t 3. Verify both tests pass without warnings Sponsored-by: Martin Renvoize <martin.renvoize@gmail.com> Signed-off-by: Hannah Dunne-Howrie <hdunne-howrie@westminster.gov.uk> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Signed-off-by: Pedro Amorim <pedro.amorim@openfifth.co.uk> |
||
|---|---|---|
| .. | ||
| account_credit_types.yml | ||
| account_debit_types.yml | ||
| auth_values.yml | ||
| class_sources.yml | ||
| illbatch_statuses.yml | ||
| patron_restriction_types.yml | ||
| sample_frequencies.yml | ||
| sample_notices.yml | ||
| sample_numberpatterns.yml | ||