yt 0.9.2 → 0.9.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 5fe9ccf19b87eba1932a8381ec62e4c7e18bb997
-  data.tar.gz: bcc2aefb8c393bac857f94157267d66ab96301ae
+  metadata.gz: ab23bb89da9f80619391db9f6d8a490609631147
+  data.tar.gz: 4e2cc2d536a284e70f43f6c94fd80cd44dc63b4f
 SHA512:
-  metadata.gz: f3f0ddfe5fcf4c20b72a77478efba519a33db0bf9565b52a47daaa6f768fe68015d3436d4535fdd759443462903c5ec788750ef5595317672c7492d903c25db2
-  data.tar.gz: 0dcc83e353408a679b723516e3801eafd2b1710ee25584040a9d9c359fa8dadac24b44949779b6b4f042a09e3ebd633b737b3c64815bf826ed571bdb0f0169e2
+  metadata.gz: 24ecc8fe727d82e972d568310e9d463f1a51f1028184ca99bf122b63e647857ec62174a214882eb1b81e681eec36127cf9b3dadb1c23544235d11ec9acfc1312
+  data.tar.gz: c118aa416a094bee9ecd3027756045720d3409c1a9c0d99479f9fe1b0e6d2544b855d4b63c299bdbbe47cfa7d55fbfa786ff1c07497b92d72a98656fcfab0cc5

data/.travis.yml

@@ -1,6 +1,6 @@
 language: ruby
 notifications:
-  email: false
+  email: true
 matrix:
   include:
   - rvm: 1.9.3

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    yt (0.9.2)
+    yt (0.9.3)
       activesupport
 
 GEM

data/HISTORY.md

@@ -4,6 +4,8 @@ v0.9 - 2014/07/28
 * [breaking change] Rename rating.update to rating.set
 * Add content_owner.references to retrieve ContentID references
 * Add content_owner.policies to list ContentID policies
+* Let 'update' methods understand both under_score and camelCased parameters
+* Add claim.third_party?
 
 
 v0.8 - 2014/07/18

data/README.md

@@ -41,7 +41,7 @@ To install on your system, run
 
 To use inside a bundled Ruby project, add this line to the Gemfile:
 
-    gem 'yt', '~> 0.9.2'
+    gem 'yt', '~> 0.9.3'
 
 Since the gem follows [Semantic Versioning](http://semver.org),
 indicating the full version in your Gemfile (~> *major*.*minor*.*patch*)

data/lib/yt/actions/list.rb

@@ -19,6 +19,7 @@ module Yt
           while next_item = find_next
             items << next_item
           end
+          @where_params = {}
         end
       end
 
@@ -41,7 +42,7 @@ module Yt
 
       def next_page
         params = list_params.dup
-        params[:params][:pageToken] = @page_token if @page_token
+        params[:params][:page_token] = @page_token if @page_token
         next_page = fetch_page params
         @page_token = next_page[:token]
         next_page[:items]
@@ -55,6 +56,7 @@ module Yt
       end
 
       def request(params = {})
+        camelize_keys! params[:params] if params.fetch(:camelize_params, true)
         @last_request = Yt::Request.new params
       end
 
@@ -80,6 +82,12 @@ module Yt
       def list_resources
         self.class
       end
+
+      def camelize_keys!(hash = {})
+        hash.dup.each_key do |key|
+          hash[key.to_s.camelize(:lower).to_sym] = hash.delete key
+        end
+      end
     end
   end
 end

data/lib/yt/collections/annotations.rb

@@ -27,6 +27,7 @@ module Yt
           params[:host] = 'www.youtube.com'
           params[:path] = '/annotations_invideo'
           params[:params] = {video_id: @parent.id}
+          params[:camelize_params] = false
           params[:expected_response] = Net::HTTPOK
         end
       end

data/lib/yt/collections/authentications.rb

@@ -40,4 +40,4 @@ module Yt
       end
     end
   end
-end
+end

data/lib/yt/collections/base.rb

@@ -20,9 +20,16 @@ module Yt
       end
 
       def where(conditions = {})
-        @items = []
-        @extra_params = conditions
-        self
+        self.tap do
+          @items = []
+          @where_params = conditions
+        end
+      end
+
+    private
+
+      def apply_where_params!(params = {})
+        params.merge!(@where_params ||= {})
       end
     end
   end

data/lib/yt/collections/channels.rb

@@ -12,7 +12,11 @@ module Yt
       # @return [Hash] the parameters to submit to YouTube to list channels.
       # @see https://developers.google.com/youtube/v3/docs/channels/list
       def list_params
-        super.tap{|params| params[:params].merge! mine: true}
+        super.tap{|params| params[:params] = channels_params}
+      end
+
+      def channels_params
+        resources_params.merge mine: true
       end
     end
   end

data/lib/yt/collections/claims.rb

@@ -19,23 +19,22 @@ module Yt
       # @see https://developers.google.com/youtube/partner/docs/v1/claims/list
       # @see https://developers.google.com/youtube/partner/docs/v1/claimSearch/list
       def list_params
-
         super.tap do |params|
-          params[:params] = claims_params
           params[:path] = claims_path
+          params[:params] = claims_params
         end
       end
 
       def claims_params
-        {onBehalfOfContentOwner: @parent.owner_name}.tap do |params|
-          (@extra_params ||= {}).each do |key, value|
-            params[key.to_s.camelize :lower] = value
-          end
-        end
+        apply_where_params! on_behalf_of_content_owner: @parent.owner_name
       end
 
+      # @private
+      # @todo: This is the only place outside of base.rb where @where_params
+      #   is accessed; it should be replaced with a filter on params instead.
       def claims_path
-        if @extra_params.empty? || @extra_params.key?(:id)
+        @where_params ||= {}
+        if @where_params.empty? || @where_params.key?(:id)
           '/youtube/partner/v1/claims'
         else
           '/youtube/partner/v1/claimSearch'

data/lib/yt/collections/content_details.rb

@@ -18,10 +18,14 @@ module Yt
       # @see https://developers.google.com/youtube/v3/docs/videos#resource
       def list_params
         super.tap do |params|
-          params[:params] = {maxResults: 50, part: 'contentDetails', id: @parent.id}
+          params[:params] = content_details_params
           params[:path] = '/youtube/v3/videos'
         end
       end
+
+      def content_details_params
+        {max_results: 50, part: 'contentDetails', id: @parent.id}
+      end
     end
   end
 end

data/lib/yt/collections/content_owners.rb

@@ -25,7 +25,7 @@ module Yt
       end
 
       def content_owners_params
-        {fetchMine: true}
+        {fetch_mine: true}
       end
     end
   end

data/lib/yt/collections/ids.rb

@@ -13,10 +13,14 @@ module Yt
 
       def list_params
         super.tap do |params|
-          params[:params] = {forUsername: @parent.username, part: 'id'}
+          params[:params] = ids_params
           params[:path] = "/youtube/v3/#{@parent.kind.pluralize}"
         end
       end
+
+      def ids_params
+        {for_username: @parent.username, part: 'id'}
+      end
     end
   end
 end

data/lib/yt/collections/partnered_channels.rb

@@ -6,13 +6,14 @@ module Yt
 
     private
 
-      # @return [Hash] the parameters to submit to YouTube to list partnered channels.
-      # @see https://developers.google.com/youtube/v3/docs/channels/list
-      def list_params
+      # @private
+      # @note Partnered Channels overwrites +channel_params+ since the query
+      #   is slightly different.
+      def channels_params
         super.tap do |params|
-          params[:params].delete :mine
-          params[:params][:managedByMe] = true
-          params[:params][:onBehalfOfContentOwner] = @parent.owner_name
+          params.delete :mine
+          params[:managed_by_me] = true
+          params[:on_behalf_of_content_owner] = @parent.owner_name
         end
       end
 

data/lib/yt/collections/playlist_items.rb

@@ -17,10 +17,14 @@ module Yt
 
     private
 
-      # @return [Hash] the parameters to submit to YouTube to list items.
-      # @see https://developers.google.com/youtube/v3/docs/playlistItems/list
+      # @return [Hash] the parameters to submit to YouTube to list channels.
+      # @see https://developers.google.com/youtube/v3/docs/channels/list
       def list_params
-        super.tap{|params| params[:params].merge! playlistId: @parent.id}
+        super.tap{|params| params[:params] = playlist_items_params}
+      end
+
+      def playlist_items_params
+        resources_params.merge playlist_id: @parent.id
       end
     end
   end

data/lib/yt/collections/playlists.rb

@@ -20,10 +20,14 @@ module Yt
 
     private
 
-      # @return [Hash] the parameters to submit to YouTube to list playlists.
-      # @see https://developers.google.com/youtube/v3/docs/playlist/list
+      # @return [Hash] the parameters to submit to YouTube to list channels.
+      # @see https://developers.google.com/youtube/v3/docs/channels/list
       def list_params
-        super.tap{|params| params[:params].merge! channelId: @parent.id}
+        super.tap{|params| params[:params] = playlists_params}
+      end
+
+      def playlists_params
+        resources_params.merge channel_id: @parent.id
       end
     end
   end

data/lib/yt/collections/policies.rb

@@ -18,7 +18,6 @@ module Yt
       #   saved by the content owner.
       # @see https://developers.google.com/youtube/partner/docs/v1/policies/list
       def list_params
-
         super.tap do |params|
           params[:params] = policies_params
           params[:path] = '/youtube/partner/v1/policies'
@@ -26,11 +25,7 @@ module Yt
       end
 
       def policies_params
-        {onBehalfOfContentOwner: @parent.owner_name}.tap do |params|
-          (@extra_params ||= {}).each do |key, value|
-            params[key.to_s.camelize :lower] = value
-          end
-        end
+        apply_where_params! on_behalf_of_content_owner: @parent.owner_name
       end
     end
   end

data/lib/yt/collections/references.rb

@@ -25,11 +25,7 @@ module Yt
       end
 
       def references_params
-        {onBehalfOfContentOwner: @parent.owner_name}.tap do |params|
-          (@extra_params ||= {}).each do |key, value|
-            params[key.to_s.camelize :lower] = value
-          end
-        end
+        apply_where_params! on_behalf_of_content_owner: @parent.owner_name
       end
     end
   end

data/lib/yt/collections/reports.rb

@@ -20,12 +20,13 @@ module Yt
       def list_params
         super.tap do |params|
           params[:path] = '/youtube/analytics/v1/reports'
-          params[:params] = @parent.reports_params.merge reports_params
+          params[:params] = reports_params
+          params[:camelize_params] = false
         end
       end
 
       def reports_params
-        {}.tap do |params|
+        @parent.reports_params.tap do |params|
           params['start-date'] = @days_range.begin
           params['end-date'] = @days_range.end
           params['metrics'] = @metric

data/lib/yt/collections/resources.rb

@@ -18,13 +18,8 @@ module Yt
         resource_class.new id: data['id'], snippet: data['snippet'], status: data['status'], auth: @auth
       end
 
-      # @return [Hash] the parameters to submit to YouTube to list items.
-      # @see https://developers.google.com/youtube/v3/docs/playlistItems/list
-      # @see https://developers.google.com/youtube/v3/docs/playlists/list
-      def list_params
-        super.tap do |params|
-          params[:params] = {maxResults: 50, part: 'snippet,status'}
-        end
+      def resources_params
+        {max_results: 50, part: 'snippet,status'}
       end
 
       def resource_class

data/lib/yt/collections/statistics_sets.rb

@@ -7,7 +7,7 @@ module Yt
 
     private
 
-      # @return [Yt::Models::StatisticsSet] a new statistics set initialized 
+      # @return [Yt::Models::StatisticsSet] a new statistics set initialized
       #   with one of the items returned by asking YouTube for a list of them.
       def new_item(data)
         Yt::StatisticsSet.new data: data['statistics']
@@ -18,10 +18,14 @@ module Yt
       # @see https://developers.google.com/youtube/v3/docs/videos#resource
       def list_params
         super.tap do |params|
-          params[:params] = {maxResults: 50, part: 'statistics', id: @parent.id}
           params[:path] = "/youtube/v3/#{@parent.kind.pluralize}"
+          params[:params] = statistics_sets_params
         end
       end
+
+      def statistics_sets_params
+        {max_results: 50, part: 'statistics', id: @parent.id}
+      end
     end
   end
 end

data/lib/yt/collections/statuses.rb

@@ -19,13 +19,11 @@ module Yt
       #   of a resource, for instance a channel.
       # @see https://developers.google.com/youtube/v3/docs/channels/list
       def list_params
-        super.tap do |params|
-          params[:params] = {id: @parent.id, part: 'status'}
-        end
+        super.tap{|params| params[:params] = {id: @parent.id, part: 'status'}}
       end
 
       # @private
-      # @note Statuses overrides +list_resources+ since the endpoint is not 
+      # @note Statuses overrides +list_resources+ since the endpoint is not
       #   '/statuses' but the endpoint related to the snippet’s resource.
       def list_resources
         @parent.class.to_s.pluralize

data/lib/yt/collections/subscriptions.rb

@@ -50,7 +50,16 @@ module Yt
       # @see https://developers.google.com/youtube/v3/docs/subscriptions/list
       def list_params
         super.tap do |params|
-          params[:params] = {maxResults: 50, forChannelId: @parent.id, mine: true, part: 'snippet'}
+          params[:params] = subscriptions_params
+        end
+      end
+
+      def subscriptions_params
+        {}.tap do |params|
+          params[:max_results] = 50
+          params[:for_channel_id] = @parent.id
+          params[:mine] = true
+          params[:part] = 'snippet'
         end
       end
 

data/lib/yt/collections/videos.rb

@@ -22,7 +22,7 @@ module Yt
       # @see https://developers.google.com/youtube/v3/docs/search/list
       def list_params
         super.tap do |params|
-          params[:params] = @parent.videos_params.merge videos_params
+          params[:params] = videos_params
           params[:path] = '/youtube/v3/search'
         end
       end
@@ -36,17 +36,21 @@ module Yt
       # that limit, the query is restarted with a publishedBefore filter in
       # case there are more videos to be listed for a channel
       def add_offset_to(items)
-        if items.count == videos_params[:maxResults]
+        if items.count == videos_params[:max_results]
           last_published = items.last['snippet']['publishedAt']
           @page_token, @published_before = '', last_published
         end
       end
 
       def videos_params
-        params = {type: :video, maxResults: 50, part: 'snippet', order: 'date'}
-        params[:publishedBefore] = @published_before if @published_before
-        @extra_params ||= {}
-        params.merge @extra_params
+        @parent.videos_params.tap do |params|
+          params[:type] = :video
+          params[:max_results] = 50
+          params[:part] = 'snippet'
+          params[:order] = 'date'
+          params[:published_before] = @published_before if @published_before
+          apply_where_params! params
+        end
       end
     end
   end

data/lib/yt/collections/viewer_percentages.rb

@@ -24,12 +24,13 @@ module Yt
       def list_params
         super.tap do |params|
           params[:path] = '/youtube/analytics/v1/reports'
-          params[:params] = @parent.reports_params.merge reports_params
+          params[:params] = reports_params
+          params[:capitalize_params] = false
         end
       end
 
       def reports_params
-        {}.tap do |params|
+        @parent.reports_params.tap do |params|
           params['start-date'] = 3.months.ago.to_date
           params['end-date'] = Date.today.to_date
           params['metrics'] = :viewerPercentage

data/lib/yt/models/claim.rb

@@ -27,6 +27,11 @@ module Yt
         @video_id ||= @data["videoId"]
       end
 
+# Status
+
+      STATUSES = %q(active appealed disputed inactive pending potential
+        takedown unknown)
+
       # @return [String] the claim’s status. Valid values are: active,
       #   appealed, disputed, inactive, pending, potential, takedown, unknown.
       # @note When updating a claim, you can update its status from active to
@@ -76,6 +81,10 @@ module Yt
         status == 'unknown'
       end
 
+# Content Type
+
+      CONTENT_TYPES = %q(audio video audiovisual)
+
       # @return [String] whether the claim covers the audio, video, or
       #   audiovisual portion of the claimed content. Valid values are: audio,
       #   audiovisual, video.
@@ -103,6 +112,11 @@ module Yt
         @created_at ||= Time.parse @data["timeCreated"]
       end
 
+      # @return [Boolean] whether a third party created the claim.
+      def third_party?
+        @third_party_claim ||= @data['thirdPartyClaim'] == true
+      end
+
       # Return whether the video should be blocked where not explicitly owned.
       # @return [Boolean] whether the claimed video should be blocked anywhere
       #   it is not explicitly owned. For example, if you upload a video for an

data/lib/yt/models/playlist.rb

@@ -24,22 +24,7 @@ module Yt
       end
 
       def update(attributes = {})
-        underscore_keys! attributes
-
-        body = {id: @id}.tap do |body|
-          update_parts.each do |part, options|
-            if (options[:keys] & attributes.keys).any? || options[:required]
-              body[part] = {}.tap do |hash|
-                options[:keys].map do |key|
-                  hash[camelize key] = attributes[key] || send(key)
-                end
-              end
-            end
-          end
-        end
-
-        part = body.except(:id).keys.join(',')
-        do_update(params: {part: part}, body: body) do |data|
+        super attributes do |data|
           @id = data['id']
           @snippet = Snippet.new data: data['snippet'] if data['snippet']
           @status = Status.new data: data['status'] if data['status']
@@ -88,18 +73,6 @@ module Yt
         {snippet: snippet, status: status}
       end
 
-      # @note If we dropped support for ActiveSupport 3, then we could simply
-      # invoke transform_keys!{|key| key.to_s.underscore.to_sym}
-      def underscore_keys!(hash)
-        hash.dup.each_key do |key|
-          hash[key.to_s.underscore.to_sym] = hash.delete key
-        end
-      end
-
-      def camelize(value)
-        value.to_s.camelize(:lower).to_sym
-      end
-
       def video_params(video_id)
         {id: video_id, kind: :video}
       end

data/lib/yt/models/resource.rb

@@ -30,6 +30,13 @@ module Yt
         @url.username if @url
       end
 
+      def update(attributes = {}, &block)
+        underscore_keys! attributes
+        body = build_update_body attributes
+        params = {part: body.keys.join(',')}
+        do_update(params: params, body: body.merge(id: @id), &block)
+      end
+
     private
 
       # @return [Hash] the parameters to submit to YouTube to update a playlist.
@@ -47,6 +54,34 @@ module Yt
       def delete_params
         super.tap{|params| params[:params] = {id: @id}}
       end
+
+      def build_update_body(attributes = {})
+        body = {}
+        update_parts.each do |name, part|
+          body[name] = {}.tap do |hash|
+            part[:keys].map{|k| hash[camelize k] = attributes.fetch k, send(k)}
+          end if should_include_part_in_update?(part, attributes)
+        end
+        body
+      end
+
+      def should_include_part_in_update?(part, attributes = {})
+        part[:required] || (part[:keys] & attributes.keys).any?
+      end
+
+      # If we dropped support for ActiveSupport 3, then we could simply
+      # invoke transform_keys!{|key| key.to_s.underscore.to_sym}
+      def underscore_keys!(hash)
+        hash.dup.each_key{|key| hash[underscore key] = hash.delete key}
+      end
+
+      def camelize(value)
+        value.to_s.camelize(:lower).to_sym
+      end
+
+      def underscore(value)
+        value.to_s.underscore.to_sym
+      end
     end
   end
 end

data/lib/yt/models/video.rb

@@ -81,21 +81,15 @@ module Yt
       #   caution, as the whole status object needs to be updated, not just
       #   privacyStatus, but also embeddable, license, publicStatsViewable,
       #   and publishAt
-      def update(options = {})
-        options[:title] ||= title
-        options[:description] ||= description
-        options[:tags] ||= tags
-        options[:categoryId] ||= category_id
-        snippet = options.slice :title, :description, :tags, :categoryId
-        body = {id: @id, snippet: snippet}
-
-        do_update(params: {part: 'snippet'}, body: body) do |data|
+      def update(attributes = {})
+        super attributes do |data|
           @id = data['id']
           @snippet = Snippet.new data: data['snippet'] if data['snippet']
           true
         end
       end
 
+
       # Returns whether the authenticated account likes the video.
       #
       # This method requires {Resource#auth auth} to return an
@@ -157,6 +151,15 @@ module Yt
           params['filters'] = "video==#{id}"
         end
       end
+
+    private
+
+      # @see https://developers.google.com/youtube/v3/docs/videos/update
+      # @todo: Add status, recording details keys
+      def update_parts
+        snippet_keys = [:title, :description, :tags, :category_id]
+        {snippet: {keys: snippet_keys}}
+      end
     end
   end
 end

data/lib/yt/version.rb

@@ -1,3 +1,3 @@
 module Yt
-  VERSION = '0.9.2'
+  VERSION = '0.9.3'
 end

data/spec/collections/claims_spec.rb

@@ -0,0 +1,30 @@
+require 'spec_helper'
+require 'yt/collections/claims'
+require 'yt/models/content_owner'
+
+describe Yt::Collections::Claims do
+  subject(:collection) { Yt::Collections::Claims.new parent: content_owner }
+  let(:content_owner) { Yt::ContentOwner.new owner_name: 'any-name' }
+  let(:page) { {items: [], token: 'any-token'} }
+  let(:query) { {q: 'search string'} }
+
+  describe '#count' do
+    context 'called once with .where(query) and once without' do
+      after do
+        collection.where(query).count
+        collection.count
+      end
+
+      it 'only applies the query on the first call' do
+        expect(collection).to receive(:fetch_page) do |options|
+          expect(options[:params]).to include query
+          page
+        end
+        expect(collection).to receive(:fetch_page) do |options|
+          expect(options[:params]).not_to include query
+          page
+        end
+      end
+    end
+  end
+end

data/spec/collections/playlist_items_spec.rb

@@ -1,5 +1,6 @@
 require 'spec_helper'
 require 'yt/models/playlist'
+require 'yt/models/playlist_item'
 require 'yt/collections/playlist_items'
 
 describe Yt::Collections::PlaylistItems do

data/spec/collections/policies_spec.rb

@@ -0,0 +1,30 @@
+require 'spec_helper'
+require 'yt/collections/policies'
+require 'yt/models/content_owner'
+
+describe Yt::Collections::Policies do
+  subject(:collection) { Yt::Collections::Policies.new parent: content_owner }
+  let(:content_owner) { Yt::ContentOwner.new owner_name: 'any-name' }
+  let(:page) { {items: [], token: 'any-token'} }
+  let(:query) { {id: 'any-id'} }
+
+  describe '#count' do
+    context 'called once with .where(query) and once without' do
+      after do
+        collection.where(query).count
+        collection.count
+      end
+
+      it 'only applies the query on the first call' do
+        expect(collection).to receive(:fetch_page) do |options|
+          expect(options[:params]).to include query
+          page
+        end
+        expect(collection).to receive(:fetch_page) do |options|
+          expect(options[:params]).not_to include query
+          page
+        end
+      end
+    end
+  end
+end

data/spec/collections/references_spec.rb

@@ -0,0 +1,30 @@
+require 'spec_helper'
+require 'yt/collections/references'
+require 'yt/models/content_owner'
+
+describe Yt::Collections::References do
+  subject(:collection) { Yt::Collections::References.new parent: content_owner }
+  let(:content_owner) { Yt::ContentOwner.new id: 'any-id' }
+  let(:page) { {items: [], token: 'any-token'} }
+  let(:query) { {id: 'reference-id'} }
+
+  describe '#count' do
+    context 'called once with .where(query) and once without' do
+      after do
+        collection.where(query).count
+        collection.count
+      end
+
+      it 'only applies the query on the first call' do
+        expect(collection).to receive(:fetch_page) do |options|
+          expect(options[:params]).to include query
+          page
+        end
+        expect(collection).to receive(:fetch_page) do |options|
+          expect(options[:params]).not_to include query
+          page
+        end
+      end
+    end
+  end
+end

data/spec/collections/videos_spec.rb

@@ -1,6 +1,30 @@
 require 'spec_helper'
 require 'yt/collections/videos'
+require 'yt/models/channel'
 
 describe Yt::Collections::Videos do
-  # Nothing to test
+  subject(:collection) { Yt::Collections::Videos.new parent: channel }
+  let(:channel) { Yt::Channel.new id: 'any-id' }
+  let(:page) { {items: [], token: 'any-token'} }
+  let(:query) { {q: 'search string'} }
+
+  describe '#count' do
+    context 'called once with .where(query) and once without' do
+      after do
+        collection.where(query).count
+        collection.count
+      end
+
+      it 'only applies the query on the first call' do
+        expect(collection).to receive(:fetch_page) do |options|
+          expect(options[:params]).to include query
+          page
+        end
+        expect(collection).to receive(:fetch_page) do |options|
+          expect(options[:params]).not_to include query
+          page
+        end
+      end
+    end
+  end
 end

data/spec/models/claim_spec.rb

@@ -189,4 +189,16 @@ describe Yt::Claim do
       it { expect(claim.block_outside_ownership?).to be false }
     end
   end
+
+  describe '#third_party?' do
+    context 'given fetching a claim returns thirdPartyClaim true' do
+      let(:data) { {"thirdPartyClaim"=>true} }
+      it { expect(claim.third_party?).to be true }
+    end
+
+    context 'given fetching a claim returns thirdPartyClaim true' do
+      let(:data) { {"thirdPartyClaim"=>false} }
+      it { expect(claim.third_party?).to be false }
+    end
+  end
 end

data/spec/requests/as_account/account_spec.rb

@@ -12,7 +12,7 @@ describe Yt::Account, :device_app do
 
   describe '.videos' do
     it { expect($account.videos).to be_a Yt::Collections::Videos }
-    it { expect($account.videos.first).to be_a Yt::Video }
+    it { expect($account.videos.where(order: 'viewCount').first).to be_a Yt::Video }
 
     describe '.where(q: query_string)' do
       let(:count) { $account.videos.where(q: query).count }

data/spec/requests/as_account/video_spec.rb

@@ -100,24 +100,70 @@ describe Yt::Video, :device_app do
   end
 
   context 'given one of my own videos that I want to update' do
-    let(:id) { $account.videos.first.id }
+    let(:id) { $account.videos.where(order: 'viewCount').first.id }
+    let!(:old_title) { video.title }
+    let!(:old_privacy_status) { video.privacy_status }
+    let(:update) { video.update attrs }
 
-    describe 'updates the attributes that I specify explicitly' do
+    context 'given I update the title' do
       # NOTE: The use of UTF-8 characters is to test that we can pass up to
       # 50 characters, independently of their representation
       let(:attrs) { {title: "Yt Example Update Video #{rand} - ®•♡❥❦❧☙"} }
-      it { expect(video.update attrs).to eq true }
-      it { expect{video.update attrs}.to change{video.title} }
+
+      specify 'only updates the title' do
+        expect(update).to be true
+        expect(video.title).not_to eq old_title
+        expect(video.privacy_status).to eq old_privacy_status
+      end
+    end
+
+    context 'given I update the description' do
+      let!(:old_description) { video.description }
+      let(:attrs) { {description: "Yt Example Description  #{rand} - ®•♡❥❦❧☙"} }
+
+      specify 'only updates the description' do
+        expect(update).to be true
+        expect(video.description).not_to eq old_description
+        expect(video.title).to eq old_title
+        expect(video.privacy_status).to eq old_privacy_status
+      end
+    end
+
+    context 'given I update the tags' do
+      let!(:old_tags) { video.tags }
+      let(:attrs) { {tags: ["Yt Test Tag #{rand}"]} }
+
+      specify 'only updates the tag' do
+        expect(update).to be true
+        expect(video.tags).not_to eq old_tags
+        expect(video.title).to eq old_title
+        expect(video.privacy_status).to eq old_privacy_status
+      end
     end
 
-    describe 'does not update the other attributes' do
-      let(:attrs) { {} }
-      it { expect(video.update attrs).to eq true }
-      it { expect{video.update attrs}.not_to change{video.title} }
-      it { expect{video.update attrs}.not_to change{video.description} }
-      it { expect{video.update attrs}.not_to change{video.tags} }
-      it { expect{video.update attrs}.not_to change{video.category_id} }
-      it { expect{video.update attrs}.not_to change{video.privacy_status} }
+    context 'given I update the category ID' do
+      let!(:old_category_id) { video.category_id }
+      let!(:new_category_id) { old_category_id == '22' ? '21' : '22' }
+
+      context 'passing the parameter in underscore syntax' do
+        let(:attrs) { {category_id: new_category_id} }
+
+        specify 'only updates the category ID' do
+          expect(update).to be true
+          expect(video.category_id).not_to eq old_category_id
+          expect(video.title).to eq old_title
+          expect(video.privacy_status).to eq old_privacy_status
+        end
+      end
+
+      context 'passing the parameter in camel-case syntax' do
+        let(:attrs) { {categoryId: new_category_id} }
+
+        specify 'only updates the category ID' do
+          expect(update).to be true
+          expect(video.category_id).not_to eq old_category_id
+        end
+      end
     end
 
     it 'returns valid reports for video-related metrics' do

data/spec/requests/as_content_owner/content_owner_spec.rb

@@ -9,7 +9,20 @@ describe Yt::ContentOwner, :partner do
   end
 
   describe '.claims' do
-    it { expect($content_owner.claims.first).to be_a Yt::Claim }
+    describe 'given the content owner has policies' do
+      let(:claim) { $content_owner.claims.first }
+
+      it 'returns valid metadata' do
+        expect(claim.id).to be_a String
+        expect(claim.asset_id).to be_a String
+        expect(claim.video_id).to be_a String
+        expect(claim.status).to be_in Yt::Claim::STATUSES
+        expect(claim.content_type).to be_in Yt::Claim::CONTENT_TYPES
+        expect(claim.created_at).to be_a Time
+        expect(claim).not_to be_third_party
+        expect(claim.block_outside_ownership?).to be_in [true, false]
+      end
+    end
 
     describe '.where(id: claim_id)' do
       let(:count) { $content_owner.claims.where(id: claim_id).count }

metadata

@@ -1,97 +1,97 @@
 --- !ruby/object:Gem::Specification
 name: yt
 version: !ruby/object:Gem::Version
-  version: 0.9.2
+  version: 0.9.3
 platform: ruby
 authors:
 - Claudio Baccigalupo
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-07-29 00:00:00.000000000 Z
+date: 2014-07-31 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: yard
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: coveralls
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
 description: Youtube V3 API client.
@@ -102,10 +102,10 @@ executables:
 extensions: []
 extra_rdoc_files: []
 files:
-- ".gitignore"
-- ".rspec"
-- ".travis.yml"
-- ".yardopts"
+- .gitignore
+- .rspec
+- .travis.yml
+- .yardopts
 - Gemfile
 - Gemfile.lock
 - HISTORY.md
@@ -189,15 +189,12 @@ files:
 - lib/yt/models/user_info.rb
 - lib/yt/models/video.rb
 - lib/yt/version.rb
-- spec/collections/annotations_spec.rb
-- spec/collections/channels_spec.rb
-- spec/collections/content_details_spec.rb
+- spec/collections/claims_spec.rb
 - spec/collections/playlist_items_spec.rb
 - spec/collections/playlists_spec.rb
-- spec/collections/ratings_spec.rb
-- spec/collections/snippets_spec.rb
+- spec/collections/policies_spec.rb
+- spec/collections/references_spec.rb
 - spec/collections/subscriptions_spec.rb
-- spec/collections/user_infos_spec.rb
 - spec/collections/videos_spec.rb
 - spec/errors/missing_auth_spec.rb
 - spec/errors/no_items_spec.rb
@@ -257,31 +254,28 @@ require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - '>='
     - !ruby/object:Gem::Version
       version: 1.9.3
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - '>='
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.2.2
+rubygems_version: 2.2.1
 signing_key: 
 specification_version: 4
 summary: Yt makes it easy to interact with Youtube V3 API by providing a modular,
   intuitive and tested Ruby-style API.
 test_files:
-- spec/collections/annotations_spec.rb
-- spec/collections/channels_spec.rb
-- spec/collections/content_details_spec.rb
+- spec/collections/claims_spec.rb
 - spec/collections/playlist_items_spec.rb
 - spec/collections/playlists_spec.rb
-- spec/collections/ratings_spec.rb
-- spec/collections/snippets_spec.rb
+- spec/collections/policies_spec.rb
+- spec/collections/references_spec.rb
 - spec/collections/subscriptions_spec.rb
-- spec/collections/user_infos_spec.rb
 - spec/collections/videos_spec.rb
 - spec/errors/missing_auth_spec.rb
 - spec/errors/no_items_spec.rb

data/spec/collections/annotations_spec.rb

@@ -1,6 +0,0 @@
-require 'spec_helper'
-require 'yt/collections/annotations'
-
-describe Yt::Collections::Annotations do
-  # Nothing to test
-end

data/spec/collections/channels_spec.rb

@@ -1,6 +0,0 @@
-require 'spec_helper'
-require 'yt/collections/channels'
-
-describe Yt::Collections::Channels do
-  # Nothing to test
-end

data/spec/collections/content_details_spec.rb

@@ -1,6 +0,0 @@
-require 'spec_helper'
-require 'yt/collections/content_details'
-
-describe Yt::Collections::ContentDetails do
-  # Nothing to test
-end

data/spec/collections/ratings_spec.rb

@@ -1,6 +0,0 @@
-require 'spec_helper'
-require 'yt/collections/ratings'
-
-describe Yt::Collections::Ratings do
-  # Nothing to test
-end

data/spec/collections/snippets_spec.rb

@@ -1,6 +0,0 @@
-require 'spec_helper'
-require 'yt/collections/snippets'
-
-describe Yt::Collections::Snippets do
-  # Nothing to test
-end

data/spec/collections/user_infos_spec.rb

@@ -1,6 +0,0 @@
-require 'spec_helper'
-require 'yt/collections/user_infos'
-
-describe Yt::Collections::UserInfos do
-  # Nothing to test
-end