croudia 0.0.2 → 1.0.0

data/lib/croudia/user.rb

@@ -1,43 +1,12 @@
-require 'croudia/identity'
-
-module Croudia
-  class User < Croudia::Identity
-    attr_reader :username, :nickname, :avatar, :self_introduction,
-      :voices_count, :following_count, :followers_count, :album_count,
-      :position, :user_url, :spreadia, :favodia,
-      :following, :followed_by
-    alias handle username
-    alias screen_name username
-    alias user_name username
-    alias name nickname
-    alias profile_image avatar
-    alias description self_introduction
-    alias voice_count voices_count
-    alias statuses_count voices_count
-    alias status_count voices_count
-    alias updates_count voices_count
-    alias update_count voices_count
-    alias followings_count following_count
-    alias friends_count following_count
-    alias friend_count following_count
-    alias follower_count followers_count
-    alias location position
-    alias url user_url
-    alias favorited_count favodia
-    alias favourited_count favodia
-    alias spreaded_count spreadia
-    alias following? following
-    alias followed_by? followed_by
-
-    def initialize(*args)
-      super
-      @id = username
-      @spreadia = spreadia.to_i if spreadia
-      @favodia = favodia.to_i if favodia
-    end
-
-    def to_s
-      username
-    end
-  end
-end
+require 'croudia/creatable'
+require 'croudia/identity'
+
+module Croudia
+  class User < Croudia::Identity
+    include Croudia::Creatable
+
+    attr_reader :description, :favorites_count, :follow_request_sent,
+      :followers_count, :fruends_count, :location, :name, :profile_image_url_https,
+      :screen_name, :statuses_count, :url
+  end
+end

data/lib/croudia/version.rb

@@ -1,3 +1,3 @@
 module Croudia
-  VERSION = '0.0.2'
+  VERSION = '1.0.0'
 end

data/lib/croudia.rb

@@ -1,23 +1,32 @@
-require 'croudia/scraper'
+require 'croudia/client'
+require 'croudia/configurable'
+require 'croudia/version'
 
 module Croudia
   class << self
-    # Delegate to Croudia::Scraper.new
+    include Croudia::Configurable
+
+    # Delegate to a Croudia::Client
     #
-    # @return [Croudia::Scraper]
-    def scraper(*args, &block)
-      Croudia::Scraper.new(*args, &block)
+    # @return [Croudia::Client]
+    def client
+      if !@client || @client.hash != options.hash
+        @client = Croudia::Client.new
+      end
+      @client
     end
-    alias new scraper
-    alias client scraper
 
-    def method_missing(name, *args, &block)
-      return super unless scraper.respond_to?(name)
-      scraper.send(name, *args, &block)
+    def respond_to?(*args)
+      super || client.respond_to?(*args)
     end
 
-    def respond_to?(name)
-      scraper.respond_to?(name) || super
+  private
+
+    def method_missing(name, *args, &block)
+      return super unless client.respond_to?(name)
+      client.send(name, *args, &block)
     end
   end
 end
+
+Croudia.setup

data/spec/croudia/api/favorites_spec.rb

@@ -0,0 +1,35 @@
+require 'helper'
+
+describe Croudia::API::Favorites do
+  before do
+    @client = Croudia::Client.new
+  end
+
+  describe '#favorite' do
+    before do
+      stub_post('/favorites/create/1234.json').to_return(
+        body: fixture(:status),
+        headers: { content_type: 'application/json; charset=utf-8' }
+      )
+    end
+
+    it 'requests the correct resource' do
+      @client.favorite(1234)
+      expect(a_post('/favorites/create/1234.json')).to have_been_made
+    end
+  end
+
+  describe '#unfavorite' do
+    before do
+      stub_delete('/favorites/destroy/1234.json').to_return(
+        body: fixture(:status),
+        headers: { content_type: 'application/json; charset=utf-8' }
+      )
+    end
+
+    it 'requests the correct resource' do
+      @client.unfavorite(1234)
+      expect(a_delete('/favorites/destroy/1234.json')).to have_been_made
+    end
+  end
+end

data/spec/croudia/api/oauth_spec.rb

@@ -0,0 +1,53 @@
+require 'cgi'
+require 'helper'
+require 'uri'
+
+describe Croudia::API::OAuth do
+  before do
+    options = { client_id: 'cid-value', client_secret: 'cs-value' }
+    @client = Croudia::Client.new(options)
+  end
+
+  describe '#authorize_url' do
+    it 'returns the correct url' do
+      uri = URI.parse(@client.authorize_url)
+      expect(uri.path).to eq '/oauth/authorize'
+      expect(CGI.parse(uri.query)).to eq({
+        'response_type' => ['code'],
+        'client_id' => ['cid-value'],
+      })
+    end
+
+    it 'does not include the client_secret in URL' do
+      url = @client.authorize_url
+      expect(url).not_to include('cs-value')
+    end
+    
+    it 'includes argument hash to the URL' do
+      url = @client.authorize_url(hoge: 'fuga', foo: :bar)
+      query = CGI.parse(URI.parse(url).query)
+      expect(query['hoge']).to include('fuga')
+      expect(query['foo']).to include('bar')
+    end
+  end
+
+  describe '#get_access_token' do
+    before do
+      stub_post('/oauth/token').to_return(
+        body: fixture(:access_token),
+        headers: { content_type: 'application/json; charset=utf-8' }
+      )
+    end
+
+    it 'requests the correct resource' do
+      @client.get_access_token(code: 'code-value')
+      expect(a_post('/oauth/token').with(
+        body: {
+          code: 'code-value',
+          grant_type: 'authorization_code',
+          client_id: 'cid-value',
+          client_secret: 'cs-value',
+      })).to have_been_made
+    end
+  end
+end

data/spec/croudia/api/statuses_spec.rb

@@ -0,0 +1,73 @@
+require 'helper'
+
+describe Croudia::API::Statuses do
+  before do
+    @client = Croudia::Client.new
+  end
+
+  describe '#update' do
+    before do
+      stub_post('/statuses/update.json').with(
+        body: {
+          status: 'Hello',
+          in_reply_to_status_id: '1234',
+        }
+      ).to_return(
+        body: fixture(:status),
+        headers: { content_type: 'application/json; charset=utf-8' }
+      )
+    end
+
+    context 'when text is passed as a string' do
+      it 'requests the correct resource' do
+        @client.update('Hello', in_reply_to_status_id: 1234)
+        expect(a_post('/statuses/update.json').with(
+          body: { 
+            status: 'Hello',
+            in_reply_to_status_id: '1234',
+          }
+        )).to have_been_made
+      end
+    end
+
+    context 'when text is passed as a value of hash' do
+      it 'requests the correct resource' do
+        @client.update(status: 'Hello', in_reply_to_status_id: 1234)
+        expect(a_post('/statuses/update.json').with(
+          body: {
+            status: 'Hello',
+            in_reply_to_status_id: '1234',
+          }
+        )).to have_been_made
+      end
+    end
+  end
+
+  describe '#destroy_status' do
+    before do
+      stub_post('/statuses/destroy/1234.json').to_return(
+        body: fixture(:status),
+        headers: { content_type: 'application/json; charset=utf-8' }
+      )
+    end
+
+    it 'requests the correct resource' do
+      @client.destroy_status(1234)
+      expect(a_post('/statuses/destroy/1234.json')).to have_been_made
+    end
+  end
+
+  describe '#status' do
+    before do
+      stub_get('/statuses/show/1234.json').to_return(
+        body: fixture(:status),
+        header: { content_type: 'application/json; charset=utf-8' }
+      )
+    end
+
+    it 'requests the correct resource' do
+      @client.status(1234)
+      expect(a_get('/statuses/show/1234.json')).to have_been_made
+    end
+  end
+end

data/spec/croudia/api/timelines_spec.rb

@@ -0,0 +1,93 @@
+require 'helper'
+
+describe Croudia::API::Timelines do
+  before do
+    @client = Croudia::Client.new
+  end
+
+  describe '#public_timeline' do
+    before do
+      stub_get('/statuses/public_timeline.json').to_return(
+        body: fixture(:timeline),
+        headers: { content_type: 'application/json; charset=utf-8' }
+      )
+    end
+
+    it 'requests the correct resource' do
+      @client.public_timeline
+      expect(a_get('/statuses/public_timeline.json')).to have_been_made
+    end
+  end
+
+  describe '#home_timeline' do
+    before do
+      stub_get('/statuses/home_timeline.json').to_return(
+        body: fixture(:timeline),
+        headers: { content_type: 'application/json; charset=utf-8' }
+      )
+    end
+
+    it 'requests the correct resource' do
+      @client.home_timeline
+      expect(a_get('/statuses/home_timeline.json')).to have_been_made
+    end
+  end
+
+  describe '#user_timeline' do
+    context 'without a screen_name' do
+      it 'is not supported' do
+        expect { @client.user_timeline }.to raise_error ArgumentError
+      end
+    end
+
+    context 'with a screen_name (string)' do
+      before do
+        stub_get('/statuses/user_timeline.json').with(
+          query: { screen_name: 'wktk' }
+        ).to_return(
+          body: fixture(:timeline),
+          headers: { content_type: 'application/json; charset=utf-8' }
+        )
+      end
+
+      it 'requests the correct resource with the screen_name passed' do
+        @client.user_timeline('wktk')
+        expect(a_get('/statuses/user_timeline.json').with(
+          query: { screen_name: 'wktk' }
+        )).to have_been_made
+      end
+    end
+
+    context 'with a user_id (integer)' do
+      before do
+        stub_get('/statuses/user_timeline.json').with(
+          query: { user_id: 1234 }
+        ).to_return(
+          body: fixture(:timeline),
+          headers: { content_type: 'application/json; charset=utf-8' }
+        )
+      end
+
+      it 'requests the correct resource with the user_id passed' do
+        @client.user_timeline(1234)
+        expect(a_get('/statuses/user_timeline.json').with(
+          query: { user_id: 1234 }
+        )).to have_been_made
+      end
+    end
+  end
+
+  describe '#mentions' do
+    before do
+      stub_get('/statuses/mentions.json').to_return(
+        body: fixture(:timeline),
+        headers: { content_type: 'application/json; charset=utf-8' }
+      )
+    end
+
+    it 'requests the correct resource' do
+      @client.mentions
+      expect(a_get('/statuses/mentions.json')).to have_been_made
+    end
+  end
+end

data/spec/croudia/base_spec.rb

@@ -0,0 +1,27 @@
+require 'helper'
+
+describe Croudia::Base do
+  before do
+    @base = Croudia::Base.new(id: 1)
+  end
+
+  describe '#[]' do
+    it 'calls methods using [] with symbol' do
+      expect(@base[:object_id]).to be_an Integer
+    end
+
+    it 'calls methods using [] with string' do
+      expect(@base['object_id']).to be_an Integer
+    end
+
+    it 'returns nil for missing method' do
+      expect(@base[:a_missing_method_hoge]).to be_nil
+    end
+  end
+
+  describe '#attrs' do
+    it 'returns a hash of attributes' do
+      expect(@base.attrs).to eq({id: 1})
+    end
+  end
+end

data/spec/croudia/client_spec.rb

@@ -0,0 +1,115 @@
+require 'helper'
+
+describe Croudia::Client do
+  context 'with module counfiguration' do
+    before do
+      Croudia.configure do |config|
+        Croudia::Configurable.keys.each do |key|
+          config.send("#{key}=", key)
+        end
+      end
+    end
+
+    after do
+      Croudia.reset!
+    end
+
+    it 'inherits the module configuration' do
+      client = Croudia::Client.new
+      Croudia::Configurable.keys.each do |key|
+        expect(client.instance_variable_get(:"@#{key}")).to eq key
+      end
+    end
+  end
+
+  context 'with class configuration' do
+    before do
+      @configuration = {
+        connection_options: { timeout: 10 },
+        client_id: 'CK',
+        client_secret: 'CS',
+        endpoint: 'http://twitter.com',
+        middleware: Proc.new { },
+        access_token: 'AT',
+      }
+    end
+
+    context 'during initialization' do
+      it 'overrides the modue configuration' do
+        client = Croudia::Client.new(@configuration)
+        Croudia::Configurable.keys.each do |key|
+          expect(client.instance_variable_get(:"@#{key}")).to eq @configuration[key]
+        end
+      end
+    end
+
+    context 'after initialization' do
+      it 'overrides the module configuration after initialization' do
+        client = Croudia::Client.new
+        client.configure do |config|
+          @configuration.each do |key, value|
+            config.send("#{key}=", value)
+          end
+        end
+        Croudia::Configurable.keys.each do |key|
+          expect(client.instance_variable_get(:"@#{key}")).to eq @configuration[key]
+        end
+      end
+    end
+  end
+
+  describe '#delete' do
+    before do
+      stub_delete('/delete').with(query: { deleted: 'object' })
+    end
+
+    it 'allows custom delete requests' do
+      Croudia::Client.new.delete('/delete', { deleted: 'object' })
+      expect(a_delete('/delete').with(
+        query: { deleted: 'object' }
+      )).to have_been_made
+    end
+  end
+
+  describe '#get' do
+    before do
+      stub_get('/get').with(query: { hoge: 'fuga' })
+    end
+
+    it 'allows custom get requests' do
+      Croudia::Client.new.get('/get', { hoge: 'fuga' })
+      expect(a_get('/get').with(
+        query: { hoge: 'fuga'}
+      )).to have_been_made
+    end
+  end
+
+  describe '#post' do
+    before do
+      stub_post('/post').with(body: { posted: 'object' })
+    end
+
+    it 'allows custom post requests' do
+      Croudia::Client.new.post('/post', { posted: 'object' })
+      expect(a_post('/post').with(
+        body: { posted: 'object' }
+      )).to have_been_made
+    end
+  end
+
+  describe '#put' do
+    before do
+      stub_put('/put').with(body: { updated: 'object'})
+    end
+
+    it 'allows custom delete requests' do
+      Croudia::Client.new.put('/put', { updated: 'object' })
+      expect(a_put('/put').with(
+        body: { updated: 'object' }
+      )).to have_been_made
+    end
+  end
+
+  describe '#request' do
+  end
+end

data/spec/croudia/identity_spec.rb

@@ -1,65 +1,29 @@
-require 'helper'
-
-describe Croudia::Identity do
-  describe '#initialize' do
-    it 'sets arguments as instance variables' do
-      Croudia::Identity.new(:hello => 'hi').instance_variable_get('@hello').should eq 'hi'
-    end
-
-    it 'converts a Unix time attribution named time to a Time object' do
-      Croudia::Identity.new(:time => '1234567890').time.should be_a Time
-    end
-
-    it 'converts a date string named time to a Time object' do
-      Croudia::Identity.new(:time => '2013-01-01 11:55').time.should be_a Time
-    end
-
-    it 'converts *_count into an Integer' do
-      Croudia::Identity.new(:foo_count => '2012').foo_count.should eq 2012
-    end
-  end
-
-  describe '#==' do
-    it 'returns true when IDs match' do
-      obj1 = Croudia::Identity.new(:id => '1', :username => 'wktk')
-      obj2 = Croudia::Identity.new(:id => '1', :username => 'croudia')
-      (obj1 == obj2).should be_true
-    end
-
-    it 'returns false when IDs do not match' do
-      obj1 = Croudia::Identity.new(:id => '1', :username => 'wktk')
-      obj2 = Croudia::Identity.new(:id => '2', :username => 'wktk')
-      (obj1 == obj2).should be_false
-    end
-
-    it 'returns false when classes do not match' do
-      obj1 = Croudia::Identity.new(:id => '1')
-      obj2 = { :id => '1' }
-      (obj1 == obj2).should be_false
-    end
-  end
-
-  describe '#[]' do
-    it 'returns the same name instance variable' do
-      Croudia::Identity.new(:foo => 'bar')[:foo].should eq 'bar'
-    end
-  end
-
-  describe '#id' do
-    it 'returns @id, not object_id' do
-      identity = Croudia::Identity.new(:id => '1')
-      identity.id.should_not eq identity.object_id
-      identity.id.should eq identity.instance_variable_get('@id')
-    end
-  end
-
-  describe '#method_missing' do
-    it 'returns the same name instance variable if set' do
-      Croudia::Identity.new(:foo => 'bar').foo.should eq 'bar'
-    end
-
-    it 'returns nil if not set' do
-      Croudia::Identity.new(:foo => 'bar').bar.should be_nil
-    end
-  end
-end
+require 'helper'
+
+describe Croudia::Identity do
+  describe '#initialize' do
+    it 'raises ArguentError if :id attr is missing' do
+      expect { Croudia::Identity.new }.to raise_error ArgumentError
+    end
+  end
+
+  describe '#==' do
+    it 'returns true when objects IDs are the same' do
+      one = Croudia::Identity.new(id: 1, text: 'hoge')
+      two = Croudia::Identity.new(id: 1, text: 'fuga')
+      expect(one == two).to be_true
+    end
+
+    it 'returns false when objects IDs are different' do
+      one = Croudia::Identity.new(id: 1)
+      two = Croudia::Identity.new(id: 2)
+      expect(one == two).to be_false
+    end
+
+    it 'returns false when classes are different' do
+      one = Croudia::Identity.new(id: 1)
+      two = Croudia::Base.new(id: 1)
+      expect(one == two).to be_false
+    end
+  end
+end

data/spec/croudia/status_spec.rb

@@ -0,0 +1,30 @@
+require 'helper'
+
+describe Croudia::Status do
+  describe '#==' do
+    it 'returns true when objects IDs are the same' do
+      status = Croudia::Status.new(id: 1, text: 'hoge')
+      other = Croudia::Status.new(id: 1, text: 'fuga')
+      expect(status == other).to be_true
+    end
+
+    it 'returns false when objects IDs are different' do
+      status = Croudia::Status.new(id: 1)
+      other = Croudia::Status.new(id: 2)
+      expect(status == other).to be_false
+    end
+
+    it 'returns false when classes are different' do
+      status = Croudia::Status.new(id: 1)
+      other = Croudia::Identity.new(id: 1)
+      expect(status == other).to be_false
+    end
+  end
+
+  describe '#created_at' do
+    it 'returns a Time' do
+      status = Croudia::Status.new(id: 3, created_at: 'Mon, 08 Jul 2013 01:23:45 +0900')
+      expect(status.created_at).to be_a Time
+    end
+  end
+end