video_info 2.6.0 → 2.6.1

data/spec/fixtures/vcr_cassettes/VideoInfo_Providers_YoutubePlaylist/with_playlist_that_does_not_exist_in_embed_path/_videos/1_4_2_1.yml

@@ -0,0 +1,56 @@
+---
+http_interactions:
+- request:
+    method: get
+    uri: https://www.googleapis.com/youtube/v3/playlistItems?fields=items&key=AIzaSyA6PYwSr1EnLFUFy1cZDk3Ifb0rxeJaeZ0&part=snippet&playlistId=PL0E8117603D70E10A
+    body:
+      encoding: US-ASCII
+      string: ''
+    headers:
+      User-Agent:
+      - VideoInfo/2.6.0
+      Accept-Encoding:
+      - gzip;q=1.0,deflate;q=0.6,identity;q=0.3
+      Accept:
+      - "*/*"
+  response:
+    status:
+      code: 200
+      message: OK
+    headers:
+      Expires:
+      - Thu, 05 May 2016 01:01:55 GMT
+      Date:
+      - Thu, 05 May 2016 01:01:55 GMT
+      Cache-Control:
+      - private, max-age=0, must-revalidate, no-transform
+      Etag:
+      - '"kiOs9cZLH2FUp6r6KJ8eyq_LIOk/ewwRz0VbTYpp2EGbOkvZ5M_1mbo"'
+      Vary:
+      - Origin
+      - X-Origin
+      Content-Type:
+      - application/json; charset=UTF-8
+      X-Content-Type-Options:
+      - nosniff
+      X-Frame-Options:
+      - SAMEORIGIN
+      X-Xss-Protection:
+      - 1; mode=block
+      Content-Length:
+      - '17'
+      Server:
+      - GSE
+      Alternate-Protocol:
+      - 443:quic
+      Alt-Svc:
+      - quic=":443"; ma=2592000; v="33,32,31,30,29,28,27,26,25"
+    body:
+      encoding: UTF-8
+      string: |
+        {
+         "items": []
+        }
+    http_version: 
+  recorded_at: Thu, 05 May 2016 01:01:55 GMT
+recorded_with: VCR 3.0.1

data/spec/fixtures/vcr_cassettes/VideoInfo_Providers_YoutubePlaylist/with_playlist_that_does_not_exist_in_embed_path/_videos/2_4_2_1.yml

@@ -0,0 +1,56 @@
+---
+http_interactions:
+- request:
+    method: get
+    uri: https://www.googleapis.com/youtube/v3/playlistItems?fields=items&key=AIzaSyA6PYwSr1EnLFUFy1cZDk3Ifb0rxeJaeZ0&part=snippet&playlistId=PL0E8117603D70E10A
+    body:
+      encoding: US-ASCII
+      string: ''
+    headers:
+      User-Agent:
+      - VideoInfo/2.6.0
+      Accept-Encoding:
+      - gzip;q=1.0,deflate;q=0.6,identity;q=0.3
+      Accept:
+      - "*/*"
+  response:
+    status:
+      code: 200
+      message: OK
+    headers:
+      Expires:
+      - Thu, 05 May 2016 01:01:57 GMT
+      Date:
+      - Thu, 05 May 2016 01:01:57 GMT
+      Cache-Control:
+      - private, max-age=0, must-revalidate, no-transform
+      Etag:
+      - '"kiOs9cZLH2FUp6r6KJ8eyq_LIOk/ewwRz0VbTYpp2EGbOkvZ5M_1mbo"'
+      Vary:
+      - Origin
+      - X-Origin
+      Content-Type:
+      - application/json; charset=UTF-8
+      X-Content-Type-Options:
+      - nosniff
+      X-Frame-Options:
+      - SAMEORIGIN
+      X-Xss-Protection:
+      - 1; mode=block
+      Content-Length:
+      - '17'
+      Server:
+      - GSE
+      Alternate-Protocol:
+      - 443:quic
+      Alt-Svc:
+      - quic=":443"; ma=2592000; v="33,32,31,30,29,28,27,26,25"
+    body:
+      encoding: UTF-8
+      string: |
+        {
+         "items": []
+        }
+    http_version: 
+  recorded_at: Thu, 05 May 2016 01:01:57 GMT
+recorded_with: VCR 3.0.1

data/spec/lib/video_info/provider_spec.rb

@@ -3,58 +3,78 @@ require 'spec_helper'
 describe VideoInfo::Provider do
   class ProviderFu < VideoInfo::Provider
     def _url_regex
-      /foo\/(\d)/
+      %r{foo\/(\d)}
     end
 
     def embed_url
-      "//foo.com"
+      '//foo.com'
     end
 
     def _default_url_attributes; {} end
+
     def _default_iframe_attributes; {} end
   end
   let(:url) { 'url' }
-  let(:options) { { } }
+  let(:options) { {} }
   let(:provider) { ProviderFu.new('foo/1', options) }
 
-  describe "initialize" do
+  describe 'initialize' do
+    it 'should raise error if #_url_regex is not implemented' do
+      error_msg = 'Provider class must implement #_url_regex private method'
+      expected = expect { VideoInfo::Provider.new(url) }
+      expected.to raise_error(NotImplementedError, error_msg)
+    end
 
-    it { expect { VideoInfo::Provider.new(url) }.to raise_error(NotImplementedError, 'Provider class must implement #_url_regex private method') }
-    it { expect { provider.data }.to raise_error(NotImplementedError, 'Provider class must implement #_api_url private method') }
+    it 'should raise error if #_api_url is not implemented' do
+      api_url_error_msg = 'Provider class must implement ' \
+                          '#_api_url private method'
+      expected = expect { provider.data }
+      expected.to raise_error(NotImplementedError, api_url_error_msg)
+    end
 
-    it "sets default user_agent options" do
-      expect(provider.options).to eq({ 'User-Agent' => "VideoInfo/#{VideoInfo::VERSION}" })
+    it 'sets default user_agent options' do
+      user_agent_hash = { 'User-Agent' => "VideoInfo/#{VideoInfo::VERSION}" }
+      expect(provider.options).to eq(user_agent_hash)
     end
 
-    context "with custom User-Agent options" do
+    context 'with custom User-Agent options' do
       let(:options) { { 'User-Agent' => 'Test User Agent / 1.0' } }
 
-      it "sets the option" do
-        expect(provider.options).to eq({ 'User-Agent' => 'Test User Agent / 1.0' })
+      it 'sets the option' do
+        expect(provider.options).to eq('User-Agent' => 'Test User Agent / 1.0')
       end
     end
 
-    context "with Referer options" do
+    context 'with Referer options' do
       let(:options) { { referer: 'http://google.com' } }
 
-      it "sets the option" do
-        expect(provider.options).to include({ 'Referer' => 'http://google.com' })
+      it 'sets the option' do
+        expect(provider.options).to include('Referer' => 'http://google.com')
       end
     end
   end
 
-  describe "embed_code" do
-    it "supports url_attributes option" do
-      expect(provider.embed_code(url_attributes: { foo: 'bar' })).to eq '<iframe src="//foo.com?foo=bar" frameborder="0"></iframe>'
+  describe 'embed_code' do
+    it 'supports url_attributes option' do
+      embed_str = '<iframe src="//foo.com?foo=bar" frameborder="0"></iframe>'
+      expected = expect(provider.embed_code(url_attributes: { foo: 'bar' }))
+      expected.to eq embed_str
     end
 
-    it "supports url_attributes option" do
-      expect(provider.embed_code(iframe_attributes: { foo: 'bar' })).to eq '<iframe src="//foo.com" frameborder="0" foo="bar"></iframe>'
+    it 'supports url_attributes option' do
+      embed_str = '<iframe src="//foo.com" frameborder="0" foo="bar"></iframe>'
+      expected = expect(provider.embed_code(iframe_attributes: { foo: 'bar' }))
+      expected.to eq embed_str
     end
   end
 
-  describe ".usable?" do
-    it { expect { VideoInfo::Provider.usable?('url') }.to raise_error(NotImplementedError, 'Provider class must implement .usable? public method') }
+  describe '.usable?' do
+    it 'should raise eror if not usable' do
+      error_msg = 'Provider class must implement .usable? public method'
+      expected = expect do
+        VideoInfo::Provider.usable?('url')
+      end
+      expected.to raise_error(NotImplementedError, error_msg)
+    end
   end
-
 end

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

@@ -2,28 +2,27 @@
 require 'spec_helper'
 
 describe VideoInfo::Providers::Dailymotion do
-
-  describe ".usable?" do
+  describe '.usable?' do
     subject { VideoInfo::Providers::Dailymotion.usable?(url) }
 
-    context "with dailymotion url" do
+    context 'with dailymotion url' do
       let(:url) { 'http://www.dailymotion.com/video/x7lni3' }
       it { is_expected.to be_truthy }
     end
 
-    context "with dai.ly url" do
+    context 'with dai.ly url' do
       let(:url) { 'http://dai.ly/video/x7lni3' }
       it { is_expected.to be_truthy }
     end
 
-    context "with other url" do
+    context 'with other url' do
       let(:url) { 'http://www.youtube.com/x7lni3' }
       it { is_expected.to be_falsey }
     end
   end
 
-  describe "#available?" do
-    context "with valid video", :vcr do
+  describe '#available?' do
+    context 'with valid video', :vcr do
       subject { VideoInfo.new('http://www.dailymotion.com/video/x7lni3') }
 
       describe '#available?' do
@@ -32,7 +31,7 @@ describe VideoInfo::Providers::Dailymotion do
       end
     end
 
-    context "with 'This video does not exist or has been deleted.", :vcr do
+    context 'with invalid video url', :vcr do
       subject { VideoInfo.new('http://www.dailymotion.com/video/inValIdiD') }
 
       describe '#available?' do
@@ -42,7 +41,7 @@ describe VideoInfo::Providers::Dailymotion do
     end
   end
 
-  context "with video x7lni3", :vcr do
+  context 'with video x7lni3', :vcr do
     subject { VideoInfo.new('http://www.dailymotion.com/video/x7lni3') }
 
     describe '#provider' do
@@ -67,7 +66,9 @@ describe VideoInfo::Providers::Dailymotion do
 
     describe '#embed_code' do
       subject { super().embed_code }
-      it { is_expected.to eq '<iframe src="//www.dailymotion.com/embed/video/x7lni3?autoplay=0" frameborder="0"></iframe>' }
+      embed_code = '<iframe src="//www.dailymotion.com/embed/video/x7lni3?' \
+                   'autoplay=0" frameborder="0"></iframe>'
+      it { is_expected.to eq embed_code }
     end
 
     describe '#title' do
@@ -77,17 +78,23 @@ describe VideoInfo::Providers::Dailymotion do
 
     describe '#description' do
       subject { super().description }
-      it { is_expected.to eq 'Super Rémi Kart est un jeu vidéo de course développé et édité par N\'Importe Quoi TV. Sa principale originalité vient de sa réalité...<br />L\'aventure continue sur www.nimportequi.com' }
+      description_text = 'Super Rémi Kart est un jeu vidéo de course ' \
+                         'développé et édité par N\'Importe Quoi TV. ' \
+                         'Sa principale originalité vient de sa réalité' \
+                         '...<br />L\'aventure continue sur www.nimportequi.com'
+      it { is_expected.to eq description_text }
     end
 
     describe '#keywords' do
       subject { super().keywords }
-      it { is_expected.to eq ["rémi","remi","gaillard","imposteur","imposture","mario","bros","kart","nintendo","jeux"] }
+      keywords_list = %w(rémi remi gaillard imposteur imposture
+                         mario bros kart nintendo jeux)
+      it { is_expected.to eq keywords_list }
     end
 
     describe '#duration' do
       subject { super().duration }
-      it { is_expected.to eq 136}
+      it { is_expected.to eq 136 }
     end
 
     describe '#width' do
@@ -102,7 +109,9 @@ describe VideoInfo::Providers::Dailymotion do
 
     describe '#date' do
       subject { super().date }
-      it { is_expected.to eq Time.parse('2008-12-03 16:29:31 UTC', Time.now.utc) }
+      it 'should show correct date' do
+        is_expected.to eq Time.parse('2008-12-03 16:29:31 UTC', Time.now.utc)
+      end
     end
 
     describe '#thumbnail_small' do
@@ -122,25 +131,35 @@ describe VideoInfo::Providers::Dailymotion do
 
     describe '#view_count' do
       subject { super().view_count }
-      it { is_expected.to be > 10000000 }
+      it { is_expected.to be > 10_000_000 }
     end
   end
 
-  context "with video x7lni3 and url_attributes", :vcr do
+  context 'with video x7lni3 and url_attributes', :vcr do
     subject { VideoInfo.new('http://www.dailymotion.com/video/x7lni3') }
 
-    it { expect(subject.embed_code(url_attributes: { autoplay: 1 })).to match(/autoplay=1/) }
+    it do
+      embed_code = subject.embed_code(url_attributes: { autoplay: 1 })
+      expect(embed_code).to match(/autoplay=1/)
+    end
   end
 
-  context "with video x7lni3 and iframe_attributes", :vcr do
+  context 'with video x7lni3 and iframe_attributes', :vcr do
     subject { VideoInfo.new('http://www.dailymotion.com/video/x7lni3') }
 
-    it { expect(subject.embed_code(iframe_attributes: { width: 800, height: 600 })).to match(/width="800"/) }
-    it { expect(subject.embed_code(iframe_attributes: { width: 800, height: 600 })).to match(/height="600"/) }
+    it 'should properly apply dimensions attributes' do
+      dimensions = { width: 800, height: 600 }
+      embed_code = subject.embed_code(iframe_attributes: dimensions)
+      expect(embed_code).to match(/width="800"/)
+      expect(embed_code).to match(/height="600"/)
+    end
   end
 
-  context "with video x7lni3 in text", :vcr do
-    subject { VideoInfo.new('<a href="http://www.dailymotion.com/video/x7lni3">http://www.dailymotion.com/video/x7lni3</a>') }
+  context 'with video x7lni3 in text', :vcr do
+    url_in_text = '<a href="http://www.dailymotion.com/video/x7lni3">' \
+                  'http://www.dailymotion.com/video/x7lni3</a>'
+
+    subject { VideoInfo.new(url_in_text) }
 
     describe '#provider' do
       subject { super().provider }
@@ -152,5 +171,4 @@ describe VideoInfo::Providers::Dailymotion do
       it { is_expected.to eq 'x7lni3' }
     end
   end
-
 end

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

@@ -1,242 +1,277 @@
 require 'spec_helper'
 
-describe VideoInfo::Providers::Vimeo do
-  before(:all) do
-    VideoInfo.provider_api_keys = { }
-  end
+[nil, '6b66b015a3504793b4f541d878f46ff6'].each do |api_key|
+  describe VideoInfo::Providers::Vimeo do
+    before(:all) do
+      VideoInfo.provider_api_keys = { vimeo: api_key }
+    end
 
-  describe '.usable?' do
-    subject { VideoInfo::Providers::Vimeo.usable?(url) }
+    describe '.usable?' do
+      subject { VideoInfo::Providers::Vimeo.usable?(url) }
 
-    context 'with vimeo url' do
-      let(:url) { 'http://www.vimeo.com/898029' }
-      it { is_expected.to be_truthy }
-    end
+      context 'with vimeo url' do
+        let(:url) { 'http://www.vimeo.com/898029' }
+        it { is_expected.to be_truthy }
+      end
 
-    context 'with Vimeo OnDemand url' do
-      let(:url) { 'https://vimeo.com/ondemand/less/101677664' }
-      it { is_expected.to be_truthy }
-    end
+      context 'with Vimeo OnDemand url' do
+        let(:url) { 'https://vimeo.com/ondemand/less/101677664' }
+        it { is_expected.to be_truthy }
+      end
 
-    context 'with Vimeo Channels url' do
-      let(:url) { 'https://vimeo.com/channels/any_channel/111431415' }
-      it { is_expected.to be_truthy }
-    end
+      context 'with Vimeo Channels url' do
+        let(:url) { 'https://vimeo.com/channels/any_channel/111431415' }
+        it { is_expected.to be_truthy }
+      end
 
-    context "with Vimeo Review url" do
-      let(:url) { 'https://vimeo.com/user39798190/review/126641548/8a56234e32' }
-      it { is_expected.to be_truthy }
-    end
+      context 'with Vimeo Review url' do
+        video_url = 'https://vimeo.com/user39798190/review/126641548/8a56234e32'
+        let(:url) { video_url }
+        it { is_expected.to be_truthy }
+      end
 
-    context 'with vimeo album url' do
-      let(:url) { 'http://vimeo.com/album/2755718' }
-      it { is_expected.to be_falsey }
-    end
+      context 'with vimeo album url' do
+        let(:url) { 'http://vimeo.com/album/2755718' }
+        it { is_expected.to be_falsey }
+      end
 
-    context 'with vimeo hubnub embed url' do
-      let(:url) { 'http://player.vimeo.com/hubnut/album/2755718' }
-      it { is_expected.to be_falsey }
-    end
+      context 'with vimeo hubnub embed url' do
+        let(:url) { 'http://player.vimeo.com/hubnut/album/2755718' }
+        it { is_expected.to be_falsey }
+      end
 
-    context 'with other url' do
-      let(:url) { 'http://www.youtube.com/898029' }
-      it { is_expected.to be_falsey }
+      context 'with other url' do
+        let(:url) { 'http://www.youtube.com/898029' }
+        it { is_expected.to be_falsey }
+      end
     end
-  end
 
-  describe '#available?' do
-    context 'with valid video', :vcr do
-      subject { VideoInfo.new('http://www.vimeo.com/898029') }
+    describe '#available?' do
+      context 'with valid video', :vcr do
+        subject { VideoInfo.new('http://vimeo.com/98605382') }
 
-      describe '#available?' do
-        it { is_expected.to be_available }
+        describe '#available?' do
+          it { is_expected.to be_available }
+        end
       end
-    end
 
-    context "with 'this video does not exist' video", :vcr do
-      subject { VideoInfo.new('http://vimeo.com/593123111') }
+      context "with 'this video does not exist' video", :vcr do
+        subject { VideoInfo.new('http://vimeo.com/59312311') }
 
-      describe '#available?' do
-        it { is_expected.to_not be_available }
+        describe '#available?' do
+          it { is_expected.to_not be_available }
+        end
       end
-    end
 
-    context "with 'password required' video", :vcr do
-      subject { VideoInfo.new('http://vimeo.com/74636562') }
+      context "with 'password required' video", :vcr do
+        subject { VideoInfo.new('http://vimeo.com/74636562') }
 
-      describe '#available?' do
-        it { is_expected.to_not be_available }
+        describe '#available?' do
+          it { is_expected.to_not be_available }
+        end
       end
     end
-  end
 
-  context 'with video 898029', :vcr do
-    subject { VideoInfo.new('http://www.vimeo.com/898029') }
+    context 'with video 136971428', :vcr do
+      subject { VideoInfo.new('https://vimeo.com/136971428') }
 
-    describe '#provider' do
-      subject { super().provider }
-      it { is_expected.to eq 'Vimeo' }
-    end
+      describe '#provider' do
+        subject { super().provider }
+        it { is_expected.to eq 'Vimeo' }
+      end
 
-    describe '#video_id' do
-      subject { super().video_id }
-      it { is_expected.to eq '898029' }
-    end
+      describe '#video_id' do
+        subject { super().video_id }
+        it { is_expected.to eq '136971428' }
+      end
 
-    describe '#url' do
-      subject { super().url }
-      it { is_expected.to eq 'http://www.vimeo.com/898029' }
-    end
+      describe '#url' do
+        subject { super().url }
+        it { is_expected.to eq 'https://vimeo.com/136971428' }
+      end
 
-    describe '#embed_url' do
-      subject { super().embed_url }
-      it { is_expected.to eq '//player.vimeo.com/video/898029' }
-    end
+      describe '#embed_url' do
+        subject { super().embed_url }
+        it { is_expected.to eq '//player.vimeo.com/video/136971428' }
+      end
 
-    describe '#embed_code' do
-      subject { super().embed_code }
-      it { is_expected.to eq '<iframe src="//player.vimeo.com/video/898029?autoplay=0&byline=0&portrait=0&title=0" frameborder="0"></iframe>' }
-    end
+      describe '#embed_code' do
+        subject { super().embed_code }
+        embed_code = '<iframe src="//player.vimeo.com/video/136971428?' \
+                     'title=0&byline=0&portrait=0&autoplay=0" ' \
+                     'frameborder="0"></iframe>'
+        it { is_expected.to eq embed_code }
+      end
 
-    describe '#title' do
-      subject { super().title }
-      it { is_expected.to eq 'Cherry Bloom - King Of The Knife' }
-    end
+      describe '#title' do
+        subject { super().title }
+        it { is_expected.to eq "Deafheaven - 'New Bermuda' Trailer" }
+      end
 
-    describe '#description' do
-      subject { super().description }
-      it { is_expected.to eq 'The first video from the upcoming album Secret Sounds, to download in-stores April 14. Checkout http://www.cherrybloom.net' }
-    end
+      describe '#description' do
+        subject { super().description }
+        description = "'New Bermuda,' the upcoming album from Deafheaven, " \
+                      'is available October 2nd'
+        it { is_expected.to eq description }
+      end
 
-    describe '#keywords' do
-      subject { super().keywords }
-      it { is_expected.to eq 'cherry bloom, secret sounds, king of the knife, rock, alternative' }
-    end
+      describe '#keywords' do
+        subject { super().keywords }
+        it 'should return a string list of keywords' do
+          keywords_list = []
+          is_expected.to eq keywords_list
+        end
+      end
 
-    describe '#duration' do
-      subject { super().duration }
-      it { is_expected.to eq 175 }
-    end
+      describe '#duration' do
+        subject { super().duration }
+        it { is_expected.to eq 59 }
+      end
 
-    describe '#width' do
-      subject { super().width }
-      it { is_expected.to eq 1280 }
-    end
+      describe '#width' do
+        subject { super().width }
+        it { is_expected.to eq 1280 }
+      end
 
-    describe '#height' do
-      subject { super().height }
-      it { is_expected.to eq 720 }
-    end
+      describe '#height' do
+        subject { super().height }
+        it { is_expected.to eq 720 }
+      end
 
-    describe '#date' do
-      subject { super().date }
-      it { is_expected.to eq Time.parse('2008-04-14T17:10:39+00:00', Time.now.utc).utc }
-    end
+      describe '#date' do
+        subject { super().date }
+        it 'should have correct upload date' do
+          is_expected.to eq Time.parse('2015-08-21T21:37:43+00:00',
+                                       Time.now.utc).utc
+        end
+      end
 
-    describe '#thumbnail_small' do
-      subject { super().thumbnail_small }
-      it { is_expected.to eq 'https://i.vimeocdn.com/video/34373130_100x75.jpg' }
-    end
+      describe '#thumbnail_small' do
+        subject { super().thumbnail_small }
+        thumbnail_url = 'https://i.vimeocdn.com/video/531688239_100x75.jpg'
+        it { is_expected.to eq thumbnail_url }
+      end
 
-    describe '#thumbnail_medium' do
-      subject { super().thumbnail_medium }
-      it { is_expected.to eq 'https://i.vimeocdn.com/video/34373130_200x150.jpg' }
-    end
+      describe '#thumbnail_medium' do
+        subject { super().thumbnail_medium }
+        thumbnail_url = 'https://i.vimeocdn.com/video/531688239_200x150.jpg'
+        it { is_expected.to eq thumbnail_url }
+      end
 
-    describe '#thumbnail_large' do
-      subject { super().thumbnail_large }
-      it { is_expected.to eq 'https://i.vimeocdn.com/video/34373130_640.jpg' }
-    end
+      describe '#thumbnail_large' do
+        subject { super().thumbnail_large }
+        thumbnail_url = 'https://i.vimeocdn.com/video/531688239_640.jpg'
+        it { is_expected.to eq thumbnail_url }
+      end
 
-    describe '#author_thumbnail' do
-      subject { super().author_thumbnail }
-      it { is_expected.to eq 'https://i.vimeocdn.com/portrait/2577152_75x75.jpg' }
-    end
+      describe '#author_thumbnail' do
+        subject { super().author_thumbnail }
+        thumbnail_url = 'https://i.vimeocdn.com/portrait/327118_75x75.jpg'
+        it { is_expected.to eq thumbnail_url }
+      end
 
-    describe '#author' do
-      subject { super().author }
-      it { is_expected.to eq 'Octave Zangs' }
-    end
+      describe '#author' do
+        subject { super().author }
+        it { is_expected.to eq 'Anti Records' }
+      end
+
+      describe '#author_url' do
+        subject { super().author_url }
+        it { is_expected.to eq 'https://vimeo.com/antirecords' }
+      end
 
-    describe '#view_count' do
-      subject { super().view_count }
-      it { is_expected.to be > 4000 }
+      describe '#view_count' do
+        subject { super().view_count }
+        it { is_expected.to be > 80 }
+      end
     end
-  end
 
-  context 'with video 898029 and url_attributes', :vcr do
-    subject { VideoInfo.new('http://www.vimeo.com/898029') }
+    context 'with video 898029 and url_attributes', :vcr do
+      subject { VideoInfo.new('http://www.vimeo.com/898029') }
 
-    it { expect(subject.embed_code(url_attributes: { autoplay: 1 })).to match(/autoplay=1/) }
-  end
+      it 'should add URL attribute' do
+        attributes = { autoplay: 1 }
+        embed_code = subject.embed_code(url_attributes: attributes)
+        expect(embed_code).to match(/autoplay=1/)
+      end
+    end
 
-  context 'with video 898029 and iframe_attributes', :vcr do
-    subject { VideoInfo.new('http://www.vimeo.com/898029') }
+    context 'with video 898029 and iframe_attributes', :vcr do
+      subject { VideoInfo.new('http://www.vimeo.com/898029') }
 
-    it { expect(subject.embed_code(iframe_attributes: { width: 800, height: 600 })).to match(/width="800"/) }
-    it { expect(subject.embed_code(iframe_attributes: { width: 800, height: 600 })).to match(/height="600"/) }
-  end
+      it 'should have proper dimensions' do
+        dimensions = { width: 800, height: 600 }
+        embed_code = subject.embed_code(iframe_attributes: dimensions)
+        expect(embed_code).to match(/width="800"/)
+        expect(embed_code).to match(/height="600"/)
+      end
+    end
 
-  context 'with video 898029 in /group/ url', :vcr do
-    subject { VideoInfo.new('http://vimeo.com/groups/1234/videos/898029') }
+    context 'with video 898029 in /group/ url', :vcr do
+      subject { VideoInfo.new('http://vimeo.com/groups/1234/videos/898029') }
 
-    describe '#provider' do
-      subject { super().provider }
-      it { is_expected.to eq 'Vimeo' }
-    end
+      describe '#provider' do
+        subject { super().provider }
+        it { is_expected.to eq 'Vimeo' }
+      end
 
-    describe '#video_id' do
-      subject { super().video_id }
-      it { is_expected.to eq '898029' }
+      describe '#video_id' do
+        subject { super().video_id }
+        it { is_expected.to eq '898029' }
+      end
     end
-  end
 
-  context 'with video 898029 in /group/ url', :vcr do
-    subject { VideoInfo.new('http://player.vimeo.com/video/898029') }
+    context 'with video 898029 in /group/ url', :vcr do
+      subject { VideoInfo.new('http://player.vimeo.com/video/898029') }
 
-    describe '#provider' do
-      subject { super().provider }
-      it { is_expected.to eq 'Vimeo' }
-    end
+      describe '#provider' do
+        subject { super().provider }
+        it { is_expected.to eq 'Vimeo' }
+      end
 
-    describe '#video_id' do
-      subject { super().video_id }
-      it { is_expected.to eq '898029' }
+      describe '#video_id' do
+        subject { super().video_id }
+        it { is_expected.to eq '898029' }
+      end
     end
-  end
 
-  context 'with video 898029 in text', :vcr do
-    subject { VideoInfo.new('<a href="http://www.vimeo.com/898029">http://www.vimeo.com/898029</a>') }
+    context 'with video 898029 in text', :vcr do
+      video_url_in_text = '<a href="http://www.vimeo.com/898029">' \
+                          'http://www.vimeo.com/898029</a>'
+      subject { VideoInfo.new(video_url_in_text) }
 
-    describe '#provider' do
-      subject { super().provider }
-      it { is_expected.to eq 'Vimeo' }
-    end
+      describe '#provider' do
+        subject { super().provider }
+        it { is_expected.to eq 'Vimeo' }
+      end
 
-    describe '#video_id' do
-      subject { super().video_id }
-      it { is_expected.to eq '898029' }
+      describe '#video_id' do
+        subject { super().video_id }
+        it { is_expected.to eq '898029' }
+      end
     end
-  end
 
-  context 'with video 101677664 in /ondemand/ url', :vcr do
-    subject { VideoInfo.new('https://vimeo.com/ondemand/less/101677664') }
+    context 'with video 101677664 in /ondemand/ url', :vcr do
+      subject { VideoInfo.new('https://vimeo.com/ondemand/less/101677664') }
 
-    its(:provider) { should eq 'Vimeo' }
-    its(:video_id) { should eq '101677664' }
-  end
+      its(:provider) { should eq 'Vimeo' }
+      its(:video_id) { should eq '101677664' }
+    end
 
-  context 'with video 111431415 in /channels/*/ url', :vcr do
-    subject { VideoInfo.new('https://vimeo.com/channels/some_channel1/111431415') }
+    context 'with video 111431415 in /channels/*/ url', :vcr do
+      video_url = 'https://vimeo.com/channels/some_channel1/111431415'
+      subject { VideoInfo.new(video_url) }
 
-    its(:provider) { should eq 'Vimeo' }
-    its(:video_id) { should eq '111431415' }
-  end
+      its(:provider) { should eq 'Vimeo' }
+      its(:video_id) { should eq '111431415' }
+    end
 
-  context "with video 126641548 in /user*/review/126641548/* url", :vcr do
-    subject { VideoInfo.new('http://www.vimeo.com/user39798190/review/126641548/8a56234e32') }
+    context 'with video 126641548 in /user*/review/126641548/* url', :vcr do
+      video_url = 'http://www.vimeo.com/user39798190/review/126641548/8a56234e32'
+      subject { VideoInfo.new(video_url) }
 
-    its(:provider) { should eq 'Vimeo' }
-    its(:video_id) { should eq '126641548' }
+      its(:provider) { should eq 'Vimeo' }
+      its(:video_id) { should eq '126641548' }
+    end
   end
 end