From b37319295dd921fe3d679f7221fda6a151da626b Mon Sep 17 00:00:00 2001 From: kou Date: Fri, 19 Nov 2004 08:49:05 +0000 Subject: * lib/rss/maker/0.9.rb (RSS::Maker::RSS09::Image#have_required_values): changed /rss/channel/image to be optional. [ruby-Bugs:1047] * test/rss/test_maker_0.9.rb: added tests for the above. git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@7320 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- ChangeLog | 7 +++++++ lib/rss/maker/0.9.rb | 3 +-- test/rss/test_maker_0.9.rb | 40 +++------------------------------------- 3 files changed, 11 insertions(+), 39 deletions(-) diff --git a/ChangeLog b/ChangeLog index 986045d9dd..35e3d63969 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,10 @@ +Fri Nov 19 17:46:56 2004 Kouhei Sutou + + * lib/rss/maker/0.9.rb (RSS::Maker::RSS09::Image#have_required_values): + changed /rss/channel/image to be optional. [ruby-Bugs:1047] + + * test/rss/test_maker_0.9.rb: added tests for the above. + Fri Nov 19 17:18:17 2004 Kouhei Sutou * lib/rss/rss.rb (RSS::VERSION): 0.1.0 -> 0.1.1. diff --git a/lib/rss/maker/0.9.rb b/lib/rss/maker/0.9.rb index 6f26e20f4f..3dd987a683 100644 --- a/lib/rss/maker/0.9.rb +++ b/lib/rss/maker/0.9.rb @@ -48,8 +48,7 @@ module RSS end def have_required_values? - @title and @link and @description and @language and - @maker.image.have_required_values? + @title and @link and @description and @language end private diff --git a/test/rss/test_maker_0.9.rb b/test/rss/test_maker_0.9.rb index 5f356cc8c4..22cff625c0 100644 --- a/test/rss/test_maker_0.9.rb +++ b/test/rss/test_maker_0.9.rb @@ -11,13 +11,11 @@ module RSS rss = RSS::Maker.make("0.9") do |maker| setup_dummy_channel(maker) - setup_dummy_image(maker) end assert_equal("0.91", rss.rss_version) rss = RSS::Maker.make("0.9") do |maker| setup_dummy_channel(maker) - setup_dummy_image(maker) maker.encoding = "EUC-JP" end @@ -26,7 +24,6 @@ module RSS rss = RSS::Maker.make("0.9") do |maker| setup_dummy_channel(maker) - setup_dummy_image(maker) maker.standalone = "yes" end @@ -35,7 +32,6 @@ module RSS rss = RSS::Maker.make("0.9") do |maker| setup_dummy_channel(maker) - setup_dummy_image(maker) maker.encoding = "EUC-JP" maker.standalone = "yes" @@ -87,8 +83,6 @@ module RSS new_hour = maker.channel.skipHours.new_hour new_hour.content = hour end - - setup_dummy_image(maker) end channel = rss.channel @@ -112,7 +106,7 @@ module RSS end assert(channel.items.empty?) - assert_not_nil(channel.image) + assert_nil(channel.image) assert_nil(channel.textInput) end @@ -123,18 +117,6 @@ module RSS language = "ja" rss = RSS::Maker.make("0.9") do |maker| - # setup_dummy_image(maker) - - maker.channel.title = title - maker.channel.link = link - maker.channel.description = description - maker.channel.language = language - end - assert_nil(rss) - - rss = RSS::Maker.make("0.9") do |maker| - setup_dummy_image(maker) - # maker.channel.title = title maker.channel.link = link maker.channel.description = description @@ -143,8 +125,6 @@ module RSS assert_nil(rss) rss = RSS::Maker.make("0.9") do |maker| - setup_dummy_image(maker) - maker.channel.title = title # maker.channel.link = link maker.channel.link = nil @@ -154,8 +134,6 @@ module RSS assert_nil(rss) rss = RSS::Maker.make("0.9") do |maker| - setup_dummy_image(maker) - maker.channel.title = title maker.channel.link = link # maker.channel.description = description @@ -164,8 +142,6 @@ module RSS assert_nil(rss) rss = RSS::Maker.make("0.9") do |maker| - setup_dummy_image(maker) - maker.channel.title = title maker.channel.link = link maker.channel.description = description @@ -231,7 +207,7 @@ module RSS maker.image.height = height maker.image.description = description end - assert_nil(rss) + assert_nil(rss.channel.image) rss = RSS::Maker.make("0.9") do |maker| setup_dummy_channel(maker) @@ -256,7 +232,7 @@ module RSS maker.image.height = height maker.image.description = description end - assert_nil(rss) + assert_nil(rss.channel.image) end def test_items @@ -266,13 +242,11 @@ module RSS rss = RSS::Maker.make("0.9") do |maker| setup_dummy_channel(maker) - setup_dummy_image(maker) end assert(rss.channel.items.empty?) rss = RSS::Maker.make("0.9") do |maker| setup_dummy_channel(maker) - setup_dummy_image(maker) item = maker.items.new_item item.title = title @@ -289,7 +263,6 @@ module RSS item_size = 5 rss = RSS::Maker.make("0.9") do |maker| setup_dummy_channel(maker) - setup_dummy_image(maker) item_size.times do |i| item = maker.items.new_item @@ -308,7 +281,6 @@ module RSS rss = RSS::Maker.make("0.9") do |maker| setup_dummy_channel(maker) - setup_dummy_image(maker) item_size.times do |i| item = maker.items.new_item @@ -336,7 +308,6 @@ module RSS rss = RSS::Maker.make("0.9") do |maker| setup_dummy_channel(maker) - setup_dummy_image(maker) maker.textinput.title = title maker.textinput.description = description @@ -351,7 +322,6 @@ module RSS rss = RSS::Maker.make("0.9") do |maker| # setup_dummy_channel(maker) - setup_dummy_image(maker) maker.textinput.title = title maker.textinput.description = description @@ -369,7 +339,6 @@ module RSS rss = RSS::Maker.make("0.9") do |maker| setup_dummy_channel(maker) - setup_dummy_image(maker) # maker.textinput.title = title maker.textinput.description = description @@ -380,7 +349,6 @@ module RSS rss = RSS::Maker.make("0.9") do |maker| setup_dummy_channel(maker) - setup_dummy_image(maker) maker.textinput.title = title # maker.textinput.description = description @@ -391,7 +359,6 @@ module RSS rss = RSS::Maker.make("0.9") do |maker| setup_dummy_channel(maker) - setup_dummy_image(maker) maker.textinput.title = title maker.textinput.description = description @@ -402,7 +369,6 @@ module RSS rss = RSS::Maker.make("0.9") do |maker| setup_dummy_channel(maker) - setup_dummy_image(maker) maker.textinput.title = title maker.textinput.description = description -- cgit v1.2.3