yt 0.32.6 → 0.33.0

data/spec/models/request_spec.rb

@@ -1,204 +0,0 @@
-require 'spec_helper'
-require 'yt/request'
-
-
-describe Yt::Request do
-  subject(:request) { Yt::Request.new host: 'example.com' }
-  let(:response) { response_class.new nil, nil, nil }
-  let(:response_body) { }
-
-  describe '#run' do
-    context 'given a request that returns' do
-      before { allow(response).to receive(:body).and_return response_body }
-      before { expect(Net::HTTP).to receive(:start).once.and_return response }
-
-      context 'a success code 2XX' do
-        let(:response_class) { Net::HTTPOK }
-
-        it { expect{request.run}.not_to fail }
-      end
-
-      context 'an error code 5XX' do
-        let(:response_class) { Net::HTTPServerError }
-        let(:retry_response) { retry_response_class.new nil, nil, nil }
-        before { allow(retry_response).to receive(:body) }
-        before { expect(Net::HTTP).to receive(:start).at_least(:once).and_return retry_response }
-
-        context 'every time' do
-          let(:retry_response_class) { Net::HTTPServerError }
-
-          it { expect{request.run}.to fail }
-        end
-
-        context 'but returns a success code 2XX the second time' do
-          let(:retry_response_class) { Net::HTTPOK }
-
-          it { expect{request.run}.not_to fail }
-        end
-      end
-
-      context 'an error code 403 with a "quota exceeded message"' do
-        let(:response_class) { Net::HTTPForbidden }
-        let(:retry_response) { retry_response_class.new nil, nil, nil }
-        let(:response_body) { "{\n \"error\": {\n  \"errors\": [\n   {\n    \"domain\": \"youtube.quota\",\n    \"reason\": \"quotaExceeded\",\n    \"message\": \"The request cannot be completed because you have exceeded your \\u003ca href=\\\"/youtube/v3/getting-started#quota\\\"\\u003equota\\u003c/a\\u003e.\"\n   }\n  ],\n  \"code\": 403,\n  \"message\": \"The request cannot be completed because you have exceeded your \\u003ca href=\\\"/youtube/v3/getting-started#quota\\\"\\u003equota\\u003c/a\\u003e.\"\n }\n}\n" }
-        before { allow(retry_response).to receive(:body) }
-        before { expect(Net::HTTP).to receive(:start).at_least(:once).and_return retry_response }
-
-        context 'every time' do
-          let(:retry_response_class) { Net::HTTPForbidden }
-
-          it { expect{request.run}.to fail }
-        end
-
-        context 'but returns a success code 2XX the second time' do
-          let(:retry_response_class) { Net::HTTPOK }
-
-          it { expect{request.run}.not_to fail }
-        end
-      end
-
-      context 'an error code 401' do
-        let(:response_class) { Net::HTTPUnauthorized }
-
-        it { expect{request.run}.to fail }
-      end
-
-      context 'any other non-2XX error code' do
-        let(:response_class) { Net::HTTPNotFound }
-
-        it { expect{request.run}.to fail }
-      end
-    end
-
-    # TODO: clean up the following tests, removing repetitions
-    context 'given a request that raises' do
-      before { expect(Net::HTTP).to receive(:start).once.and_raise http_error }
-
-      # NOTE: This test is just a reflection of YouTube irrational behavior of
-      # being unavailable once in a while, and therefore causing Net::HTTP to
-      # fail, although retrying after some seconds works.
-      context 'an OpenSSL::SSL::SSLError' do
-        let(:http_error) { OpenSSL::SSL::SSLError.new }
-
-        context 'every time' do
-          before { expect(Net::HTTP).to receive(:start).at_least(:once).and_raise http_error }
-
-          it { expect{request.run}.to fail }
-        end
-
-        context 'but works the second time' do
-          before { expect(Net::HTTP).to receive(:start).at_least(:once).and_return retry_response }
-          before { allow(retry_response).to receive(:body) }
-          let(:retry_response) { Net::HTTPOK.new nil, nil, nil }
-
-          it { expect{request.run}.not_to fail }
-        end
-      end
-
-      # NOTE: This test is just a reflection of YouTube irrational behavior of
-      # being unavailable once in a while, and therefore causing Net::HTTP to
-      # fail, although retrying after some seconds works.
-      context 'an Errno::ETIMEDOUT' do
-        let(:http_error) { Errno::ETIMEDOUT.new }
-
-        context 'every time' do
-          before { expect(Net::HTTP).to receive(:start).at_least(:once).and_raise http_error }
-
-          it { expect{request.run}.to fail }
-        end
-
-        context 'but works the second time' do
-          before { expect(Net::HTTP).to receive(:start).at_least(:once).and_return retry_response }
-          before { allow(retry_response).to receive(:body) }
-          let(:retry_response) { Net::HTTPOK.new nil, nil, nil }
-
-          it { expect{request.run}.not_to fail }
-        end
-      end
-
-      # NOTE: This test is just a reflection of YouTube irrational behavior of
-      # being unavailable once in a while, and therefore causing Net::HTTP to
-      # fail, although retrying after some seconds works.
-      context 'an Errno::ENETUNREACH' do
-        let(:http_error) { Errno::ENETUNREACH.new }
-
-        context 'every time' do
-          before { expect(Net::HTTP).to receive(:start).at_least(:once).and_raise http_error }
-
-          it { expect{request.run}.to fail }
-        end
-
-        context 'but works the second time' do
-          before { expect(Net::HTTP).to receive(:start).at_least(:once).and_return retry_response }
-          before { allow(retry_response).to receive(:body) }
-          let(:retry_response) { Net::HTTPOK.new nil, nil, nil }
-
-          it { expect{request.run}.not_to fail }
-        end
-      end
-
-      # NOTE: This test is just a reflection of YouTube irrational behavior of
-      # being unavailable once in a while, and therefore causing Net::HTTP to
-      # fail, although retrying after some seconds works.
-      context 'an Errno::EHOSTUNREACH' do
-        let(:http_error) { Errno::EHOSTUNREACH.new }
-
-        context 'every time' do
-          before { expect(Net::HTTP).to receive(:start).at_least(:once).and_raise http_error }
-
-          it { expect{request.run}.to fail }
-        end
-
-        context 'but works the second time' do
-          before { expect(Net::HTTP).to receive(:start).at_least(:once).and_return retry_response }
-          before { allow(retry_response).to receive(:body) }
-          let(:retry_response) { Net::HTTPOK.new nil, nil, nil }
-
-          it { expect{request.run}.not_to fail }
-        end
-      end
-
-      # NOTE: This test is just a reflection of YouTube irrational behavior of
-      # being unavailable once in a while, and therefore causing Net::HTTP to
-      # fail, although retrying after some seconds works.
-      context 'an OpenSSL::SSL::SSLErrorWaitReadable', ruby21: true do
-        let(:http_error) { OpenSSL::SSL::SSLErrorWaitReadable.new }
-
-        context 'every time' do
-          before { expect(Net::HTTP).to receive(:start).at_least(:once).and_raise http_error }
-
-          it { expect{request.run}.to fail }
-        end
-
-        context 'but works the second time' do
-          before { expect(Net::HTTP).to receive(:start).at_least(:once).and_return retry_response }
-          before { allow(retry_response).to receive(:body) }
-          let(:retry_response) { Net::HTTPOK.new nil, nil, nil }
-
-          it { expect{request.run}.not_to fail }
-        end
-      end
-
-      # NOTE: This test is just a reflection of YouTube irrational behavior of
-      # being unavailable once in a while, and therefore causing Net::HTTP to
-      # fail, although retrying after some seconds works.
-      context 'a SocketError', ruby21: true do
-        let(:http_error) { SocketError.new }
-
-        context 'every time' do
-          before { expect(Net::HTTP).to receive(:start).at_least(:once).and_raise http_error }
-
-          it { expect{request.run}.to fail }
-        end
-
-        context 'but works the second time' do
-          before { expect(Net::HTTP).to receive(:start).at_least(:once).and_return retry_response }
-          before { allow(retry_response).to receive(:body) }
-          let(:retry_response) { Net::HTTPOK.new nil, nil, nil }
-
-          it { expect{request.run}.not_to fail }
-        end
-      end
-    end
-  end
-end

data/spec/models/resource_spec.rb

@@ -1,42 +0,0 @@
-require 'spec_helper'
-require 'yt/models/resource'
-
-describe Yt::Resource do
-  subject(:resource) { Yt::Resource.new attrs }
-
-  describe '#public?' do
-    context 'given fetching a status returns privacyStatus "public"' do
-      let(:attrs) { {status: {"privacyStatus"=>"public"}} }
-      it { expect(resource).to be_public }
-    end
-
-    context 'given fetching a status does not return privacyStatus "public"' do
-      let(:attrs) { {status: {}} }
-      it { expect(resource).not_to be_public }
-    end
-  end
-
-  describe '#private?' do
-    context 'given fetching a status returns privacyStatus "private"' do
-      let(:attrs) { {status: {"privacyStatus"=>"private"}} }
-      it { expect(resource).to be_private }
-    end
-
-    context 'given fetching a status does not return privacyStatus "private"' do
-      let(:attrs) { {status: {}} }
-      it { expect(resource).not_to be_private }
-    end
-  end
-
-  describe '#unlisted?' do
-    context 'given fetching a status returns privacyStatus "unlisted"' do
-      let(:attrs) { {status: {"privacyStatus"=>"unlisted"}} }
-      it { expect(resource).to be_unlisted }
-    end
-
-    context 'given fetching a status does not return privacyStatus "unlisted"' do
-      let(:attrs) { {status: {}} }
-      it { expect(resource).not_to be_unlisted }
-    end
-  end
-end

data/spec/models/right_owner_spec.rb

@@ -1,71 +0,0 @@
-require 'spec_helper'
-require 'yt/models/right_owner'
-
-describe Yt::RightOwner do
-  subject(:right_owner) { Yt::RightOwner.new data: data }
-  let(:data) { {} }
-
-  describe '#ratio' do
-    context 'given fetching an owner returns a ratio' do
-      let(:data) { {"ratio"=>"20.0"} }
-      it { expect(right_owner.ratio).to eq 20 }
-    end
-  end
-
-  describe '#owner' do
-    context 'given fetching an owner returns an owner name' do
-      let(:data) { {"owner"=>"FullScreen"} }
-      it { expect(right_owner.owner).to eq 'FullScreen' }
-    end
-  end
-
-  describe '#owner' do
-    context 'given fetching an owner returns a publisher name' do
-      let(:data) { {"publisher"=>"Third Party"} }
-      it { expect(right_owner.publisher).to eq 'Third Party' }
-    end
-
-    context 'given fetching an owner does not return a publisher name' do
-      it { expect(right_owner.publisher).to be_nil }
-    end
-  end
-
-  describe '#included_territories' do
-    context 'given fetching an owner returns included territories' do
-      let(:data) { {"type"=>"include", "territories"=>["US", "CA"]} }
-      it { expect(right_owner.included_territories).to eq %w(US CA) }
-    end
-
-    context 'given fetching an owner does not return included territories' do
-      it { expect(right_owner.included_territories).to be_nil }
-    end
-  end
-
-  describe '#excluded_territories' do
-    context 'given fetching an owner returns excluded territories' do
-      let(:data) { {"type"=>"exclude", "territories"=>["US", "CA"]} }
-      it { expect(right_owner.excluded_territories).to eq %w(US CA) }
-    end
-
-    context 'given fetching an owner does not return excluded territories' do
-      it { expect(right_owner.excluded_territories).to be_nil }
-    end
-  end
-
-  describe '#everywhere?' do
-    context 'given fetching an owner returns zero excluded territories' do
-      let(:data) { {"type"=>"exclude", "territories"=>[]} }
-      it { expect(right_owner).to be_everywhere }
-    end
-
-    context 'given fetching an owner returns no excluded territories' do
-      let(:data) { {"type"=>"exclude"} }
-      it { expect(right_owner).to be_everywhere }
-    end
-
-    context 'given fetching an owner returns included territories' do
-      let(:data) { {"type"=>"include", "territories"=>[]} }
-      it { expect(right_owner).not_to be_everywhere }
-    end
-  end
-end

data/spec/models/snippet_spec.rb

@@ -1,13 +0,0 @@
-require 'spec_helper'
-require 'yt/models/snippet'
-
-describe Yt::Snippet do
-  subject(:snippet) { Yt::Snippet.new data: data }
-
-  describe '#data' do
-    let(:data) { {"key"=>"value"} }
-    specify 'returns the data the snippet was initialized with' do
-      expect(snippet.data).to eq data
-    end
-  end
-end

data/spec/models/statistics_set_spec.rb

@@ -1,13 +0,0 @@
-require 'spec_helper'
-require 'yt/models/statistics_set'
-
-describe Yt::StatisticsSet do
-  subject(:statistics_set) { Yt::StatisticsSet.new data: data }
-
-  describe '#data' do
-    let(:data) { {"key"=>"value"} }
-    specify 'returns the data the statistics set was initialized with' do
-      expect(statistics_set.data).to eq data
-    end
-  end
-end

data/spec/models/status_spec.rb

@@ -1,13 +0,0 @@
-require 'spec_helper'
-require 'yt/models/status'
-
-describe Yt::Status do
-  subject(:status) { Yt::Status.new data: data }
-
-  describe '#data' do
-    let(:data) { {"key"=>"value"} }
-    specify 'returns the data the status was initialized with' do
-      expect(status.data).to eq data
-    end
-  end
-end

data/spec/models/subscription_spec.rb

@@ -1,30 +0,0 @@
-require 'spec_helper'
-require 'yt/models/subscription'
-
-describe Yt::Subscription do
-  subject(:subscription) { Yt::Subscription.new id: id }
-
-  describe '#exists?' do
-    context 'given a subscription with an id' do
-      let(:id) { 'CBl6OoF0BpiV' }
-      it { expect(subscription).to exist }
-    end
-
-    context 'given a subscription without an id' do
-      let(:id) { nil }
-      it { expect(subscription).not_to exist }
-    end
-  end
-
-  describe '#delete' do
-    let(:id) { 'CBl6OoF0BpiV' }
-    before { expect(subscription).to behave }
-
-    context 'given an existing subscription' do
-      let(:behave) { receive(:do_delete).and_yield }
-
-      it { expect(subscription.delete).to be true }
-      it { expect{subscription.delete}.to change{subscription.exists?} }
-    end
-  end
-end

data/spec/models/url_spec.rb

@@ -1,78 +0,0 @@
-require 'spec_helper'
-require 'yt/models/url'
-
-describe Yt::URL do
-  subject(:url) { Yt::URL.new text }
-
-  context 'given a YouTube playlist URL' do
-    let(:text) { "https://www.youtube.com/playlist?list=#{id}" }
-
-    describe 'works with existing playlists' do
-      let(:id) { 'LLxO1tY8h1AhOz0T4ENwmpow' }
-      it {expect(url.id).to eq id }
-    end
-
-    describe 'works with unknown playlists' do
-      let(:id) { 'PL12--not-a-playlist' }
-      it {expect(url.id).to eq id }
-    end
-  end
-
-  context 'given a YouTube video URL' do
-    let(:text) { "https://www.youtube.com/watch?v=#{id}" }
-
-    describe 'works with existing videos' do
-      let(:id) { 'gknzFj_0vvY' }
-      it {expect(url.id).to eq id }
-    end
-
-    describe 'works with unknown videos' do
-      let(:id) { 'abc123abc12' }
-      it {expect(url.id).to eq id }
-    end
-  end
-
-  context 'given a YouTube channel URL in the ID form' do
-    let(:text) { "https://www.youtube.com/channel/#{id}" }
-
-    describe 'works with existing channels' do
-      let(:id) { 'UC4lU5YG9QDgs0X2jdnt7cdQ' }
-      it {expect(url.id).to eq id }
-    end
-
-    describe 'works with unknown channels' do
-      let(:id) { 'UC-not-an-actual-channel' }
-      it {expect(url.id).to eq id }
-    end
-  end
-
-  context 'given an existing YouTube channel' do
-    let(:text) { 'youtube.com/channel/UCxO1tY8h1AhOz0T4ENwmpow' }
-    it {expect(url.kind).to eq :channel }
-  end
-
-  context 'given an existing YouTube video' do
-    let(:text) { 'youtube.com/watch?v=gknzFj_0vvY' }
-    it {expect(url.kind).to eq :video }
-  end
-
-  context 'given an existing YouTube playlist' do
-    let(:text) { 'youtube.com/playlist?list=LLxO1tY8h1AhOz0T4ENwmpow' }
-    it {expect(url.kind).to eq :playlist }
-  end
-
-  context 'given an unknown YouTube channel URL' do
-    let(:text) { 'youtube.com/channel/UC-too-short-to-be-an-id' }
-    it {expect(url.kind).to eq :channel }
-  end
-
-  context 'given an unknown YouTube video URL' do
-    let(:text) { 'youtu.be/not-an-id' }
-    it {expect(url.kind).to eq :unknown }
-  end
-
-  context 'given an unknown text' do
-    let(:text) { 'not-really-anything---' }
-    it {expect(url.kind).to eq :unknown }
-  end
-end