Bug 15160

Summary: GetTopIssues.t should create random data
Product: Koha Reporter: Tomás Cohen Arazi <tomascohen>
Component: Test SuiteAssignee: Chris Cormack <chris>
Status: CLOSED FIXED QA Contact: Galen Charlton <gmcharlt>
Severity: enhancement    
Priority: P5 - low CC: tomascohen
Version: master   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 15160: GetTopIssues.t should create random data
Bug 15160: GetTopIssues.t should create random data
Bug 15160: GetTopIssues.t should create random data

Description Tomás Cohen Arazi 2015-11-09 14:35:58 UTC
GetTopIssues.t currently creates fixed-value objects on the DB. If the id's it tries to use already exist on the DB it would fail.

TestBuilder should take care of that.
Comment 1 Tomás Cohen Arazi 2015-11-09 14:42:30 UTC Comment hidden (obsolete)
Comment 2 Frédéric Demians 2015-11-10 09:03:14 UTC Comment hidden (obsolete)
Comment 3 Jonathan Druart 2015-11-10 10:02:07 UTC
Created attachment 44687 [details] [review]
Bug 15160: GetTopIssues.t should create random data

This patch refactors t/db_dependent/Circulation/GetTopIssues.t so it
creates its own data with random values. It does so by replacing (legacy)
SQL inserts and fixed value data inserting by TestBuilder generated data.

There is no expected behaviour change.

To test:
- Create an itemtyp directly on your DB with GTI_I_TEST as its itemtype
  > INSERT INTO itemtypes (itemtype) VALUES ('GTI_I_TEST');
- Run:
  $ prove t/db_dependent/Circulation/GetTopIssues.t
=> FAIL: The insert line fails
- Apply the patch
- Run:
  $ prove t/db_dependent/Circulation/GetTopIssues.t
=> SUCCESS: tests pass due to random data usage.
- Sign off

Signed-off-by: Frederic Demians <f.demians@tamil.fr>
  Test plan produces the expected result, failing, then OK.

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 4 Tomás Cohen Arazi 2015-11-10 12:49:34 UTC
Patch pushed to master.