Project

General

Profile

Feature #4657

add option to hide skip messages on unit/test

Added by sorah (Sorah Fukumori) over 9 years ago. Updated over 9 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Target version:
-
[ruby-dev:43493]

Description

最近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

Updated by shyouhei (Shyouhei Urabe) over 9 years ago

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

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

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

Updated by shyouhei (Shyouhei Urabe) over 9 years ago

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

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

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

Updated by sorah (Sorah Fukumori) over 9 years ago

2011/5/9 KOSAKI Motohiro kosaki.motohiro@jp.fujitsu.com:

(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/

Updated by sorah (Sorah Fukumori) over 9 years ago

2011/5/9 KOSAKI Motohiro kosaki.motohiro@jp.fujitsu.com:

(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/

Updated by sorah (Sorah Fukumori) over 9 years ago

2011/5/9 KOSAKI Motohiro kosaki.motohiro@jp.fujitsu.com:

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

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

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

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

Updated by sorah (Sorah Fukumori) over 9 years ago

2011/5/9 KOSAKI Motohiro kosaki.motohiro@jp.fujitsu.com:

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

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

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

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

Updated by naruse (Yui NARUSE) over 9 years ago

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

2011/5/9 KOSAKI Motohiro kosaki.motohiro@jp.fujitsu.com:

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

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

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

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

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

--
NARUSE, Yui naruse@airemix.jp

Updated by naruse (Yui NARUSE) over 9 years ago

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

2011/5/9 KOSAKI Motohiro kosaki.motohiro@jp.fujitsu.com:

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

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

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

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

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

--
NARUSE, Yui naruse@airemix.jp

Updated by sorah (Sorah Fukumori) over 9 years ago

  • Status changed from Open to Closed

r31502 でコミットしました。

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

Updated by whiteleaf (whiteleaf _) over 9 years ago

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

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

Updated by whiteleaf (whiteleaf _) over 9 years ago

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

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

Updated by usa (Usaku NAKAMURA) over 9 years ago

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

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, kosaki.motohiro@jp.fujitsu.com wrote:

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

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

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

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

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

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

それでは。
--
U.Nakamura usa@garbagecollect.jp

Updated by usa (Usaku NAKAMURA) over 9 years ago

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

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, kosaki.motohiro@jp.fujitsu.com wrote:

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

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

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

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

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

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

それでは。
--
U.Nakamura usa@garbagecollect.jp

Also available in: Atom PDF