[#38323] [1.8.7][1.9.1][tk] 自前実装の拡張 widget を使いたい場合 — oshida@...

押田です。

22 messages 2009/04/24
[#38331] Re: [1.8.7][1.9.1][tk] 自前実装の拡張 widget を使いたい場合 — Hidetoshi NAGAI <nagai@...> 2009/04/26

永井@知能.九工大です.

[#38339] Re: [1.8.7][1.9.1][tk] 自前実装の拡張 widget を使いたい場合 — oshida@... 2009/04/27

押田です。

[#38340] Re: [1.8.7][1.9.1][tk] 自前実装の拡張 widget を使いたい場合 — Hidetoshi NAGAI <nagai@...> 2009/04/27

永井@知能.九工大です.

[#38697] Re: [1.8.7][1.9.1][tk] 自前実装の拡張 widget を使いたい場合 — Hidetoshi NAGAI <nagai@...> 2009/06/21

永井@知能.九工大です.

[#38711] Re: [1.8.7][1.9.1][tk] 自前実装の拡張 widget を使いたい場合 — oshida@... 2009/06/24

押田です。

[#38723] Re: [1.8.7][1.9.1][tk] 自前実装の拡張 widget を使いたい場合 — Hidetoshi NAGAI <nagai@...> 2009/07/01

永井@知能.九工大です.

[#38743] Re: [1.8.7][1.9.1][tk] 自前実装の拡張 widget を使いたい場合 — oshida@... 2009/07/07

押田です。

[#38747] Re: [1.8.7][1.9.1][tk] 自前実装の拡張 widget を使いたい場合 — Hidetoshi NAGAI <nagai@...> 2009/07/08

永井@知能.九工大です.

[#38748] Re: [1.8.7][1.9.1][tk] 自前実装の拡張 widget を使いたい場合 — oshida@... 2009/07/08

押田です。

[#38749] Re: [1.8.7][1.9.1][tk] 自前実装の拡張 widget を使いたい場合 — Hidetoshi NAGAI <nagai@...> 2009/07/08

永井@知能.九工大です.

[ruby-dev:38249] Re: 1.8.7 RSS test failure

From: Kouhei Sutou <kou@...>
Date: 2009-04-02 03:31:16 UTC
List: ruby-dev #38249
須藤です。

2009/04/02 11:40 Urabe Shyouhei <shyouhei@ruby-lang.org>:

> 1.8.7をリリースできてない理由の一つにRSSのテストが通らないからというのがあります(下記)。
> これは1.8.8devからテストだけ持ってきたからなので、なんらか対応しないといけないのですが、
> (1) このテストが正しくて1.8.7の挙動は誤りなのでしょうか、それとも
> (2) 1.8.7は誤りとは言えず、テストをrevertすべきなのでしょうか。

AtomではBase64のデータ内に改行が入ってもOKなので、実装もテストもどちらもAtom的には正
しい値です。なので、Atom的にはテストが改行を無視するようにもう少し寛容になるのがよいと
思います。ただ、RSS Parserのテストとしては、現在のようにどちらか一方を決め打ちでよいと思
います。

ということで、
(3) r14303のlib/rss/以下の部分を適用し、テストが期待しているように改行のないBase64のデータを
    返すようにする(実装の挙動を変える: 改行無しBase64)
(4) テストで改行を削除しないようにする(実装の挙動は変えない: 改行付きBase64)
のどちらかがよいと思います。

Index: test/rss/rss-assertions.rb
===================================================================
--- test/rss/rss-assertions.rb	(revision 23116)
+++ test/rss/rss-assertions.rb	(working copy)
@@ -564,7 +564,7 @@

         png_file = File.join(File.dirname(__FILE__), "dot.png")
         png = File.open(png_file, "rb") {|file| file.read}
-        base64_content = [png].pack("m").delete("\n")
+        base64_content = [png].pack("m")

         [false, true].each do |with_space|
           xml_content = base64_content

個人的には(4)がよいと思います。

In This Thread