Project

General

Profile

Actions

Misc #11770

closed

[PATCH] $KCODEへの代入時にwarningが発生することのテストを追加

Added by yui-knk (Kaneko Yuichiro) over 8 years ago. Updated over 8 years ago.


Description

testケースがなかったので、追加します。


Files

Updated by nobu (Nobuyoshi Nakada) over 8 years ago

どちらかと言えば、将来的には$KCODEの特別扱いをなしにしたい気持ちがあるので、ちょっと微妙な印象です。
それと、$=$KCODEとは別です。

Updated by yui-knk (Kaneko Yuichiro) over 8 years ago

将来的には$KCODEの特別扱いをなしにしたい

$KCODEを削除するという意味でしょうか?
現時点ではwarningがでることをテストしておいてもいいのかなと思っています。

$=は$KCODEとは別です。

誤解していました、ありがとうございます。
test_KCODE_warningtest_ignorecase_warningに分割します。
また細かい修正になりますが、以下のテストの3・4番目のasserttionをtest_ignorecaseに切り出したほうがいいですね。

  def test_KCODE
    assert_nil($KCODE)
    assert_nothing_raised { $KCODE = nil }
    assert_equal(false, $=)
    assert_nothing_raised { $= = nil }
  end

Updated by sorah (Sorah Fukumori) over 8 years ago

  • Assignee set to yui-knk (Kaneko Yuichiro)

Updated by naruse (Yui NARUSE) over 8 years ago

ご自分でコミットなさるのがよいのではないでしょうか。

よいですよね>matz
redmineのruby-coreグループには追加したので、その他の手続きお願いします>しばたさん

Updated by yui-knk (Kaneko Yuichiro) over 8 years ago

ありがとうございます:)

Updated by yui-knk (Kaneko Yuichiro) over 8 years ago

  • Status changed from Open to Closed
Actions

Also available in: Atom PDF

Like0
Like0Like0Like0Like0Like0Like0Like0Like0