Project

General

Profile

Misc #11770

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

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

Status:
Closed
Priority:
Normal
[ruby-dev:49398]

Description

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


Files

0001-Add-test-cases-for-KCODE-warning.patch (749 Bytes) 0001-Add-test-cases-for-KCODE-warning.patch yui-knk (Kaneko Yuichiro), 12/04/2015 02:10 AM
0001-Add-test-cases-for-KCODE-and-warning.patch (1005 Bytes) 0001-Add-test-cases-for-KCODE-and-warning.patch yui-knk (Kaneko Yuichiro), 12/05/2015 01:09 PM

Updated by nobu (Nobuyoshi Nakada) over 4 years ago

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

Updated by yui-knk (Kaneko Yuichiro) over 4 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 4 years ago

  • Assignee set to yui-knk (Kaneko Yuichiro)

Updated by naruse (Yui NARUSE) over 4 years ago

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

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

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

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

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

  • Status changed from Open to Closed

Also available in: Atom PDF