Bug 19776

Summary: Test failing randomly - fix categorycode vs category_type
Product: Koha Reporter: Jonathan Druart <jonathan.druart>
Component: Test SuiteAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: nick
Version: unspecified   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:
Bug Depends on: 19403, 19423, 19455, 19487, 19517, 19529    
Bug Blocks: 25551    
Attachments: Bug 19776: Fix random failures (category_type vs categorycode='X')
Bug 19776: (follow-up) remove dup category_type keys

Description Jonathan Druart 2017-12-07 17:40:48 UTC
From CanBookBeIssued:
 726     if ( $patron->category->category_type eq 'X' && (  $item->{barcode}  )) {

Tests are failing randomly if the category_type of the patron is 'X'.

The previous tries to fix this issue set categorycode='X' (QAers, what are you doing?)
Comment 1 Jonathan Druart 2017-12-07 17:43:07 UTC
Created attachment 69619 [details] [review]
Bug 19776: Fix random failures (category_type vs categorycode='X')

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 2 Jonathan Druart 2017-12-07 17:43:49 UTC
Pushed to master for 18.05
Comment 3 Jonathan Druart 2017-12-14 19:11:12 UTC
Created attachment 69805 [details] [review]
Bug 19776: (follow-up) remove dup category_type keys

The previous patch was obviously wrong, the category_type key appeared
twice.
categories.category_type is a varchar(1) and 'P' is not 'X'.

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 4 Jonathan Druart 2017-12-14 19:11:40 UTC
Last patch pushed to master.
Comment 5 Nick Clemens (kidclamp) 2017-12-20 11:44:44 UTC
Skipping for 17.11