Bug 35985 - ./t/db_dependent/check_sysprefs.t --showsql since Bug 34979
Summary: ./t/db_dependent/check_sysprefs.t --showsql since Bug 34979
Status: NEW
Alias: None
Product: Koha
Classification: Unclassified
Component: Installation and upgrade (command-line installer) (show other bugs)
Version: Main
Hardware: All All
: P5 - low normal (vote)
Assignee: Bugs List
QA Contact: Testopia
URL:
Keywords:
Depends on: 34979
Blocks:
  Show dependency treegraph
 
Reported: 2024-02-02 08:21 UTC by Fridolin Somers
Modified: 2024-02-02 08:37 UTC (History)
0 users

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Fridolin Somers 2024-02-02 08:21:20 UTC
With Bug 34979 looks like we need to adapt ./t/db_dependent/check_sysprefs.t --showsql

Example :
kohadev-koha@kohadevbox:koha(master)$ ./t/db_dependent/check_sysprefs.t --showsql
1..2
# Subtest: Compare database with sysprefs.sql file
('MARCOrgCode','OSt','','Define MARC Organization Code for MARC21 records - http://www.loc.gov/marc/organizations/orgshome.html','free'),
    1..0
    # No tests run!
not ok 1 - No tests run for subtest "Compare database with sysprefs.sql file"
#   Failed test 'No tests run for subtest "Compare database with sysprefs.sql file"'
#   at ./t/db_dependent/check_sysprefs.t line 53.
# Subtest: Compare sysprefs.sql with YAML files
    1..2
    ok 1 - No system preference entries missing from sysprefs.sql
    ok 2 - No system preference entries missing from YAML files
ok 2 - Compare sysprefs.sql with YAML files
# Looks like you failed 1 test of 2.

Too much verbose.

Should we skip subtest "Compare database with sysprefs.sql file" ?
Comment 1 Katrin Fischer 2024-02-02 08:37:28 UTC
I wrote the patch, but the different comparisons are confusing :( Happy to have some help with the fix really. I think the --showsql is to help fix an already found problem, so it probably makes sense to skip tests when the script is run with this parameter.