Bug 13673

Summary: Remove warnings from t/db_dependent/Barcodes.t
Product: Koha Reporter: Tomás Cohen Arazi <tomascohen>
Component: Test SuiteAssignee: Mark Tompsett <mtompset>
Status: CLOSED FIXED QA Contact: Tomás Cohen Arazi <tomascohen>
Severity: trivial    
Priority: P5 - low CC: indradg, mtompset, tomascohen
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 13673: Fix whitespace issues
Bug 13673: Fix whitespace issues
Bug 13673: Remove diags
Bug 13673: Silence noisy warnings in t/db_dependent/Barcodes.t
Bug 13758: (QA followup) revert case change that broke the tests
Bug 13673: Remove diags
Bug 13673: Remove diags
Bug 13673: Fix whitespace issues
Bug 13673: Fix whitespace issues
Bug 13673: Remove diags
Bug 13673: Silence noisy warnings in t/db_dependent/Barcodes.t
Bug 13673: Silence noisy warnings in t/db_dependent/Barcodes.t
Bug 13673: Fix whitespace issues
Bug 13673: Remove diags
Bug 13673: Silence noisy warnings in t/db_dependent/Barcodes.t
[PASSED QA] Bug 13673: Fix whitespace issues
[PASSED QA] Bug 13673: Remove diags
[PASSED QA] Bug 13673: Silence noisy warnings in t/db_dependent/Barcodes.t
Bug 13673: (QA followup) Add license

Description Tomás Cohen Arazi 2015-02-05 18:34:43 UTC

    
Comment 1 Tomás Cohen Arazi 2015-02-05 18:35:24 UTC
All diags should be removed.
Comment 2 Mark Tompsett 2015-05-08 20:28:33 UTC Comment hidden (obsolete)
Comment 3 Mark Tompsett 2015-05-08 20:32:16 UTC Comment hidden (obsolete)
Comment 4 Mark Tompsett 2015-05-08 20:49:51 UTC Comment hidden (obsolete)
Comment 5 Mark Tompsett 2015-05-08 21:19:37 UTC Comment hidden (obsolete)
Comment 6 Indranil Das Gupta 2015-05-09 13:46:15 UTC Comment hidden (obsolete)
Comment 7 Indranil Das Gupta 2015-05-09 14:24:36 UTC Comment hidden (obsolete)
Comment 8 Indranil Das Gupta 2015-05-09 14:27:50 UTC Comment hidden (obsolete)
Comment 9 Mark Tompsett 2015-05-09 14:48:46 UTC Comment hidden (obsolete)
Comment 10 Mark Tompsett 2015-05-09 14:49:42 UTC Comment hidden (obsolete)
Comment 11 Mark Tompsett 2015-05-09 14:49:47 UTC Comment hidden (obsolete)
Comment 12 Mark Tompsett 2015-05-09 14:49:52 UTC Comment hidden (obsolete)
Comment 13 Indranil Das Gupta 2015-05-10 05:17:46 UTC Comment hidden (obsolete)
Comment 14 Indranil Das Gupta 2015-05-10 05:25:54 UTC Comment hidden (obsolete)
Comment 15 Indranil Das Gupta 2015-05-10 05:26:28 UTC Comment hidden (obsolete)
Comment 16 Indranil Das Gupta 2015-05-10 05:27:05 UTC Comment hidden (obsolete)
Comment 17 Indranil Das Gupta 2015-05-10 05:33:00 UTC
1/ All three test plans were followed. 
2/ All three patches cleared koha-qa.pl -c 1 -v 2 without errors.
Comment 18 Tomás Cohen Arazi 2015-05-15 18:58:54 UTC
Created attachment 39220 [details] [review]
[PASSED QA] Bug 13673: Fix whitespace issues

Indentation was horrible. Fixed to meet 4 space standard.

TEST PLAN
---------
1) Notice tabs in t/db_dependent/Barcodes.t
2) $ prove t/db_dependent/Barcodes.t
3) apply patch
4) $ prove t/db_dependent/Barcodes.t
   -- same output as before.
5) $ git diff -w origin/master
   -- code is identical except for indentations.
6) koha qa test tools

Signed-off-by: Indranil Das Gupta <indradg@gmail.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
Comment 19 Tomás Cohen Arazi 2015-05-15 18:59:03 UTC
Created attachment 39221 [details] [review]
[PASSED QA] Bug 13673: Remove diags

TEST PLAN
---------
1) Apply first two patches
2) $ prove t/db_dependent/Barcodes.t
   -- Diags have been removed, just noisy warnings left.
3) $ git diff -w origin/master
   -- only excess print's and diag's removed.
4) koha qa test tools

Signed-off-by: Indranil Das Gupta <indradg@gmail.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
Comment 20 Tomás Cohen Arazi 2015-05-15 18:59:13 UTC
Created attachment 39222 [details] [review]
[PASSED QA] Bug 13673: Silence noisy warnings in t/db_dependent/Barcodes.t

This last patch catches all the warning messages in tests.

TEST PLAN
---------
1) Apply first two patches
2) $ prove t/db_dependent/Barcodes.t
   -- still noisy warnings.
3) apply last patch
4) $ prove t/db_dependent/Barcodes.t
   -- All noisy warnings are caught.
5) koha qa test tools.

Signed-off-by: Indranil Das Gupta <indradg@gmail.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
Comment 21 Tomás Cohen Arazi 2015-05-15 18:59:28 UTC
Created attachment 39223 [details] [review]
Bug 13673: (QA followup) Add license

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
Comment 22 Tomás Cohen Arazi 2015-05-15 19:03:48 UTC
Patches pushed to master.

Thanks Mark!