Bug 14116

Summary: Silence noise t/Scrubber.t
Product: Koha Reporter: Mark Tompsett <mtompset>
Component: Test SuiteAssignee: Mark Tompsett <mtompset>
Status: CLOSED FIXED QA Contact: Galen Charlton <gmcharlt>
Severity: minor    
Priority: P5 - low CC: bgkriegel, jonathan.druart, tomascohen
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on:    
Bug Blocks: 8879    
Attachments: Bug 14116: Silence noisy output for t/Scrubber.
[SIGNED-OFF] Bug 14116: Silence noisy output for t/Scrubber.
Bug 14116: Silence noisy output for t/Scrubber.
Bug 14116: Clean up output, not just noise.
Bug 14116: Silence noisy output for t/Scrubber.
[SIGNED OFF] Bug 14116: Silence noisy output for t/Scrubber.
[PASSED QA] Bug 14116: Silence noisy output for t/Scrubber.

Description Mark Tompsett 2015-05-02 01:16:27 UTC
Some diag's are better suited as prints.
Comment 1 Mark Tompsett 2015-05-02 01:19:27 UTC Comment hidden (obsolete)
Comment 2 Bernardo Gonzalez Kriegel 2015-05-06 20:10:15 UTC Comment hidden (obsolete)
Comment 3 Jonathan Druart 2015-05-22 08:29:02 UTC
Actually I think we should just put these strings as comments.
I don't see any usefulness to print them.
Comment 4 Mark Tompsett 2015-05-22 17:42:31 UTC
(In reply to Jonathan Druart from comment #3)
> Actually I think we should just put these strings as comments.
> I don't see any usefulness to print them.

The point of the patch was to silence noise, not necessarily clean up output. ;)
Comment 5 Tomás Cohen Arazi 2015-05-22 18:21:26 UTC
(In reply to M. Tompsett from comment #4)
> (In reply to Jonathan Druart from comment #3)
> > Actually I think we should just put these strings as comments.
> > I don't see any usefulness to print them.
> 
> The point of the patch was to silence noise, not necessarily clean up
> output. ;)

It only silences a part of the noise. New patch needed /me thinks.
Comment 6 Mark Tompsett 2015-05-23 03:33:20 UTC
(In reply to Tomás Cohen Arazi from comment #5)
> It only silences a part of the noise. New patch needed /me thinks.

So, another commit on this bug? Or another bug report?
Comment 7 Mark Tompsett 2015-05-27 18:11:09 UTC Comment hidden (obsolete)
Comment 8 Mark Tompsett 2015-05-27 18:11:30 UTC Comment hidden (obsolete)
Comment 9 Mark Tompsett 2015-05-27 18:12:00 UTC
Added second commit that needs sign off.
Comment 10 Mark Tompsett 2015-05-27 18:16:24 UTC Comment hidden (obsolete)
Comment 11 Tomás Cohen Arazi 2015-05-27 18:41:20 UTC Comment hidden (obsolete)
Comment 12 Kyle M Hall 2015-05-29 15:39:20 UTC
Created attachment 39694 [details] [review]
[PASSED QA] Bug 14116: Silence noisy output for t/Scrubber.

Why diag or print out things, unless it is -v?
And why print anything except success or failure type messages?
This cleans up the output.

TEST PLAN
---------
1) $ prove t/Scrubber.t
   -- there's some noise.
2) apply patch
3) $ prove t/Scrubber.t
   -- noise issue resolved.
      Output is still reasonable for -v.
4) koha qa test tools

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 13 Tomás Cohen Arazi 2015-06-01 17:19:07 UTC
Patch pushed to master.

Thanks Mark!