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

• Status: Closed
• Assignee: yui-knk (Kaneko Yuichiro)

[yui-knk (Kaneko Yuichiro)]

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

[nobu (Nobuyoshi Nakada)]

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

[yui-knk (Kaneko Yuichiro)]

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

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

$=は$KCODEとは別です。

誤解していました、ありがとうございます。
test_KCODE_warningとtest_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

[yui-knk (Kaneko Yuichiro)]

• File 0001-Add-test-cases-for-KCODE-and-warning.patch 0001-Add-test-cases-for-KCODE-and-warning.patch added

[sorah (Sorah Fukumori)]

• Assignee set to yui-knk (Kaneko Yuichiro)

[naruse (Yui NARUSE)]

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

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

[matz (Yukihiro Matsumoto)]

OK.

Matz.

[yui-knk (Kaneko Yuichiro)]

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

[yui-knk (Kaneko Yuichiro)]

• Status changed from Open to Closed