Project

General

Profile

Bug #10342

Don't print test suite information to STDERR

Added by anonym over 4 years ago. Updated about 4 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Category:
Test suite
Target version:
Start date:
10/06/2015
Due date:
% Done:

100%

Feature Branch:
test/10151-organize-artifacts
Type of work:
Code
Blueprint:
Starter:
Affected tool:

Description

The formatters meant for human consumption (like pretty and our debug) will make use of things written to STDOUT and write it at an appropriate place, unlike STDERR which will only be printed to the terminal => this info doesn't end up in files when we log to them. We should eliminate all instances where we print to STDERR, perhaps using something like:

require 'cucumber/formatter/console'
def puts_info(m = "")
  # This trick allows us to use a module's (~private) method on a one-off basis.
  puts Class.new.extend(Cucumber::Formatter::Console).format_string(m, :white)
end


Related issues

Blocked by Tails - Feature #10151: Organize automated test suite artifacts in per-run directories Resolved 09/02/2015

Associated revisions

Revision 1284f3ad (diff)
Added by anonym over 4 years ago

Log useful information via Cucumber's formatters.

By writing to stderr such information is not logged to files when
using `--out`, but when writing to stdout (which `puts` does by
default) the human-readable formatters often do something smart (at
least 'pretty' and 'debug' do, which are the ones we care most about).

We only continue writing to stderr for the output of the options that
requires interaction, i.e. --retry-find and --pause-on-fail.

Will-fix: #10342

Revision cf79bcb8
Added by intrigeri about 4 years ago

Merge remote-tracking branch 'origin/test/10151-organize-artifacts' into testing

Fix-committed: #10151, #10342

History

#1 Updated by anonym over 4 years ago

  • Status changed from Confirmed to In Progress

#2 Updated by anonym over 4 years ago

  • Assignee changed from anonym to kytv
  • % Done changed from 0 to 50
  • QA Check set to Ready for QA
  • Feature Branch set to test/10151-organize-artifacts

I squeezed this work into #10151, which is dealing with slightly related stuff.

#3 Updated by anonym over 4 years ago

  • Related to Feature #10151: Organize automated test suite artifacts in per-run directories added

#4 Updated by kytv about 4 years ago

Code review passes. Testing…

#5 Updated by kytv about 4 years ago

  • Assignee changed from kytv to intrigeri

Code review and testing passed.

#6 Updated by intrigeri about 4 years ago

  • Related to deleted (Feature #10151: Organize automated test suite artifacts in per-run directories)

#7 Updated by intrigeri about 4 years ago

  • Blocked by Feature #10151: Organize automated test suite artifacts in per-run directories added

#8 Updated by intrigeri about 4 years ago

  • Assignee changed from intrigeri to anonym
  • QA Check changed from Ready for QA to Pass

Code review passes, but blocked by #10151.

#9 Updated by intrigeri about 4 years ago

  • Status changed from In Progress to 11
  • % Done changed from 50 to 100

#10 Updated by intrigeri about 4 years ago

  • Assignee deleted (anonym)

#11 Updated by anonym about 4 years ago

  • Status changed from 11 to Resolved

Also available in: Atom PDF