feedjira 2.0.0 → 2.1.0

data/spec/feedjira/parser/atom_youtube_entry_spec.rb

@@ -0,0 +1,86 @@
+require File.join(File.dirname(__FILE__), %w(.. .. spec_helper))
+
+describe Feedjira::Parser::AtomYoutubeEntry do
+  describe 'parsing' do
+    before do
+      @feed = Feedjira::Parser::AtomYoutube.parse(sample_youtube_atom_feed)
+      @entry = @feed.entries.first
+    end
+
+    it 'should have the title' do
+      expect(@entry.title).to eq 'The Google app: Questions Title'
+    end
+
+    it 'should have the url' do
+      expect(@entry.url).to eq 'http://www.youtube.com/watch?v=5shykyfmb28'
+    end
+
+    it 'should have the entry id' do
+      expect(@entry.entry_id).to eq 'yt:video:5shykyfmb28'
+    end
+
+    it 'should have the published date' do
+      expect(@entry.published).to eq Time.parse_safely('2015-05-04T00:01:27+00:00') # rubocop:disable Metrics/LineLength
+    end
+
+    it 'should have the updated date' do
+      expect(@entry.updated).to eq Time.parse_safely('2015-05-13T17:38:30+00:00') # rubocop:disable Metrics/LineLength
+    end
+
+    it 'should have the content populated from the media:description element' do
+      expect(@entry.content).to eq 'A question is the most powerful force in the world. It can start you on an adventure or spark a connection. See where a question can take you. The Google app is available on iOS and Android. Download the app here: http://www.google.com/search/about/download' # rubocop:disable Metrics/LineLength
+    end
+
+    it 'should have the summary but blank' do
+      expect(@entry.summary).to be_nil
+    end
+
+    it 'should have the custom youtube video id' do
+      expect(@entry.youtube_video_id).to eq '5shykyfmb28'
+    end
+
+    it 'should have the custom media title' do
+      expect(@entry.media_title).to eq 'The Google app: Questions'
+    end
+
+    it 'should have the custom media url' do
+      expect(@entry.media_url).to eq 'https://www.youtube.com/v/5shykyfmb28?version=3' # rubocop:disable Metrics/LineLength
+    end
+
+    it 'should have the custom media type' do
+      expect(@entry.media_type).to eq 'application/x-shockwave-flash'
+    end
+
+    it 'should have the custom media width' do
+      expect(@entry.media_width).to eq '640'
+    end
+
+    it 'should have the custom media height' do
+      expect(@entry.media_height).to eq '390'
+    end
+
+    it 'should have the custom media thumbnail url' do
+      expect(@entry.media_thumbnail_url).to eq 'https://i2.ytimg.com/vi/5shykyfmb28/hqdefault.jpg' # rubocop:disable Metrics/LineLength
+    end
+
+    it 'should have the custom media thumbnail width' do
+      expect(@entry.media_thumbnail_width).to eq '480'
+    end
+
+    it 'should have the custom media thumbnail height' do
+      expect(@entry.media_thumbnail_height).to eq '360'
+    end
+
+    it 'should have the custom media star count' do
+      expect(@entry.media_star_count).to eq '3546'
+    end
+
+    it 'should have the custom media star average' do
+      expect(@entry.media_star_average).to eq '4.79'
+    end
+
+    it 'should have the custom media views' do
+      expect(@entry.media_views).to eq '251497'
+    end
+  end
+end

data/spec/feedjira/parser/atom_youtube_spec.rb

@@ -0,0 +1,43 @@
+require File.join(File.dirname(__FILE__), %w(.. .. spec_helper))
+
+describe Feedjira::Parser::AtomYoutube do
+  describe '#will_parse?' do
+    it 'should return true for an atom youtube feed' do
+      expect(Feedjira::Parser::AtomYoutube).to be_able_to_parse(sample_youtube_atom_feed) # rubocop:disable Metrics/LineLength
+    end
+
+    it 'should return fase for an atom feed' do
+      expect(Feedjira::Parser::AtomYoutube).to_not be_able_to_parse(sample_atom_feed) # rubocop:disable Metrics/LineLength
+    end
+
+    it 'should return false for an rss feedburner feed' do
+      expect(Feedjira::Parser::AtomYoutube).to_not be_able_to_parse(sample_rss_feed_burner_feed) # rubocop:disable Metrics/LineLength
+    end
+  end
+
+  describe 'parsing' do
+    before(:each) do
+      @feed = Feedjira::Parser::AtomYoutube.parse(sample_youtube_atom_feed)
+    end
+
+    it 'should parse the title' do
+      expect(@feed.title).to eq 'Google'
+    end
+
+    it 'should parse the author' do
+      expect(@feed.author).to eq 'Google Author'
+    end
+
+    it 'should parse the url' do
+      expect(@feed.url).to eq 'http://www.youtube.com/user/Google'
+    end
+
+    it 'should parse the feed_url' do
+      expect(@feed.feed_url).to eq 'http://www.youtube.com/feeds/videos.xml?user=google'
+    end
+
+    it 'should parse the YouTube channel id' do
+      expect(@feed.youtube_channel_id).to eq 'UCK8sQmJBp8GCxrOtXWBpyEA'
+    end
+  end
+end

data/spec/feedjira/parser/google_docs_atom_entry_spec.rb

@@ -1,9 +1,10 @@
-require File.join(File.dirname(__FILE__), %w[.. .. spec_helper])
+require 'spec_helper'
 
 describe Feedjira::Parser::GoogleDocsAtomEntry do
   describe 'parsing' do
     before do
-      @feed = Feedjira::Parser::GoogleDocsAtom.parse(sample_google_docs_list_feed)
+      xml = sample_google_docs_list_feed
+      @feed = Feedjira::Parser::GoogleDocsAtom.parse xml
       @entry = @feed.entries.first
     end
 
@@ -12,11 +13,11 @@ describe Feedjira::Parser::GoogleDocsAtomEntry do
     end
 
     it 'should have the custom filename element' do
-      expect(@entry.original_filename).to eq "MyFile.pdf"
+      expect(@entry.original_filename).to eq 'MyFile.pdf'
     end
 
     it 'should have the custom suggested filename element' do
-      expect(@entry.suggested_filename).to eq "TaxDocument.pdf"
+      expect(@entry.suggested_filename).to eq 'TaxDocument.pdf'
     end
   end
 end

data/spec/feedjira/parser/google_docs_atom_spec.rb

@@ -1,19 +1,19 @@
-require File.join(File.dirname(__FILE__), %w[.. .. spec_helper])
+require 'spec_helper'
 
-describe Feedjira::Parser::GoogleDocsAtom do
+module Feedjira::Parser
   describe '.able_to_parser?' do
     it 'should return true for Google Docs feed' do
-      expect(Feedjira::Parser::GoogleDocsAtom).to be_able_to_parse(sample_google_docs_list_feed)
+      expect(GoogleDocsAtom).to be_able_to_parse(sample_google_docs_list_feed)
     end
 
     it 'should not be able to parse another Atom feed' do
-      expect(Feedjira::Parser::GoogleDocsAtom).to_not be_able_to_parse(sample_atom_feed)
+      expect(GoogleDocsAtom).to_not be_able_to_parse(sample_atom_feed)
     end
   end
 
   describe 'parsing' do
     before do
-      @feed = Feedjira::Parser::GoogleDocsAtom.parse(sample_google_docs_list_feed)
+      @feed = GoogleDocsAtom.parse(sample_google_docs_list_feed)
     end
 
     it 'should return a bunch of objects' do
@@ -25,7 +25,7 @@ describe Feedjira::Parser::GoogleDocsAtom do
     end
 
     it 'should return a bunch of entries of type GoogleDocsAtomEntry' do
-      expect(@feed.entries.first).to be_a Feedjira::Parser::GoogleDocsAtomEntry
+      expect(@feed.entries.first).to be_a GoogleDocsAtomEntry
     end
   end
 end

data/spec/feedjira/parser/itunes_rss_item_spec.rb

@@ -1,63 +1,67 @@
-require File.join(File.dirname(__FILE__), %w[.. .. spec_helper])
+require 'spec_helper'
 
 describe Feedjira::Parser::ITunesRSSItem do
   before(:each) do
-    # I don't really like doing it this way because these unit test should only rely on ITunesRssItem,
-    # but this is actually how it should work. You would never just pass entry xml straight to the ITunesRssItem
+    # I don't really like doing it this way because these unit test should only
+    # rely on ITunesRssItem, but this is actually how it should work. You would
+    # never just pass entry xml straight to the ITunesRssItem
     @item = Feedjira::Parser::ITunesRSS.parse(sample_itunes_feed).entries.first
   end
 
-  it "should parse the title" do
-    expect(@item.title).to eq "Shake Shake Shake Your Spices"
+  it 'should parse the title' do
+    expect(@item.title).to eq 'Shake Shake Shake Your Spices'
   end
 
-  it "should parse the author" do
-    expect(@item.itunes_author).to eq "John Doe"
+  it 'should parse the author' do
+    expect(@item.itunes_author).to eq 'John Doe'
   end
 
-  it "should parse the subtitle" do
-    expect(@item.itunes_subtitle).to eq "A short primer on table spices"
+  it 'should parse the subtitle' do
+    expect(@item.itunes_subtitle).to eq 'A short primer on table spices'
   end
 
-  it "should parse the summary" do
-    expect(@item.itunes_summary).to eq "This week we talk about salt and pepper shakers, comparing and contrasting pour rates, construction materials, and overall aesthetics. Come and join the party!"
+  it 'should parse the summary' do
+    summary = 'This week we talk about salt and pepper shakers, comparing and contrasting pour rates, construction materials, and overall aesthetics. Come and join the party!' # rubocop:disable Metrics/LineLength
+    expect(@item.itunes_summary).to eq summary
   end
 
-  it "should parse the enclosure" do
-    expect(@item.enclosure_length).to eq "8727310"
-    expect(@item.enclosure_type).to eq "audio/x-m4a"
-    expect(@item.enclosure_url).to eq "http://example.com/podcasts/everything/AllAboutEverythingEpisode3.m4a"
+  it 'should parse the enclosure' do
+    expect(@item.enclosure_length).to eq '8727310'
+    expect(@item.enclosure_type).to eq 'audio/x-m4a'
+    expect(@item.enclosure_url).to eq 'http://example.com/podcasts/everything/AllAboutEverythingEpisode3.m4a'
   end
 
-  it "should parse the guid as id" do
-    expect(@item.id).to eq "http://example.com/podcasts/archive/aae20050615.m4a"
+  it 'should parse the guid as id' do
+    expect(@item.id).to eq 'http://example.com/podcasts/archive/aae20050615.m4a'
   end
 
-  it "should parse the published date" do
-    expect(@item.published).to eq Time.parse_safely("Wed Jun 15 19:00:00 UTC 2005")
+  it 'should parse the published date' do
+    published = Time.parse_safely 'Wed Jun 15 19:00:00 UTC 2005'
+    expect(@item.published).to eq published
   end
 
-  it "should parse the duration" do
-    expect(@item.itunes_duration).to eq "7:04"
+  it 'should parse the duration' do
+    expect(@item.itunes_duration).to eq '7:04'
   end
 
-  it "should parse the keywords" do
-    expect(@item.itunes_keywords).to eq "salt, pepper, shaker, exciting"
+  it 'should parse the keywords' do
+    expect(@item.itunes_keywords).to eq 'salt, pepper, shaker, exciting'
   end
 
-  it "should parse the image" do
-    expect(@item.itunes_image).to eq "http://example.com/podcasts/everything/AllAboutEverything.jpg"
+  it 'should parse the image' do
+    expect(@item.itunes_image).to eq 'http://example.com/podcasts/everything/AllAboutEverything.jpg'
   end
 
-  it "should parse the order" do
+  it 'should parse the order' do
     expect(@item.itunes_order).to eq '12'
   end
 
-  it "should parse the closed captioned flag" do
+  it 'should parse the closed captioned flag' do
     expect(@item.itunes_closed_captioned).to eq 'yes'
   end
 
-  it "should parse the encoded content" do
-    expect(@item.content).to eq "<p><strong>TOPIC</strong>: Gooseneck Options</p>"
+  it 'should parse the encoded content' do
+    content = '<p><strong>TOPIC</strong>: Gooseneck Options</p>'
+    expect(@item.content).to eq content
   end
 end

data/spec/feedjira/parser/itunes_rss_owner_spec.rb

@@ -1,18 +1,19 @@
-require File.join(File.dirname(__FILE__), %w[.. .. spec_helper])
+require 'spec_helper'
 
 describe Feedjira::Parser::ITunesRSSOwner do
   before(:each) do
-    # I don't really like doing it this way because these unit test should only rely on RSSEntry,
-    # but this is actually how it should work. You would never just pass entry xml straight to the ITunesRssOwner
-    @owner = Feedjira::Parser::ITunesRSS.parse(sample_itunes_feed).itunes_owners.first
+    # I don't really like doing it this way because these unit test should only
+    # rely on RSSEntry, but this is actually how it should work. You would
+    # never just pass entry xml straight to the ITunesRssOwner
+    feed = Feedjira::Parser::ITunesRSS.parse sample_itunes_feed
+    @owner = feed.itunes_owners.first
   end
 
-  it "should parse the name" do
-    expect(@owner.name).to eq "John Doe"
+  it 'should parse the name' do
+    expect(@owner.name).to eq 'John Doe'
   end
 
-  it "should parse the email" do
-    expect(@owner.email).to eq "john.doe@example.com"
+  it 'should parse the email' do
+    expect(@owner.email).to eq 'john.doe@example.com'
   end
-
 end

data/spec/feedjira/parser/itunes_rss_spec.rb

@@ -1,62 +1,115 @@
-require File.join(File.dirname(__FILE__), %w[.. .. spec_helper])
+require 'spec_helper'
 
-describe Feedjira::Parser::ITunesRSS do
-  describe "#will_parse?" do
-    it "should return true for an itunes RSS feed" do
-      expect(Feedjira::Parser::ITunesRSS).to be_able_to_parse(sample_itunes_feed)
+module Feedjira::Parser
+  describe '#will_parse?' do
+    it 'should return true for an itunes RSS feed' do
+      expect(ITunesRSS).to be_able_to_parse(sample_itunes_feed)
     end
 
-    it "should return true for an itunes RSS feed with spaces between attribute names, equals sign, and values" do
-      expect(Feedjira::Parser::ITunesRSS).to be_able_to_parse(sample_itunes_feed_with_spaces)
+    it 'should return true for an itunes RSS feed with spaces between attribute names, equals sign, and values' do # rubocop:disable Metrics/LineLength
+      expect(ITunesRSS).to be_able_to_parse(sample_itunes_feed_with_spaces)
     end
 
-    it "should return fase for an atom feed" do
-      expect(Feedjira::Parser::ITunesRSS).to_not be_able_to_parse(sample_atom_feed)
+    it 'should return true for an itunes RSS feed with single-quoted attributes' do # rubocop:disable Metrics/LineLength
+      expect(ITunesRSS).to be_able_to_parse(sample_itunes_feed_with_single_quotes)  # rubocop:disable Metrics/LineLength
     end
 
-    it "should return false for an rss feedburner feed" do
-      expect(Feedjira::Parser::ITunesRSS).to_not be_able_to_parse(sample_rss_feed_burner_feed)
+    it 'should return fase for an atom feed' do
+      expect(ITunesRSS).to_not be_able_to_parse(sample_atom_feed)
+    end
+
+    it 'should return false for an rss feedburner feed' do
+      expect(ITunesRSS).to_not be_able_to_parse(sample_rss_feed_burner_feed)
     end
   end
 
-  describe "parsing" do
+  describe 'parsing' do
     before(:each) do
-      @feed = Feedjira::Parser::ITunesRSS.parse(sample_itunes_feed)
+      @feed = ITunesRSS.parse(sample_itunes_feed)
+    end
+
+    it 'should parse the ttl' do
+      expect(@feed.ttl).to eq '60'
+    end
+
+    it 'should parse the last build date' do
+      expect(@feed.last_built).to eq 'Sat, 07 Sep 2002 09:42:31 GMT'
     end
 
-    it "should parse the subtitle" do
-      expect(@feed.itunes_subtitle).to eq "A show about everything"
+    it 'should parse the subtitle' do
+      expect(@feed.itunes_subtitle).to eq 'A show about everything'
     end
 
-    it "should parse the author" do
-      expect(@feed.itunes_author).to eq "John Doe"
+    it 'should parse the author' do
+      expect(@feed.itunes_author).to eq 'John Doe'
     end
 
-    it "should parse an owner" do
+    it 'should parse an owner' do
       expect(@feed.itunes_owners.size).to eq 1
     end
 
-    it "should parse an image" do
-      expect(@feed.itunes_image).to eq "http://example.com/podcasts/everything/AllAboutEverything.jpg"
+    it 'should parse an image' do
+      expect(@feed.itunes_image).to eq 'http://example.com/podcasts/everything/AllAboutEverything.jpg'
+    end
+
+    it 'should parse the image url' do
+      expect(@feed.image.url).to eq 'http://example.com/podcasts/everything/AllAboutEverything.jpg'
+    end
+
+    it 'should parse the image title' do
+      expect(@feed.image.title).to eq 'All About Everything'
+    end
+
+    it 'should parse the image link' do
+      expect(@feed.image.link).to eq 'http://www.example.com/podcasts/everything/index.html'
+    end
+
+    it 'should parse the image width' do
+      expect(@feed.image.width).to eq '88'
+    end
+
+    it 'should parse the image height' do
+      expect(@feed.image.height).to eq '31'
+    end
+
+    it 'should parse the image description' do
+      description = 'All About Everything is a show about everything. Each week we dive into any subject known to man and talk about it as much as we can. Look for our Podcast in the iTunes Music Store' # rubocop:disable Metrics/LineLength
+      expect(@feed.image.description).to eq description
+    end
+
+    it 'should parse categories' do
+      expect(@feed.itunes_categories).to eq [
+        'Technology',
+        'Gadgets',
+        'TV & Film',
+        'Arts',
+        'Design',
+        'Food'
+      ]
+
+      expect(@feed.itunes_category_paths).to eq [
+        %w(Technology Gadgets),
+        ['TV & Film'],
+        %w(Arts Design),
+        %w(Arts Food)
+      ]
     end
 
-    it "should parse categories" do
-      expect(@feed.itunes_categories.size).to eq 3
-      expect(@feed.itunes_categories[0]).to eq "Technology"
-      expect(@feed.itunes_categories[1]).to eq "Gadgets"
-      expect(@feed.itunes_categories[2]).to eq "TV & Film"
+    it 'should parse the summary' do
+      summary = 'All About Everything is a show about everything. Each week we dive into any subject known to man and talk about it as much as we can. Look for our Podcast in the iTunes Music Store' # rubocop:disable Metrics/LineLength
+      expect(@feed.itunes_summary).to eq summary
     end
 
-    it "should parse the summary" do
-      expect(@feed.itunes_summary).to eq "All About Everything is a show about everything. Each week we dive into any subject known to man and talk about it as much as we can. Look for our Podcast in the iTunes Music Store"
+    it 'should parse the complete tag' do
+      expect(@feed.itunes_complete).to eq 'yes'
     end
 
-    it "should parse entries" do
+    it 'should parse entries' do
       expect(@feed.entries.size).to eq 3
     end
 
-    it "should parse the new-feed-url" do
-      expect(@feed.itunes_new_feed_url).to eq "http://example.com/new.xml"
+    it 'should parse the new-feed-url' do
+      expect(@feed.itunes_new_feed_url).to eq 'http://example.com/new.xml'
     end
   end
 end