[ruby-dev:34498] Re: ERB::Util#url_encode bug fix

須藤です。

本題と関係ないところなんですが。。。

In <[e-mail]>
  "[ruby-dev:34497] ERB::Util#url_encode bug fix" on Wed, 23 Apr 2008 22:30:51 +0900,
  [e-mail] wrote:

> ERB::Util#url_encodeが日本語文字列で正しい結果にならないバグを修正しました。

> Index: test/erb/test_erb.rb
> ===================================================================
> --- test/erb/test_erb.rb	(revision 16174)
> +++ test/erb/test_erb.rb	(working copy)
> @@ -411,3 +411,18 @@
>     assert_equal(ans, ERB.new(src, nil, '-%').result)
>    end
>  end
> +
> +class TestERBUtil < Test::Unit::TestCase
> +  include ERB::Util
> +
> +  def test_01
> +    assert_equal("Programming%20Ruby%3A%20%20The%20Pragmatic%20Programmer%27s%20Guide",
> +      url_encode("Programming Ruby:  The Pragmatic Programmer's Guide"))
> +  end
> +
> +  def test_02
> +    assert_equal("%A5%B5%A5%F3%A5%D7%A5%EB",
> +      url_encode("\xA5\xB5\xA5\xF3\xA5\xD7\xA5\xEB".force_encoding("EUC-JP")))
> +  end
> +  
> +end

おそらく、TestERBCoreが01とかついていたのでそうしたのだと思う
のですが、もう少し意味のあるメソッド名を使った方がよい気がし
ます。（TestERBCoreのメソッド名は直した方がよいと思う。）

おそらく、ちょっと不思議なインデントをしているのは文字列の位
置を揃えたいからだと思うのですが、新しくassertなんとかを定義
して普通のインデントをした方がよい気がします。

  def assert_url_encode(expected, input)
    assert_equal(expected, url_encode(input))
  end

  def test_XXX
    assert_url_encode("%A5%B5%A5%F3%A5%D7%A5%EB",
                      "\xA5\xB5\xA5\xF3\xA5\xD7\xA5\xEB".force_encoding("EUC-JP"))
  end