video_info 2.6.0 → 2.6.1

data/spec/lib/video_info/providers/vkontakte_spec.rb

@@ -3,144 +3,200 @@ require 'spec_helper'
 require 'webmock/rspec'
 
 describe VideoInfo::Providers::Vkontakte do
-
-  describe ".usable?" do
+  describe '.usable?' do
     subject { VideoInfo::Providers::Vkontakte.usable?(url) }
 
-    context "with vkontakte url" do
-      context "old style", :vcr do
+    context 'with vkontakte url' do
+      context 'old style', :vcr do
         let(:url) { 'http://vk.com/video39576223_108370515' }
         it { should be_truthy }
       end
 
-      context "new style", :vcr do
-        let(:url) { 'https://vk.com/kirill.lyanoi?z=video2152699_168591741%2F56fd229a9dfe2dcdbe' }
+      context 'new style', :vcr do
+        let(:url) do
+          'https://vk.com/kirill.lyanoi?z=video2152699' \
+          '_168591741%2F56fd229a9dfe2dcdbe'
+        end
         it { should be_truthy }
       end
     end
 
-    context "with other url" do
+    context 'with other url' do
       let(:url) { 'http://www.youtube.com/898029' }
       it { should be_falsey }
     end
 
-    context "with invalid url" do
+    context 'with invalid url' do
       let(:url) { 'http://vk.com just random data' }
       it { should be_falsey }
     end
 
-    context "with spaces at end" do
+    context 'with spaces at end' do
       let(:url) { 'http://vk.com/video39576223_108370515        ' }
       it { should be_truthy }
     end
 
-    context "with spaces at start" do
+    context 'with spaces at start' do
       let(:url) { '      http://vk.com/video39576223_108370515' }
       it { should be_truthy }
     end
 
-    context "with spaces around url" do
+    context 'with spaces around url' do
       let(:url) { '      http://vk.com/video39576223_108370515      ' }
       it { should be_truthy }
     end
   end
 
-  describe "#available?" do
-    context "with valid video", :vcr do
+  describe '#available?' do
+    context 'with valid video', :vcr do
       subject { VideoInfo.new('http://vk.com/video39576223_108370515') }
-      its(:available?)       { should be_truthy }
+      its(:available?) { should be_truthy }
     end
 
-    context "with invalid video", :vcr do
+    context 'with invalid video', :vcr do
       subject { VideoInfo.new('http://vk.com/video39576223_invalid') }
-      its(:available?)       { should be_falsey }
+      its(:available?) { should be_falsey }
     end
 
-    context "with private video", :vcr do
+    context 'with private video', :vcr do
       subject { VideoInfo.new('http://vk.com/video39576223_166315543') }
-      its(:available?)       { should be_falsey }
+      its(:available?) { should be_falsey }
     end
 
-    context "with redirect", :vcr do
-      subject { VideoInfo.new('http://vk.com/polinka_zh?z=video186965901_168185427%2Fbfb2bd2e674031520a') }
-      its(:available?)       { should be_truthy }
+    context 'with redirect', :vcr do
+      video_url = 'http://vk.com/polinka_zh?z=' \
+                  'video186965901_168185427%2Fbfb2bd2e674031520a'
+      subject { VideoInfo.new(video_url) }
+      its(:available?) { should be_truthy }
     end
 
-    context "with redirect to main page for auth", :vcr do
-      subject { VideoInfo.new('http://vk.com/video?z=video1472940_169081944%2Falbum1472940') }
-      its(:available?)       { should be_falsey }
+    context 'with redirect to main page for auth', :vcr do
+      video_url = 'http://vk.com/video?z=video1472940_169081944%2Falbum1472940'
+      subject { VideoInfo.new(video_url) }
+      its(:available?) { should be_falsey }
     end
 
-    context "with hashes", :vcr do
-      subject { VideoInfo.new('https://vk.com/videos43640822#/video43640822_168790809') }
-      its(:available?)       { should be_truthy }
+    context 'with hashes', :vcr do
+      video_url = 'https://vk.com/videos43640822#/video43640822_168790809'
+      subject { VideoInfo.new(video_url) }
+      its(:available?) { should be_truthy }
     end
-
   end
 
-  context "with video https://vk.com/id44052340?z=video61291456_159590018%2F2521d92730a272a9ea", :vcr do
-    subject { VideoInfo.new('https://vk.com/id44052340?z=video61291456_159590018%2F2521d92730a272a9ea') }
+  context 'with video https://vk.com/id44052340?z=' \
+          'video61291456_159590018%2F2521d92730a272a9ea', :vcr do
+    video_url = 'https://vk.com/id44052340?z=' \
+                'video61291456_159590018%2F2521d92730a272a9ea'
+
+    subject { VideoInfo.new(video_url) }
+
+    embed_code = '<iframe src="//vk.com/video_ext.php?oid=61291456&' \
+                 'id=159590018&hash=68174b2af560c54c" frameborder="0" ' \
+                 'allowfullscreen="allowfullscreen"></iframe>'
+    embed_url = '//vk.com/video_ext.php?oid=61291456&' \
+                'id=159590018&hash=68174b2af560c54c'
 
     its(:provider)         { should eq 'Vkontakte' }
     its(:video_owner)      { should eq '61291456' }
     its(:video_id)         { should eq '159590018' }
     its(:title)            { should eq 'Happy Birthday To You' }
-    its(:embed_url)        { should eq '//vk.com/video_ext.php?oid=61291456&id=159590018&hash=68174b2af560c54c' }
-    its(:embed_code)       { should eq '<iframe src="//vk.com/video_ext.php?oid=61291456&id=159590018&hash=68174b2af560c54c" frameborder="0" allowfullscreen="allowfullscreen"></iframe>' }
+    its(:embed_url)        { should eq embed_url }
+    its(:embed_code)       { should eq embed_code }
+    its(:author)           { should eq('Tanka Malesh').or(eq('Танька Малеш')) }
   end
 
-  context "with video videos43640822#/video43640822_168790809", :vcr do
-    subject { VideoInfo.new('https://vk.com/videos43640822#/video43640822_168790809') }
+  context 'with video videos43640822#/video43640822_168790809', :vcr do
+    video_url = 'https://vk.com/videos43640822#/video43640822_168790809'
+    subject { VideoInfo.new(video_url) }
+
+    author_thumbnail = 'https://pp.vk.me/c604531/v604531822' \
+                       '/2beb/JrLWCWTA7y0.jpg'
+    video_title = 'UDC open cup 2014/ 3 place / Saley Daria (solo)'
 
     its(:provider)         { should eq 'Vkontakte' }
     its(:video_owner)      { should eq '43640822' }
     its(:video_id)         { should eq '168790809' }
-    its(:title)            { should eq 'UDC open cup 2014/ 3 place / Saley Daria (solo)' }
+    its(:title)            { should eq video_title }
+    its(:author)           { should eq('Dasha Saley').or(eq('Даша Салей')) }
+    its(:author_thumbnail) { should eq author_thumbnail }
+    its(:author_url)       { should eq 'https://vk.com/dariasaley' }
   end
 
-  context "with video kirill.lyanoi?z=video2152699_168591741%2F56fd229a9dfe2dcdbe", :vcr do
-    subject { VideoInfo.new('https://vk.com/kirill.lyanoi?z=video2152699_168591741%2F56fd229a9dfe2dcdbe') }
+  context 'with video from kirill.lyanoi?z=' \
+          'video2152699_168591741%2F56fd229a9dfe2dcdbe', :vcr do
+    video_url = 'https://vk.com/kirill.lyanoi?' \
+                'z=video2152699_168591741%2F56fd229a9dfe2dcdbe'
+    subject { VideoInfo.new(video_url) }
+
+    author_thumbnail = 'https://pp.vk.me/c623824/v623824699' \
+                       '/55575/CCQZ29l0B9k.jpg'
+    description_text = 'BEAT SOUL STEP ★ Project818 Russian ' \
+                       'Dance Championship ★ 1-2 мая, Москва 2014'
+    video_title = 'BEAT SOUL STEP — RDC14 Project818 Russian ' \
+                  'Dance Championship, May 1-2, Moscow 2014'
+    embed_code = '<iframe src="https://www.youtube.com/embed/' \
+                 '4Thws5wq5GI?enablejsapi=1&autoplay=0&start=0' \
+                 '&autohide=1&wmode=opaque&showinfo=0&origin=' \
+                 'https://vk.com&fs=1&rel=0&cc_load_policy=0' \
+                 '&iv_load_policy=3" frameborder="0" ' \
+                 'allowfullscreen="allowfullscreen"></iframe>'
+    embed_url = 'https://www.youtube.com/embed/4Thws5wq5GI?enablejsapi=1' \
+                '&autoplay=0&start=0&autohide=1&wmode=opaque&showinfo=0' \
+                '&origin=https://vk.com&fs=1&rel=0&cc_load_policy=0' \
+                '&iv_load_policy=3'
+    author = 'Kirill Lyanoy'
+    author_alt = 'Кирилл Льяной'
 
     its(:provider)         { should eq 'Vkontakte' }
     its(:video_owner)      { should eq '2152699' }
     its(:video_id)         { should eq '168591741' }
-    its(:url)              { should eq 'https://vk.com/kirill.lyanoi?z=video2152699_168591741%2F56fd229a9dfe2dcdbe' }
-    its(:embed_url)        { should eq '//vk.com/video_ext.php?oid=2152699&id=168591741&hash=' }
-    its(:embed_code)       { should eq '<iframe src="//vk.com/video_ext.php?oid=2152699&id=168591741&hash=" frameborder="0" allowfullscreen="allowfullscreen"></iframe>' }
-    its(:title)            { should eq 'BEAT SOUL STEP — RDC14 Project818 Russian Dance Championship, May 1-2, Moscow 2014' }
-    its(:description)      { should start_with 'BEAT SOUL STEP ★ Project818 Russian Dance Championship ★ 1-2 мая, Москва 2014' }
-    its(:keywords)         { should start_with 'BEAT SOUL STEP ★ Project818 Russian Dance Championship ★ 1-2 мая, Москва 2014' }
+    its(:url)              { should eq video_url }
+    its(:embed_url)        { should eq embed_url }
+    its(:embed_code)       { should eq embed_code }
+    its(:title)            { should eq video_title }
+    its(:description)      { should start_with description_text }
+    its(:keywords)         { should be_nil }
     its(:duration)         { should eq 299 }
     its(:width)            { should eq 0 }
     its(:height)           { should eq 0 }
     its(:view_count)       { should be > 10 }
+    its(:author)           { should eq(author).or(eq(author_alt)) }
+    its(:author_thumbnail) { should eq author_thumbnail }
+    its(:author_url)       { should eq 'https://vk.com/lyanoi.kirill' }
   end
 
-  context "with video video39576223_108370515", :vcr do
+  context 'with video video39576223_108370515', :vcr do
     subject { VideoInfo.new('http://vk.com/video39576223_108370515') }
 
-    its(:provider)         { should eq 'Vkontakte' }
-    its(:video_owner)      { should eq '39576223' }
-    its(:video_id)         { should eq '108370515' }
-    its(:url)              { should eq 'http://vk.com/video39576223_108370515' }
-    its(:embed_url)        { should eq '//vk.com/video_ext.php?oid=39576223&id=108370515&hash=15184dbd085c47af' }
-    its(:embed_code)       { should eq '<iframe src="//vk.com/video_ext.php?oid=39576223&id=108370515&hash=15184dbd085c47af" frameborder="0" allowfullscreen="allowfullscreen"></iframe>' }
-    its(:title)            { should eq 'Я уточка)))))' }
-    its(:description)      { should eq 'это ВЗРЫВ МОЗГА!!!<br>Просто отвал башки...' }
-    its(:keywords)         { should eq 'это ВЗРЫВ МОЗГА!!!<br>Просто отвал башки...' }
-    its(:duration)         { should eq 183 }
-    its(:width)            { should eq 320 }
-    its(:height)           { should eq 240 }
-    its(:view_count)       { should be > 10 }
+    description_text = 'это ВЗРЫВ МОЗГА!!!<br>Просто отвал башки...'
+    embed_url = '//vk.com/video_ext.php?oid=39576223&' \
+                'id=108370515&hash=15184dbd085c47af'
+    embed_code = '<iframe src="//vk.com/video_ext.php?oid=39576223&' \
+                 'id=108370515&hash=15184dbd085c47af" frameborder="0" ' \
+                 'allowfullscreen="allowfullscreen"></iframe>'
+
+    its(:provider)     { should eq 'Vkontakte' }
+    its(:video_owner)  { should eq '39576223' }
+    its(:video_id)     { should eq '108370515' }
+    its(:url)          { should eq 'http://vk.com/video39576223_108370515' }
+    its(:embed_url)    { should eq embed_url }
+    its(:embed_code)   { should eq embed_code }
+    its(:title)        { should eq 'Я уточка)))))' }
+    its(:description)  { should eq description_text }
+    its(:keywords)     { should be_nil }
+    its(:duration)     { should eq 183 }
+    its(:width)        { should eq 320 }
+    its(:height)       { should eq 240 }
+    its(:view_count)   { should be > 10 }
   end
 
-  context "with video video-54799401_165822734", :vcr do
+  context 'with video video-54799401_165822734', :vcr do
     subject { VideoInfo.new('http://vk.com/video-54799401_165822734') }
 
-    its(:provider)         { should eq 'Vkontakte' }
-    its(:video_owner)      { should eq '-54799401' }
-    its(:video_id)         { should eq '165822734' }
-    its(:title)            { should eq 'SpaceGlasses are the future of computing' }
+    its(:provider)     { should eq 'Vkontakte' }
+    its(:video_owner)  { should eq '-54799401' }
+    its(:video_id)     { should eq '165822734' }
+    its(:title)        { should eq 'SpaceGlasses are the future of computing' }
   end
 
   context 'with valid video and connection timeout' do
@@ -162,7 +218,8 @@ describe VideoInfo::Providers::Vkontakte do
 
     describe '#title' do
       before do
-        stub_request(:any, /.*vk.com.*/).to_raise(OpenURI::HTTPError.new('error', :nop))
+        stub = stub_request(:any, /.*vk.com.*/)
+        stub.to_raise(OpenURI::HTTPError.new('error', :nop))
       end
 
       it 'raises VideoInfo::HttpError exception' do
@@ -170,4 +227,16 @@ describe VideoInfo::Providers::Vkontakte do
       end
     end
   end
+
+  context 'with video video3552522_171340713', :vcr do
+    subject { VideoInfo.new('http://vk.com/video3552522_171340713') }
+
+    embed_url = 'https://www.youtube.com/embed/C3_e-e6qoSg?' \
+                'enablejsapi=1&autoplay=0&start=0&autohide=1' \
+                '&wmode=opaque&showinfo=0&origin=' \
+                'https://vk.com&fs=1&rel=0&cc_load_policy=0' \
+                '&iv_load_policy=3'
+
+    its(:embed_url) { should eq embed_url }
+  end
 end

data/spec/lib/video_info/providers/wistia_spec.rb

@@ -2,7 +2,6 @@
 require 'spec_helper'
 
 describe VideoInfo::Providers::Wistia do
-
   describe '.usable?' do
     subject { VideoInfo::Providers::Wistia.usable?(url) }
 
@@ -19,7 +18,8 @@ describe VideoInfo::Providers::Wistia do
 
   describe '#available?' do
     context 'with valid video', :vcr do
-      subject { VideoInfo.new('http://fast.wistia.net/embed/iframe/rs1me54mpw') }
+      video_url = 'http://fast.wistia.net/embed/iframe/rs1me54mpw'
+      subject { VideoInfo.new(video_url) }
 
       describe '#available?' do
         subject { super().available? }
@@ -35,7 +35,6 @@ describe VideoInfo::Providers::Wistia do
         it { is_expected.to be_falsey }
       end
     end
-
   end
 
   context 'with video rs1me54mpw', :vcr do
@@ -63,7 +62,9 @@ describe VideoInfo::Providers::Wistia do
 
     describe '#embed_code' do
       subject { super().embed_code }
-      it { is_expected.to eq '<iframe src="//fast.wistia.net/embed/iframe/rs1me54mpw" frameborder="0"></iframe>' }
+      embed_code = '<iframe src="//fast.wistia.net/embed/iframe/rs1me54mpw" '\
+                   'frameborder="0"></iframe>'
+      it { is_expected.to eq embed_code }
     end
 
     describe '#title' do
@@ -88,17 +89,26 @@ describe VideoInfo::Providers::Wistia do
 
     describe '#thumbnail_small' do
       subject { super().thumbnail_small }
-      it { is_expected.to eq 'https://embed-ssl.wistia.com/deliveries/dc005e6a7583a25c4b7c284c08afc42fb67fdb95.jpg?image_crop_resized=960x540' }
+      thumbnail_url = 'https://embed-ssl.wistia.com/deliveries/' \
+                      'dc005e6a7583a25c4b7c284c08afc42fb67fdb95.jpg' \
+                      '?image_crop_resized=960x540'
+      it { is_expected.to eq thumbnail_url }
     end
 
     describe '#thumbnail_medium' do
       subject { super().thumbnail_small }
-      it { is_expected.to eq 'https://embed-ssl.wistia.com/deliveries/dc005e6a7583a25c4b7c284c08afc42fb67fdb95.jpg?image_crop_resized=960x540' }
+      thumbnail_url = 'https://embed-ssl.wistia.com/deliveries/' \
+                      'dc005e6a7583a25c4b7c284c08afc42fb67fdb95.jpg?' \
+                      'image_crop_resized=960x540'
+      it { is_expected.to eq thumbnail_url }
     end
-    
+
     describe '#thumbnail_large' do
       subject { super().thumbnail_small }
-      it { is_expected.to eq 'https://embed-ssl.wistia.com/deliveries/dc005e6a7583a25c4b7c284c08afc42fb67fdb95.jpg?image_crop_resized=960x540' }
+      thumbnail_url = 'https://embed-ssl.wistia.com/deliveries/' \
+                      'dc005e6a7583a25c4b7c284c08afc42fb67fdb95.jpg' \
+                      '?image_crop_resized=960x540'
+      it { is_expected.to eq thumbnail_url }
     end
   end
 
@@ -127,7 +137,9 @@ describe VideoInfo::Providers::Wistia do
 
     describe '#embed_code' do
       subject { super().embed_code }
-      it { is_expected.to eq '<iframe src="//fast.wistia.net/embed/iframe/pxonqr42is" frameborder="0"></iframe>' }
+      embed_code = '<iframe src="//fast.wistia.net/embed/iframe/pxonqr42is" ' \
+                   'frameborder="0"></iframe>'
+      it { is_expected.to eq embed_code }
     end
 
     describe '#title' do
@@ -149,20 +161,29 @@ describe VideoInfo::Providers::Wistia do
       subject { super().height }
       it { is_expected.to eq 568 }
     end
-    
+
     describe '#thumbnail_small' do
       subject { super().thumbnail_small }
-      it { is_expected.to eq 'https://embed-ssl.wistia.com/deliveries/0fccbdc60ade35723f79f1c002bc61b135b610fa.jpg?image_crop_resized=960x540' }
+      thumbnail_url = 'https://embed-ssl.wistia.com/deliveries/' \
+                      '0fccbdc60ade35723f79f1c002bc61b135b610fa.jpg' \
+                      '?image_crop_resized=960x540'
+      it { is_expected.to eq thumbnail_url }
     end
 
     describe '#thumbnail_medium' do
       subject { super().thumbnail_small }
-      it { is_expected.to eq 'https://embed-ssl.wistia.com/deliveries/0fccbdc60ade35723f79f1c002bc61b135b610fa.jpg?image_crop_resized=960x540' }
+      thumbnail_url = 'https://embed-ssl.wistia.com/deliveries/' \
+                      '0fccbdc60ade35723f79f1c002bc61b135b610fa.jpg' \
+                      '?image_crop_resized=960x540'
+      it { is_expected.to eq thumbnail_url }
     end
 
     describe '#thumbnail_large' do
       subject { super().thumbnail_small }
-      it { is_expected.to eq 'https://embed-ssl.wistia.com/deliveries/0fccbdc60ade35723f79f1c002bc61b135b610fa.jpg?image_crop_resized=960x540' }
+      thumbnail_url = 'https://embed-ssl.wistia.com/deliveries/' \
+                      '0fccbdc60ade35723f79f1c002bc61b135b610fa.jpg' \
+                      '?image_crop_resized=960x540'
+      it { is_expected.to eq thumbnail_url }
     end
   end
 end

data/spec/lib/video_info/providers/youtube_playlist_spec.rb

@@ -1,184 +1,230 @@
 require 'spec_helper'
 
-describe VideoInfo::Providers::YoutubePlaylist do
-  before(:all) do
-    VideoInfo.provider_api_keys = {}
-  end
+[nil, 'AIzaSyA6PYwSr1EnLFUFy1cZDk3Ifb0rxeJaeZ0'].each do |api_key|
+  describe VideoInfo::Providers::YoutubePlaylist do
+    before(:each) do
+      api_key = 'AIzaSyA6PYwSr1EnLFUFy1cZDk3Ifb0rxeJaeZ0'
+      VideoInfo.provider_api_keys = { youtube: api_key }
+    end
 
-  describe ".usable?" do
-    subject { VideoInfo::Providers::YoutubePlaylist.usable?(url) }
+    describe '.usable?' do
+      subject { VideoInfo::Providers::YoutubePlaylist.usable?(url) }
 
-    context "with youtube.com/playlist?p= url" do
-      let(:url) { 'http://www.youtube.com/playlist?p=PLA575C81A1FBC04CF' }
-      it { is_expected.to be_truthy }
-    end
+      context 'with youtube.com/playlist?p= url' do
+        let(:url) { 'http://www.youtube.com/playlist?p=PLA575C81A1FBC04CF' }
+        it { is_expected.to be_truthy }
+      end
 
-    context "with youtube.com/playlist?list= url" do
-      let(:url) { 'http://www.youtube.com/playlist?list=PLA575C81A1FBC04CF' }
-      it { is_expected.to be_truthy }
-    end
+      context 'with youtube.com/playlist?list= url' do
+        let(:url) { 'http://www.youtube.com/playlist?list=PLA575C81A1FBC04CF' }
+        it { is_expected.to be_truthy }
+      end
 
-    context "with youtube.com url" do
-      let(:url) { 'http://www.youtube.com/watch?v=Xp6CXF' }
-      it { is_expected.to be_falsey }
-    end
+      context 'with youtube.com url' do
+        let(:url) { 'http://www.youtube.com/watch?v=Xp6CXF' }
+        it { is_expected.to be_falsey }
+      end
 
-    context "with other url" do
-      let(:url) { 'http://example.com/video1' }
-      it { is_expected.to be_falsey }
+      context 'with other url' do
+        let(:url) { 'http://example.com/video1' }
+        it { is_expected.to be_falsey }
+      end
     end
-  end
 
-  describe "#available?" do
-    context "with valid playlist", :vcr do
-      subject { VideoInfo.new('http://www.youtube.com/playlist?p=PLA575C81A1FBC04CF') }
+    describe '#available?' do
+      context 'with valid playlist', :vcr do
+        playlist_url = 'http://www.youtube.com/playlist?p=PLA575C81A1FBC04CF'
+        subject { VideoInfo.new(playlist_url) }
 
-      describe '#available?' do
-        subject { super().available? }
-        it { is_expected.to be_truthy }
+        describe '#available?' do
+          subject { super().available? }
+          it { is_expected.to be_truthy }
+        end
       end
-    end
 
-    context "with invalid playlist", :vcr do
-      subject { VideoInfo.new('http://www.youtube.com/playlist?p=PLA575C81A1FBC04CF_invalid') }
+      context 'with invalid playlist', :vcr do
+        invalid_playlist_url = 'http://www.youtube.com/playlist?' \
+                               'p=PLA575C81A1FBC04CF_invalid'
+        subject { VideoInfo.new(invalid_playlist_url) }
 
-      describe '#available?' do
-        subject { super().available? }
-        it { is_expected.to be_falsey }
+        describe '#available?' do
+          subject { super().available? }
+          it { is_expected.to be_falsey }
+        end
       end
-    end
 
-    context "with &list= url", :vcr do
-      subject { VideoInfo.new('http://www.youtube.com/playlist?list=PLA575C81A1FBC04CF') }
+      context 'with &list= url', :vcr do
+        playlist_url = 'http://www.youtube.com/playlist?list=PLA575C81A1FBC04CF'
+        subject { VideoInfo.new(playlist_url) }
 
-      describe '#available?' do
-        subject { super().available? }
-        it { is_expected.to be_truthy }
+        describe '#available?' do
+          subject { super().available? }
+          it { is_expected.to be_truthy }
+        end
       end
     end
-  end
 
-  context "with playlist PL9hW1uS6HUftLdHI6RIsaf", :vcr do
-    let(:videos) {
-      [
-        VideoInfo.new('http://www.youtube.com/watch?v=Oi67QjrXy2w'),
-        VideoInfo.new('http://www.youtube.com/watch?v=_Bt3-WsHfB0'),
-        VideoInfo.new('http://www.youtube.com/watch?v=9g2U12SsRns'),
-        VideoInfo.new('http://www.youtube.com/watch?v=8b0aEoxqqC0'),
-        VideoInfo.new('http://www.youtube.com/watch?v=6c3mHikRz0I'),
-        VideoInfo.new('http://www.youtube.com/watch?v=OQVHWsTHcoc')
-      ]
-    }
-    subject { VideoInfo.new('http://www.youtube.com/playlist?p=PL9hW1uS6HUftLdHI6RIsaf-iXTm09qnEr') }
-
-    describe '#provider' do
-      subject { super().provider }
-      it { is_expected.to eq 'YouTube' }
-    end
+    context 'with playlist PL9hW1uS6HUftLdHI6RIsaf', :vcr do
+      let(:videos) do
+        [
+          VideoInfo.new('http://www.youtube.com/watch?v=Oi67QjrXy2w'),
+          VideoInfo.new('http://www.youtube.com/watch?v=_Bt3-WsHfB0'),
+          VideoInfo.new('http://www.youtube.com/watch?v=9g2U12SsRns'),
+          VideoInfo.new('http://www.youtube.com/watch?v=8b0aEoxqqC0'),
+          VideoInfo.new('http://www.youtube.com/watch?v=6c3mHikRz0I'),
+          VideoInfo.new('http://www.youtube.com/watch?v=OQVHWsTHcoc')
+        ]
+      end
 
-    describe '#playlist_id' do
-      subject { super().playlist_id }
-      it { is_expected.to eq 'PL9hW1uS6HUftLdHI6RIsaf-iXTm09qnEr' }
-    end
+      playlist_url = 'http://www.youtube.com/playlist?' \
+                     'p=PL9hW1uS6HUftLdHI6RIsaf-iXTm09qnEr'
 
-    describe '#url' do
-      subject { super().url }
-      it { is_expected.to eq 'http://www.youtube.com/playlist?p=PL9hW1uS6HUftLdHI6RIsaf-iXTm09qnEr' }
-    end
+      subject { VideoInfo.new(playlist_url) }
 
-    describe '#embed_url' do
-      subject { super().embed_url }
-      it { is_expected.to eq '//www.youtube.com/embed/videoseries?list=PL9hW1uS6HUftLdHI6RIsaf-iXTm09qnEr' }
-    end
+      describe '#provider' do
+        subject { super().provider }
+        it { is_expected.to eq 'YouTube' }
+      end
 
-    describe '#embed_code' do
-      subject { super().embed_code }
-      it { is_expected.to eq '<iframe src="//www.youtube.com/embed/videoseries?list=PL9hW1uS6HUftLdHI6RIsaf-iXTm09qnEr" frameborder="0" allowfullscreen="allowfullscreen"></iframe>' }
-    end
+      describe '#playlist_id' do
+        subject { super().playlist_id }
+        it { is_expected.to eq 'PL9hW1uS6HUftLdHI6RIsaf-iXTm09qnEr' }
+      end
 
-    describe '#title' do
-      subject { super().title }
-      it { is_expected.to eq 'YouTube Policy and Copyright' }
-    end
+      describe '#url' do
+        subject { super().url }
+        it { is_expected.to eq playlist_url }
+      end
 
-    describe '#description' do
-      subject { super().description }
-      it { is_expected.to eq 'Learn more about copyright basics, flagging, and the YouTube community.' }
-    end
+      describe '#embed_url' do
+        subject { super().embed_url }
+        embed_url = '//www.youtube.com/embed/videoseries' \
+                    '?list=PL9hW1uS6HUftLdHI6RIsaf-iXTm09qnEr'
+        it { is_expected.to eq embed_url }
+      end
 
-    describe '#keywords' do
-      subject { super().keywords }
-      it { is_expected.to be_nil }
-    end
+      describe '#embed_code' do
+        subject { super().embed_code }
+        embed_code = '<iframe src="//www.youtube.com/embed/videoseries' \
+                     '?list=PL9hW1uS6HUftLdHI6RIsaf-iXTm09qnEr" ' \
+                     'frameborder="0" allowfullscreen="allowfullscreen">' \
+                     '</iframe>'
+        it { is_expected.to eq embed_code }
+      end
 
-    describe '#duration' do
-      subject { super().duration }
-      it { is_expected.to be_nil }
-    end
+      describe '#title' do
+        subject { super().title }
+        it { is_expected.to eq 'YouTube Policy and Copyright' }
+      end
 
-    describe '#width' do
-      subject { super().width }
-      it { is_expected.to be_nil }
-    end
+      describe '#description' do
+        subject { super().description }
+        description_text = 'Learn more about copyright basics, flagging, ' \
+                           'and the YouTube community.'
+        it { is_expected.to eq description_text }
+      end
 
-    describe '#height' do
-      subject { super().height }
-      it { is_expected.to be_nil }
-    end
+      describe '#keywords' do
+        subject { super().keywords }
+        it { is_expected.to be_nil }
+      end
 
-    describe '#date' do
-      subject { super().date }
-      it { is_expected.to be_nil }
-    end
+      describe '#duration' do
+        subject { super().duration }
+        it { is_expected.to be_nil }
+      end
 
-    describe '#thumbnail_small' do
-      subject { super().thumbnail_small }
-      it { is_expected.to eq 'https://i.ytimg.com/vi/8b0aEoxqqC0/default.jpg' }
-    end
+      describe '#width' do
+        subject { super().width }
+        it { is_expected.to be_nil }
+      end
 
-    describe '#thumbnail_medium' do
-      subject { super().thumbnail_medium }
-      it { is_expected.to eq 'https://i.ytimg.com/vi/8b0aEoxqqC0/mqdefault.jpg' }
-    end
+      describe '#height' do
+        subject { super().height }
+        it { is_expected.to be_nil }
+      end
 
-    describe '#thumbnail_large' do
-      subject { super().thumbnail_large }
-      it { is_expected.to eq 'https://i.ytimg.com/vi/8b0aEoxqqC0/hqdefault.jpg' }
-    end
+      describe '#date' do
+        subject { super().date }
+        it { is_expected.to be_nil }
+      end
 
-    describe '#videos' do
-      subject { super().videos }
-      it { expect { subject }.to raise_error(NotImplementedError) }
-    end
+      describe '#thumbnail_small' do
+        subject { super().thumbnail_small }
+        thumbnail_url = 'https://i.ytimg.com/vi/8b0aEoxqqC0/default.jpg'
+        it { is_expected.to eq thumbnail_url }
+      end
 
-    describe '#view_count' do
-      subject { super().view_count }
-      it { is_expected.to be_nil }
-    end
-  end
+      describe '#thumbnail_medium' do
+        subject { super().thumbnail_medium }
+        thumbnail_url = 'https://i.ytimg.com/vi/8b0aEoxqqC0/mqdefault.jpg'
+        it { is_expected.to eq thumbnail_url }
+      end
 
-  context "with playlist PL0E8117603D70E10A in embed path", :vcr do
-    subject { VideoInfo.new('http://www.youtube.com/embed/videoseries?list=PL0E8117603D70E10A') }
+      describe '#thumbnail_large' do
+        subject { super().thumbnail_large }
+        thumbnail_url = 'https://i.ytimg.com/vi/8b0aEoxqqC0/hqdefault.jpg'
+        it { is_expected.to eq thumbnail_url }
+      end
 
-    describe '#playlist_id' do
-      subject { super().playlist_id }
-      it { is_expected.to eq 'PL0E8117603D70E10A' }
-    end
+      describe '#videos' do
+        subject { super().videos }
+        it 'returns list of videos in playlist' do
+          pending('waiting for bug in Youtube API to be fixed')
+          is_expected.to match_array(videos)
+        end
+      end
 
-    describe '#videos' do
-      subject { super().videos }
-      it { expect { subject }.to raise_error(NotImplementedError) }
+      describe '#view_count' do
+        subject { super().view_count }
+        it { is_expected.to be_nil }
+      end
+
+      describe '#author' do
+        subject { super().author }
+        it { is_expected.to eq 'YouTube Help' }
+      end
+
+      describe '#author_thumbnail' do
+        subject { super().author_thumbnail }
+        author_thumbnail = 'https://yt3.ggpht.com/-ni_VaN38-AE/AAAAAAAAAAI/' \
+                           'AAAAAAAAAAA/bJCTTfihBl0/s88-c-k-no-rj-c0xffffff' \
+                           '/photo.jpg'
+        it { is_expected.to eq author_thumbnail }
+      end
+
+      describe '#author_url' do
+        subject { super().author_url }
+        author_url = 'https://www.youtube.com/channel/UCMDQxm7cUx3yXkfeHa5zJIQ'
+        it { is_expected.to eq author_url }
+      end
     end
-  end
 
-  context "with playlist PL9hW1uS6HUftLdHI6RIsaf-iXTm09qnEr in embed path", :vcr do
-    subject { VideoInfo.new('http://www.youtube.com/embed/videoseries?list=PL9hW1uS6HUftLdHI6RIsaf-iXTm09qnEr') }
+    context 'with playlist that does not exist in embed path', :vcr do
+      playlist_url = 'http://www.youtube.com/embed/videoseries?' \
+                     'list=PL0E8117603D70E10A'
+      subject { VideoInfo.new(playlist_url) }
+
+      describe '#playlist_id' do
+        subject { super().playlist_id }
+        it { is_expected.to eq 'PL0E8117603D70E10A' }
+      end
 
-    describe '#playlist_id' do
-      subject { super().playlist_id }
-      it { is_expected.to eq 'PL9hW1uS6HUftLdHI6RIsaf-iXTm09qnEr' }
+      describe '#videos' do
+        subject { super().videos }
+        it { is_expected.to eq [] }
+      end
     end
-  end
 
+    context 'with playlist valid playlist in embed path', :vcr do
+      playlist_url = 'http://www.youtube.com/embed/videoseries' \
+                     '?list=PL9hW1uS6HUftLdHI6RIsaf-iXTm09qnEr'
+      subject { VideoInfo.new(playlist_url) }
 
+      describe '#playlist_id' do
+        subject { super().playlist_id }
+        it { is_expected.to eq 'PL9hW1uS6HUftLdHI6RIsaf-iXTm09qnEr' }
+      end
+    end
+  end
 end