youku_client 0.0.1 → 0.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: f76877012802d8935a608f187c306eec028155eb
-  data.tar.gz: 42da00ca401e82d36bead60eabaef77340e5c91a
+  metadata.gz: 88241102eb4635e4dd5c689499ffe9059708ee56
+  data.tar.gz: c934e26daab56f7e827d18b15f72caf13915b429
 SHA512:
-  metadata.gz: 76b6cb74aa40cb19966a356e2c313cb377db4c592c06800475731064662c2c73dd3d163991d673de8344e920bd7eca3b31c3cf54180f24a5cc2d88152776d830
-  data.tar.gz: dc2a550d00565bcb80e9841db7241d6531315d97ed02ace0a5ea0410f94b644241d94970b22942f27e4e48d1a158e78e8c1c4bdb01a7e8ffb4edc6efd2892161
+  metadata.gz: 4c0ba3dc6da07a1a17b3d26ddcfa6b01b571433ac4017584ce5dca2129da268d808b873d78844a75c84de644677ea55901a1c95e26b0f58ad8e02f7219bbcde8
+  data.tar.gz: ad4a0f89d148781291e40f387f9cef780ff7e26fd7928a963dbc5aa2be6144e5ff005ea2a48af6e0d3402fb97e94b4f24a281f8191b585b78539b7b695b6151a

data/.travis.yml

@@ -0,0 +1,13 @@
+script: 'rspec spec'
+notifications:
+  email:
+    - avokhmin@gmail.com
+    - be9@be9.ru
+    - dave@recurser.com
+branches:
+  only:
+    - master
+rvm:
+  - 2.0.0
+  - 2.1.0
+  - 2.2.2

data/CHANGELOG.md

@@ -0,0 +1,6 @@
+0.0.2
+------
+* [Fixed generated urls not matching webmocks for `nil` parameters.](https://github.com/Shuttlerock/youku_client/pull/1) ([@BenjaminSchaaf](https://github.com/BenjaminSchaaf))
+* [Added missing gems and requires](https://github.com/Shuttlerock/youku_client/pull/2) ([@BenjaminSchaaf](https://github.com/BenjaminSchaaf))
+* [Implemented users.show](https://github.com/Shuttlerock/youku_client/pull/3) ([@BenjaminSchaaf](https://github.com/BenjaminSchaaf))
+* [Implemented comments.by_video](https://github.com/Shuttlerock/youku_client/pull/4) ([@BenjaminSchaaf](https://github.com/BenjaminSchaaf))

data/README.md

@@ -1,5 +1,7 @@
 # Youku Ruby Gem
 
+[![Build Status](https://secure.travis-ci.org/Shuttlerock/youku_client.png)](http://travis-ci.org/Shuttlerock/youku_client) [![Code Climate](https://codeclimate.com/github/Shuttlerock/youku_client/badges/gpa.svg)](https://codeclimate.com/github/Shuttlerock/youku_client)
+
 This is the unofficial Ruby wrapper for the Youku v2 API. It supports not all
 endpoints currently available on the
 [Youku API](http://open.youku.com/docs?id=0).
@@ -33,7 +35,7 @@ please if you would like to contribute, let me know and throw me a pull request!
 
 ### Requirements
 
-* Ruby 1.9.x or 2.x.x
+* Ruby 2.x.x
 
 ---
 

data/lib/youku/client.rb

@@ -34,6 +34,12 @@ module Youku
       Youku::V2::Users.new(self)
     end
 
+    # Public: Get users API.
+    #
+    # Returns the instance of Youku::V2::Users.
+    def comments
+      Youku::V2::Comments.new(self)
+    end
   end
 end
 
@@ -41,3 +47,4 @@ require 'youku/v2/base'
 require 'youku/v2/searches'
 require 'youku/v2/users'
 require 'youku/v2/videos'
+require 'youku/v2/comments'

data/lib/youku/v2/comments.rb

@@ -0,0 +1,25 @@
+module Youku
+  module V2
+    class Comments < Base
+      BASE_URI = "#{BASE_URI}/comments"
+
+      # Public: Get comments by a video ID
+      #
+      # See: http://open.youku.com/docs?id=35
+      #
+      # video_id - The string video ID.
+      # page     - The Integer page number.
+      # count    - The Integer page size.
+      #
+      # Returns the instance of Youku::V2::Request.
+      def by_video(video_id: nil, page: 1, count: 20)
+        Youku::V2::Request.new "#{BASE_URI}/by_video.json", {
+          client_id: client.client_id,
+          video_id:  video_id,
+          page:      page,
+          count:     count,
+        }
+      end
+    end
+  end
+end

data/lib/youku/v2/request.rb

@@ -1,4 +1,6 @@
+require 'hashie'
 require 'typhoeus'
+require 'multi_json'
 
 module Youku
   module V2

data/lib/youku/v2/searches.rb

@@ -34,7 +34,6 @@ module Youku
           }
         end
       end
-
     end
   end
 end

data/lib/youku/v2/users.rb

@@ -15,10 +15,25 @@ module Youku
         Youku::V2::Request.new "#{BASE_URI}/show_batch.json", {
           client_id:  client.client_id,
           user_ids:   user_ids,
-          user_names: user_names
+          user_names: user_names,
         }
       end
 
+      # Public: Get user by ID
+      #
+      # See: http://open.youku.com/docs?id=24
+      #
+      # user_id   - The string of a youku user ID
+      # user_name - The string of a youku user name.
+      #
+      # Returns the instance of Youku::V2::Request.
+      def show(user_id: nil, user_name: nil)
+        Youku::V2::Request.new "#{BASE_URI}/show.json", {
+          client_id: client.client_id,
+          user_id:   user_id,
+          user_name: user_name,
+        }
+      end
     end
   end
 end

data/lib/youku/version.rb

@@ -1,5 +1,5 @@
 module Youku
 
-  VERSION = '0.0.1'
+  VERSION = '0.0.2'
 
 end

data/spec/youku/v2/comments_spec.rb

@@ -0,0 +1,24 @@
+require 'spec_helper'
+
+describe Youku::V2::Comments do
+
+  let(:client) { Youku::Client.new(client_id: 'client-id') }
+
+  describe '#by_video' do
+    let(:url)   { 'https://openapi.youku.com/v2/comments/by_video.json' }
+    let(:query) { {
+      client_id: client.client_id,
+      video_id:  123,
+      page:      1,
+      count:     20
+    } }
+
+    before do
+      stub_request(:get, url).with(query: query).to_return(status: 200)
+    end
+
+    subject { client.comments.by_video(video_id: 123) }
+
+    it_should_behave_like 'a base Youku API V2 requests'
+  end
+end

data/spec/youku/v2/users_spec.rb

@@ -8,8 +8,8 @@ describe Youku::V2::Users do
   describe '#show_batch' do
     let(:url)   { 'https://openapi.youku.com/v2/users/show_batch.json' }
     let(:query) { {
-      client_id: client.client_id,
-      user_ids:  nil,
+      client_id:  client.client_id,
+      user_ids:   '',
       user_names: 'jackie_chan',
     } }
 
@@ -22,4 +22,20 @@ describe Youku::V2::Users do
     it_should_behave_like 'a base Youku API V2 requests'
   end
 
+  describe '#show' do
+    let(:url)   { 'https://openapi.youku.com/v2/users/show.json' }
+    let(:query) { {
+      client_id: client.client_id,
+      user_id:   '',
+      user_name: 'jackie_chan',
+    } }
+
+    before do
+      stub_request(:get, url).with(query: query).to_return(status: 200)
+    end
+
+    subject { users.show(user_name: 'jackie_chan') }
+
+    it_should_behave_like 'a base Youku API V2 requests'
+  end
 end

data/spec/youku/v2/videos_spec.rb

@@ -10,7 +10,7 @@ describe Youku::V2::Videos do
     let(:query) { {
       client_id: client.client_id,
       video_ids: 123,
-      ext:       nil,
+      ext:       '',
     } }
 
     before do
@@ -27,7 +27,7 @@ describe Youku::V2::Videos do
     let(:query) { {
       client_id: client.client_id,
       video_id: 123,
-      ext:      nil,
+      ext:      '',
     } }
 
     before do
@@ -44,7 +44,7 @@ describe Youku::V2::Videos do
     let(:query) { {
       client_id: client.client_id,
       user_id:   123,
-      user_name: nil,
+      user_name: '',
       orderby:   'published',
       page:      1,
       count:     20

data/youku_client.gemspec

@@ -4,6 +4,7 @@ require File.join(File.dirname(__FILE__), 'lib/youku/version')
 Gem::Specification.new do |gem|
   gem.add_dependency 'typhoeus'
   gem.add_dependency 'hashie'
+  gem.add_dependency 'multi_json'
   gem.add_development_dependency 'rake'
   gem.add_development_dependency 'rspec'
   gem.add_development_dependency 'webmock'

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: youku_client
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.2
 platform: ruby
 authors:
 - Alexey Vokhmin
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-07-20 00:00:00.000000000 Z
+date: 2015-11-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: typhoeus
@@ -40,6 +40,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: multi_json
+  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: rake
   requirement: !ruby/object:Gem::Requirement
@@ -92,12 +106,15 @@ extensions: []
 extra_rdoc_files: []
 files:
 - ".gitignore"
+- ".travis.yml"
+- CHANGELOG.md
 - Gemfile
 - LICENSE
 - README.md
 - Rakefile
 - lib/youku/client.rb
 - lib/youku/v2/base.rb
+- lib/youku/v2/comments.rb
 - lib/youku/v2/request.rb
 - lib/youku/v2/searches.rb
 - lib/youku/v2/users.rb
@@ -107,6 +124,7 @@ files:
 - spec/spec_helper.rb
 - spec/support/shared_examples/youku_api_v2_examples.rb
 - spec/youku/client_spec.rb
+- spec/youku/v2/comments_spec.rb
 - spec/youku/v2/searches_spec.rb
 - spec/youku/v2/users_spec.rb
 - spec/youku/v2/videos_spec.rb
@@ -131,7 +149,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: 1.3.6
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.7
+rubygems_version: 2.4.8
 signing_key: 
 specification_version: 4
 summary: Youku API wrapper
@@ -139,6 +157,7 @@ test_files:
 - spec/spec_helper.rb
 - spec/support/shared_examples/youku_api_v2_examples.rb
 - spec/youku/client_spec.rb
+- spec/youku/v2/comments_spec.rb
 - spec/youku/v2/searches_spec.rb
 - spec/youku/v2/users_spec.rb
 - spec/youku/v2/videos_spec.rb