Feature #4657: add option to hide skip messages on unit/test

• Status: Closed
• Assignee: -
• Target version: -

[sorah (Sorah Fukumori)]

最近test-allでSkipが多いので、skipのメッセージ表示を抑制するオプションを追加してはどうでしょう。

ちなみに、ruby-jaで以下のような流れがありました。

nurse: そういえば、うささんの今のtest-allのskipって、test-allの最後で表示されたほうが嬉しいの?
unak: まったくうれしくない。
nurse: 今表示されてるのも消えたほうがいい?
unak: skipの表示を抑制するオプションはほしい。

以下のパッチを適用するとtest/unit (test-all)に--hide-skipオプションが追加され、
--hide-skipオプションを付けて実行するとSkippedメッセージが表示されなくなります。

従来通り、Skip数は末尾のレポートで確認することができるようになっています。

出力例:

$ make TESTS='--hide-skip date' test-all
./miniruby -I../../lib -I. -I.ext/common ../../tool/runruby.rb --extout=.ext -- "../../test/runner.rb" --ruby="./miniruby -I../../lib -I. -I.ext/common ../../tool/runruby.rb --extout=.ext --" --hide-skip date
Run options: "--ruby=./miniruby -I../../lib -I. -I.ext/common ../../tool/runruby.rb --extout=.ext --" --hide-skip

Running tests:

..............................SSS....SSSSSSSSSSSSSSSS.SS........................................................................

Finished tests in 11.376260s, 11.2515 tests/s, 14151.3116 assertions/s.

128 tests, 160989 assertions, 0 failures, 0 errors, 21 skips

= patch below

diff --git lib/test/unit.rb lib/test/unit.rb
index 1f1bb09..e88309f 100644
--- lib/test/unit.rb
+++ lib/test/unit.rb
@@ -103,6 +103,10 @@ module Test
opts.on '--ruby VAL', "Path to ruby; It'll have used at -j option" do |a|
options[:ruby] = a.split(/ /).reject(&:empty?)
end
+

•    opts.on '--hide-skip', 'Hide skipped tests' do
  

•      options[:hide_skip] = true
  

•    end
   end
  
   def non_options(files, options)
  

@@ -547,6 +551,7 @@ module Test
end
}
end

•    report.reject!{|r| r.start_with? "Skipped:" } if @opts[:hide_skip]
     result
   end
  

diff --git test/testunit/test4test_hideskip.rb test/testunit/test4test_hideskip.rb
new file mode 100644
index 0000000..6fe3284
--- /dev/null
+++ test/testunit/test4test_hideskip.rb
@@ -0,0 +1,7 @@
+require 'test/unit'
+
+class TestForTestHideSkip < Test::Unit::TestCase

• def test_skip
• skip
• end
  +end
  diff --git test/testunit/test_hideskip.rb test/testunit/test_hideskip.rb
  new file mode 100644
  index 0000000..967ecaf
  --- /dev/null
  +++ test/testunit/test_hideskip.rb
  @@ -0,0 +1,20 @@
  +require 'test/unit'

+class TestHideSkip < Test::Unit::TestCase

• def test_hideskip
• test_out, o = IO.pipe
• spawn(*@options[:ruby], "#{File.dirname(FILE)}/test4test_hideskip.rb",

•      out: o, err: o)
  

• o.close
• assert_match(/assertions/s.\n\n 1) Skipped/,test_out.read)
• test_out.close
• test_out, o = IO.pipe
• spawn(*@options[:ruby], "#{File.dirname(FILE)}/test4test_hideskip.rb",

•      "--hide-skip", out: o, err: o)
  

• o.close
• assert_match(/assertions/s.\n\n1 tests, 0 assertions, 0 failures, 0 errors, 1 skips/,

•             test_out.read)
  

• test_out.close
• end
  +end

[shyouhei (Shyouhei Urabe)]

(05/09/2011 06:31 PM), Shota Fukumori wrote:

最近test-allでSkipが多いので、skipのメッセージ表示を抑制するオプションを追加してはどうでしょう。

賛成です。個人的にもtest-all結果からはskipをまず消してから読んでいます(という
スクリプトを書いてしまった程度には邪魔)。

[shyouhei (Shyouhei Urabe)]

(05/09/2011 06:31 PM), Shota Fukumori wrote:

最近test-allでSkipが多いので、skipのメッセージ表示を抑制するオプションを追加してはどうでしょう。

賛成です。個人的にもtest-all結果からはskipをまず消してから読んでいます(という
スクリプトを書いてしまった程度には邪魔)。

[sorah (Sorah Fukumori)]

2011/5/9 KOSAKI Motohiro [e-mail]:

(05/09/2011 06:31 PM), Shota Fukumori wrote:

最近test-allでSkipが多いので、skipのメッセージ表示を抑制するオプションを追加してはどうでしょう。

賛成です。個人的にもtest-all結果からはskipをまず消してから読んでいます(という
スクリプトを書いてしまった程度には邪魔)。

+1.

この skip メッセージを読みたい人はおそらく普段 make test-all してる人には
いないので、make test-all はデフォルトで抑制していただきたい。
これがマージされれば zlib などのテストで returnを使ってる箇所を skipに
変更することに対してもいいと思います。

デフォルト抑制についてはchkbuildへの影響など影響が多いことから今回はパスという流れでした。

--
Shota Fukumori a.k.a. @sora_h - http://codnote.net/

[sorah (Sorah Fukumori)]

2011/5/9 KOSAKI Motohiro [e-mail]:

(05/09/2011 06:31 PM), Shota Fukumori wrote:

最近test-allでSkipが多いので、skipのメッセージ表示を抑制するオプションを追加してはどうでしょう。

賛成です。個人的にもtest-all結果からはskipをまず消してから読んでいます(という
スクリプトを書いてしまった程度には邪魔)。

+1.

この skip メッセージを読みたい人はおそらく普段 make test-all してる人には
いないので、make test-all はデフォルトで抑制していただきたい。
これがマージされれば zlib などのテストで returnを使ってる箇所を skipに
変更することに対してもいいと思います。

デフォルト抑制についてはchkbuildへの影響など影響が多いことから今回はパスという流れでした。

--
Shota Fukumori a.k.a. @sora_h - http://codnote.net/

[sorah (Sorah Fukumori)]

2011/5/9 KOSAKI Motohiro [e-mail]:

デフォルト抑制についてはchkbuildへの影響など影響が多いことから今回はパスという流れでした。

具体的には、なにが起きますでしょうか？

とりあえずchkbuildには修正が必要なようです (naruseさん曰く)。
他のCIシステムなどはわかりません。

--
Shota Fukumori a.k.a. @sora_h - http://codnote.net/

[sorah (Sorah Fukumori)]

2011/5/9 KOSAKI Motohiro [e-mail]:

デフォルト抑制についてはchkbuildへの影響など影響が多いことから今回はパスという流れでした。

具体的には、なにが起きますでしょうか？

とりあえずchkbuildには修正が必要なようです (naruseさん曰く)。
他のCIシステムなどはわかりません。

--
Shota Fukumori a.k.a. @sora_h - http://codnote.net/

[naruse (Yui NARUSE)]

(2011/05/09 20:50), Shota Fukumori (sora_h) wrote:

2011/5/9 KOSAKI Motohiro [e-mail]:

デフォルト抑制についてはchkbuildへの影響など影響が多いことから今回はパスという流れでした。

具体的には、なにが起きますでしょうか？

とりあえずchkbuildには修正が必要なようです (naruseさん曰く)。
他のCIシステムなどはわかりません。

chkbuild には Skip の情報も見せるべきだと思っているので修正は必要でしょう。
が、1行の修正だろうので「影響が多い」とは思っていません。

まぁ、どうせおれらは現状でも TESTS=-v とかデフォルトと違うことをしているわけで、
おれらが --hide-skip って手元ではすればいいんじゃね、というような議論でした。

--
NARUSE, Yui [e-mail]

[naruse (Yui NARUSE)]

(2011/05/09 20:50), Shota Fukumori (sora_h) wrote:

2011/5/9 KOSAKI Motohiro [e-mail]:

デフォルト抑制についてはchkbuildへの影響など影響が多いことから今回はパスという流れでした。

具体的には、なにが起きますでしょうか？

とりあえずchkbuildには修正が必要なようです (naruseさん曰く)。
他のCIシステムなどはわかりません。

chkbuild には Skip の情報も見せるべきだと思っているので修正は必要でしょう。
が、1行の修正だろうので「影響が多い」とは思っていません。

まぁ、どうせおれらは現状でも TESTS=-v とかデフォルトと違うことをしているわけで、
おれらが --hide-skip って手元ではすればいいんじゃね、というような議論でした。

--
NARUSE, Yui [e-mail]

[sorah (Sorah Fukumori)]

• Status changed from Open to Closed

r31502 でコミットしました。

-q, --hide-skipでskipメッセージが表示されなくなります。

[whiteleaf (whiteleaf _)]

-q, --hide-skipでskipメッセージが表示されなくなります。

skip のメッセージを隠す「だけ」なのに -q という指定は違和感を感じるのですがどうなんでしょうか。
-q ってつけると全部のメッセージが消えるイメージが。

[whiteleaf (whiteleaf _)]

-q, --hide-skipでskipメッセージが表示されなくなります。

skip のメッセージを隠す「だけ」なのに -q という指定は違和感を感じるのですがどうなんでしょうか。
-q ってつけると全部のメッセージが消えるイメージが。

[usa (Usaku NAKAMURA)]

こんにちは、なかむら(う)です。

In message "[ruby-dev:43511] Re: [Ruby 1.9 - Feature #4657][Closed] add option to hide skip messages on unit/test"
on May.10,2011 15:24:02, [e-mail] wrote:

-q, --hide-skipでskipメッセージが表示されなくなります。

skip のメッセージを隠す「だけ」なのに -q という指定は違和感を感じるのですがどうなんでしょうか。
-q ってつけると全部のメッセージが消えるイメージが。

たしかに、TESTS="-v -q" って -v と -q が同時についていると一瞬違和感が
あるかもですねぇ。代替案はありますでしょうか？

実のところ、慣れたら気にならなくなるんじゃないかという気もしなくは
ないのですが。

慣れたら気にならなくなると思います。

そもそも、せっかくtestしてんのにメッセージを全部消したいなん
ていうニーズは存在しないわけで(とか断言しちゃう)、で、あれば、
-qが「全部消す」という意味でないことは明らかでしょう。

それでは。

U.Nakamura [e-mail]

[usa (Usaku NAKAMURA)]

こんにちは、なかむら(う)です。

In message "[ruby-dev:43511] Re: [Ruby 1.9 - Feature #4657][Closed] add option to hide skip messages on unit/test"
on May.10,2011 15:24:02, [e-mail] wrote:

-q, --hide-skipでskipメッセージが表示されなくなります。

skip のメッセージを隠す「だけ」なのに -q という指定は違和感を感じるのですがどうなんでしょうか。
-q ってつけると全部のメッセージが消えるイメージが。

たしかに、TESTS="-v -q" って -v と -q が同時についていると一瞬違和感が
あるかもですねぇ。代替案はありますでしょうか？

実のところ、慣れたら気にならなくなるんじゃないかという気もしなくは
ないのですが。

慣れたら気にならなくなると思います。

そもそも、せっかくtestしてんのにメッセージを全部消したいなん
ていうニーズは存在しないわけで(とか断言しちゃう)、で、あれば、
-qが「全部消す」という意味でないことは明らかでしょう。

それでは。

U.Nakamura [e-mail]