Description
Mark Tompsett
2016-02-10 01:13:09 UTC
Created attachment 47941 [details] [review] Bug 15777 - Test to prove initialization isn't working correctly. TEST PLAN --------- 1) Apply this testing patch 2) prove -v t/RecordProcessor.t -- tests will fail. 3) run koha qa test tools Created attachment 47942 [details] [review] Bug 15777 - Fix base class to prove refactor and fix required TEST PLAN --------- 1) Apply the testing patch and this patch 2) prove -v t/RecordProcess.t -- will complain about string use with "use strict". This is because it is using the module name, instead of the object instantiated with new(). 3) run koha qa test tools. Created attachment 47943 [details] [review] Bug 15777 - Refactor and fix the RecordProcessor class TEST PLAN --------- 1) Apply all three patches 2) prove -v t/RecordProcessor.t -- all should be successful 3) run koha qa test tools Created attachment 47978 [details] [review] Bug 15777 - Test to prove initialization isn't working correctly. TEST PLAN --------- 1) Apply this testing patch 2) prove -v t/RecordProcessor.t -- tests will fail. 3) run koha qa test tools Signed-off-by: Marc Véron <veron@veron.ch> Created attachment 47979 [details] [review] Bug 15777 - Fix base class to prove refactor and fix required TEST PLAN --------- 1) Apply the testing patch and this patch 2) prove -v t/RecordProcess.t -- will complain about string use with "use strict". This is because it is using the module name, instead of the object instantiated with new(). 3) run koha qa test tools. Signed-off-by: Marc Véron <veron@veron.ch> Created attachment 47980 [details] [review] Bug 15777 - Refactor and fix the RecordProcessor class TEST PLAN --------- 1) Apply all three patches 2) prove -v t/RecordProcessor.t -- all should be successful 3) run koha qa test tools Signed-off-by: Marc Véron <veron@veron.ch> Created attachment 47988 [details] [review] Bug 15777 - Test to prove initialization isn't working correctly. TEST PLAN --------- 1) Apply this testing patch 2) prove -v t/RecordProcessor.t -- tests will fail. 3) run koha qa test tools Signed-off-by: Marc Véron <veron@veron.ch> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Created attachment 47990 [details] [review] Bug 15777 - Fix base class to prove refactor and fix required TEST PLAN --------- 1) Apply the testing patch and this patch 2) prove -v t/RecordProcess.t -- will complain about string use with "use strict". This is because it is using the module name, instead of the object instantiated with new(). 3) run koha qa test tools. Signed-off-by: Marc Véron <veron@veron.ch> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Created attachment 47991 [details] [review] Bug 15777 - Refactor and fix the RecordProcessor class TEST PLAN --------- 1) Apply all three patches 2) prove -v t/RecordProcessor.t -- all should be successful 3) run koha qa test tools Signed-off-by: Marc Véron <veron@veron.ch> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Created attachment 48000 [details] [review] Bug 15777 - Refactor and fix the RecordProcessor class TEST PLAN --------- 1) Apply all three patches 2) prove -v t/RecordProcessor.t -- all should be successful 3) run koha qa test tools Signed-off-by: Marc Véron <veron@veron.ch> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Sorry... git bz attach gone awry. Nothing changed. pushed to Master - Should be in the May 2016 Release. Thanks Patches pushed to 3.22.x, will be in 3.22.4 |