termtter 1.6.0 → 1.7.0

data/spec/plugins/draft_spec.rb

@@ -0,0 +1,59 @@
+require File.dirname(__FILE__) + '/../spec_helper'
+
+module Termtter
+  describe 'plugin draft' do
+    before do
+      Client.setup_task_manager
+      Client.plug 'draft'
+      Client.public_storage[:drafts].clear
+      Client.public_storage[:drafts] << 'foo'
+      Client.public_storage[:drafts] << 'bar'
+      Client.stub!(:puts)
+    end
+
+    it 'should puts drafts' do
+      Client.should_receive(:puts).with("0: foo")
+      Client.should_receive(:puts).with("1: bar")
+      Client.execute('draft list')
+    end
+
+    it 'should clear drafts' do
+      Client.execute('draft clear')
+      Client.public_storage[:drafts].size.should == 0
+    end
+
+    it 'should exec last draft' do
+      Client.should_receive(:execute).with('bar')
+      Client.get_command(:'draft exec').call('draft exec')
+    end
+
+    it 'should exec specified draft' do
+      Client.should_receive(:execute).with('foo')
+      Client.get_command(:'draft exec').call('draft exec', '0')
+    end
+
+    it 'should not exec draft if index is wrong' do
+      Client.should_not_receive(:execute)
+      Client.get_command(:'draft exec').call('draft exec', '2')
+      Client.should_not_receive(:execute)
+      Client.get_command(:'draft exec').call('draft exec', 'a')
+    end
+
+    it 'should delete draft' do
+      Client.get_command(:'draft delete').call('draft delete')
+      Client.public_storage[:drafts].should == ["foo"]
+    end
+
+    it 'should delete specified draft' do
+      Client.get_command(:'draft delete').call('draft delete', '0')
+      Client.public_storage[:drafts].should == ["bar"]
+    end
+
+    it 'should not delete draft if index is wrong' do
+      Client.get_command(:'draft delete').call('draft delete', '2')
+      Client.public_storage[:drafts].should == ["foo", "bar"]
+      Client.get_command(:'draft delete').call('draft delete', 'a')
+      Client.public_storage[:drafts].should == ["foo", "bar"]
+    end
+  end
+end

data/spec/plugins/expand-tinyurl_spec.rb

@@ -0,0 +1,21 @@
+# -*- coding: utf-8 -*-
+
+require File.dirname(__FILE__) + '/../spec_helper'
+
+describe Termtter::Client, 'when the plugin expand-tinyurl is loaded' do
+  before do
+    be_quiet { Termtter::Client.plug 'expand-tinyurl' }
+  end
+
+  it 'should define expand_url method' do
+    # TODO: 直接ネットにアクセスしに行かないようにしたい。 fakeweb?
+
+    expand_url('tinyurl.com', '/kotu').should == 'http://example.com/'
+
+    expand_url('tinyurl.com', '/de5my6').should == 'http://example.com/テスト'
+
+    expand_url('is.gd', '/5oDxw').should == 'http://example.com/'
+
+    expand_url('goo.gl', '/e').should == 'http://www.google.com/'
+  end
+end

data/spec/plugins/footer_spec.rb

@@ -0,0 +1,50 @@
+#-*- coding: utf-8 -*-
+require File.dirname(__FILE__) + '/../spec_helper.rb'
+
+describe Termtter::Client, 'when the plugin footer is loaded' do
+  @r = nil
+  before(:all) do
+    @r = nil
+    Termtter::Client.clear_hooks
+    Termtter::Client.setup_task_manager
+
+    config.footer = "[termtter]"
+
+  end
+
+  before do
+    @r = nil
+    Termtter::Client.register_command(:name => :update, :aliases => [:u], :exec => lambda{|arg| @r = arg })
+    Termtter::Client.plug 'footer'
+  end
+
+  it 'should add hook add_footer and command footer' do
+    Termtter::Client.should_receive(:register_hook).once
+    Termtter::Client.should_receive(:register_command).once
+    Termtter::Client.plug 'footer'
+  end
+
+  it 'should add footer at update' do
+    @r.should be_nil
+    Termtter::Client.execute('update foo')
+    config.footer.should == '[termtter]'
+    @r.should == 'foo [termtter]'
+  end
+
+  it 'should call footer command to change config.footer' do
+    Termtter::Client.execute('footer #termtter')
+    config.footer.should == '#termtter'
+    @r.should be_nil
+    Termtter::Client.execute('update bar')
+    @r.should == 'bar #termtter'
+  end
+
+  it 'should call footer no argument to set config.footer to nil' do
+    Termtter::Client.execute('footer')
+    config.footer.should == nil
+    @r.should be_nil
+    Termtter::Client.execute('update hoge')
+    @r.should == 'hoge'
+  end
+end
+

data/spec/plugins/storage/sqlite3_spec.rb

@@ -0,0 +1,41 @@
+require 'tmpdir'
+require File.dirname(__FILE__) + '/../../spec_helper'
+require File.dirname(__FILE__) + '/../../../lib/plugins/storage/sqlite3'
+
+module Termtter::Storage
+  describe "sqlite3" do
+    DB_FILE = File.join(Dir.tmpdir, 'test.db')
+    before(:each) do
+      File.delete(DB_FILE) if File.exists?(DB_FILE)
+      @db = Termtter::Storage::SQLite3.new(DB_FILE)
+    end
+
+    after do
+      File.delete(DB_FILE) if File.exists?(DB_FILE)
+    end
+
+    it 'update should not return false' do
+      h = {
+        :post_id => 1,
+        :created_at => 12345,
+        :in_reply_to_status_id => -1,
+        :in_reply_to_user_id => -1,
+        :text => 'bomb',
+        :user_id => 1
+      }
+      @db.update(h).should_not be_false
+    end
+
+    it 'find_id returns status' do
+    end
+
+    it 'find_text returns status' do
+    end
+
+    it 'find_user returns status' do
+    end
+
+    it 'size of statuses' do
+    end
+  end
+end

data/spec/termtter/api_spec.rb

@@ -61,7 +61,7 @@ module Termtter
         it_should_examine_with('test', nil) { nil }
       }
       result[:stdout].should ==
-        "Please enter your Twitter login:\nPassword: \n"
+        "Please enter your Twitter login:\nUsername: test\nPassword: \n"
     end
 
     it 'can examine username and password (only pass)' do

data/spec/termtter/client_spec.rb

@@ -48,7 +48,8 @@ module Termtter
       config.system.eval_scripts = []
       config.system.cmd_mode = false
       Client.should_receive(:call_hooks).with(:initialize)
-      Client.should_receive(:call_hooks).with(:launched)
+      Client.should_receive(:call_hooks).with(:init_command_line)
+      # NOTE: :launched も呼ばれるはず
       Client.run
     end
 
@@ -163,9 +164,9 @@ module Termtter
         ['test  foo bar ',  'foo bar'],
         ['test  foo  bar ', 'foo  bar'],
       ].each do |input, args|
-        status = Client.call_commands(input)
+        status = Client.execute(input)
         command_arg.should == args
-        status.should == 0
+        status.should == true
       end
     end
 
@@ -191,7 +192,7 @@ module Termtter
       input_command.should be_nil
       input_arg.should be_nil
       decided_arg.should be_nil
-      Client.call_commands('u foo')
+      Client.execute('u foo')
       input_command.should == 'u'
       input_arg.should == 'foo'
       decided_arg.should == 'FOO'
@@ -205,11 +206,11 @@ module Termtter
       Client.register_command(:name => :update, :exec => lambda{|arg|})
 
       hook_called.should == false
-      Client.call_commands('')
+      Client.execute('')
       hook_called.should == true
 
       hook_called = false
-      Client.call_commands('update foo')
+      Client.execute('update foo')
       hook_called.should == true
     end
 
@@ -221,10 +222,10 @@ module Termtter
       Client.register_command(:name => :update, :exec => lambda{|arg|})
 
       hook_called.should == false
-      Client.call_commands('')
+      Client.execute('')
       hook_called.should == false
 
-      Client.call_commands('update foo')
+      Client.execute('update foo')
       hook_called.should == true
     end
 
@@ -236,7 +237,7 @@ module Termtter
       Client.register_command(:name => :update, :exec => lambda{|arg|})
 
       hook_called.should == false
-      Client.call_commands('update foo')
+      Client.execute('update foo')
       hook_called.should == true
     end
 
@@ -248,7 +249,7 @@ module Termtter
       Client.register_command(:name => :update, :exec_proc => lambda {|arg| command_called = true})
 
       command_called.should == false
-      Client.call_commands('update foo')
+      Client.execute('update foo')
       command_called.should == false
     end
 
@@ -260,7 +261,7 @@ module Termtter
       Client.register_command(:name => :update, :exec_proc => lambda {|arg| 'foo'})
 
       command_result.should == nil
-      Client.call_commands('update foo')
+      Client.execute('update foo')
       command_result.should == 'foo'
     end
 
@@ -444,7 +445,7 @@ module Termtter
         }
       )
       command.should_not_receive(:call)
-      Client.call_commands('test')
+      Client.execute('test')
     end
 
     it 'gets default help' do
@@ -533,10 +534,8 @@ module Termtter
       text = 'text'
       command = mock('command', :null_object => true)
       command.stub(:call) { raise CommandCanceled }
-      Client.stub(:find_commands).with(text).and_return([command])
-      lambda {
-        Client.call_commands(text).should == 1
-      }.should_not raise_error
+      Client.stub(:find_command).with(text).and_return(command)
+      Client.execute(text).should == false
     end
 
     describe 'add commands' do
@@ -545,16 +544,17 @@ module Termtter
         Client.register_command(:name => :foo1)
         Client.register_command(:name => :foo2)
         Client.register_command(:name => :bar)
+        Client.register_command(:name => 'bar xxx')
       end
 
       it 'commands number is 3' do
-        Client.commands.size.should == 3
+        Client.commands.size.should == 4
       end
 
       it 'finds a command' do
-        Client.find_commands('foo1').size.should == 1
-        Client.find_commands('foo1')[0].name.should == :foo1
-        Client.find_commands('bar').size.should == 1
+        Client.find_command('foo1').name.should == :foo1
+        Client.find_command('bar').name.should == :bar
+        Client.find_command('bar xxx').name.should == :'bar xxx'
       end
 
       it 'check command exists' do
@@ -565,7 +565,7 @@ module Termtter
       end
 
       it 'finds no command' do
-        Client.find_commands('foo').size.should == 0
+        Client.find_command('foo').should be_nil
       end
     end
 

data/spec/termtter/command_spec.rb

@@ -5,19 +5,19 @@ require File.dirname(__FILE__) + '/../spec_helper'
 module Termtter
   describe 'Command#initialize' do
     it 'requires the name element in the argument hash' do
-      lambda { Command.new(:nama => 1) }.should raise_error(ArgumentError)
-      lambda { Command.new(:name => 1) }.should_not raise_error(ArgumentError)
+      lambda { Command.new(:nama => :a) }.should raise_error(ArgumentError)
+      lambda { Command.new(:name => :a) }.should_not raise_error(ArgumentError)
     end
 
     it 'does not destroy the argument hash' do
       hash = {
-        :name => 1,
+        :name => 'a',
         :exec => 3
       }
       Command.new hash
 
       hash.should eql(hash)
-      hash[:name].should == 1
+      hash[:name].should == 'a'
       hash[:exec].should == 3
       hash[:exec_proc].should be_nil
     end
@@ -66,11 +66,9 @@ module Termtter
     it 'returns candidates for completion' do
       # complement
       [
-        ['upd',       ['update']],
-        [' upd',      []],
-        [' upd ',     []],
+        ['update  ', ['complete1', 'complete2']],
+        [' update  ', ['complete1', 'complete2']],
         ['update a',  ['complete1', 'complete2']],
-        [' update  ', []],
         ['u foo',     ['complete1', 'complete2']],
       ].each do |input, comp|
         @command.complement(input).should == comp
@@ -143,6 +141,8 @@ module Termtter
           should == ['test', 'foo  bar']
         @command.split_command_line(' test   foo  bar ').
           should == ['test', 'foo  bar']
+        @command.split_command_line('test ').
+          should == ['test', '']
       end
     end
 

data/spec/termtter/config_spec.rb

@@ -72,8 +72,8 @@ module Termtter
 
     it 'can raise error when add by prohibited name' do
       lambda {
-        @config.set_default('sub.aaa', :value)
-        @config.sub.aaa
+        @config.set_default('open.aaa', :value)
+        @config.open.aaa
       }.should raise_error
     end
 

data/spec/termtter/memory_cache_spec.rb

@@ -0,0 +1,20 @@
+# -*- coding: utf-8 -*-
+require File.dirname(__FILE__) + '/../spec_helper'
+
+module Termtter
+  describe MemoryCache do
+    it 'is like Hash' do
+      cache = MemoryCache.new(2)
+      cache[:foo] = 'foo'
+      cache[:bar] = 'bar'
+
+      cache[:foo].should == 'foo'
+      cache[:foo].should == 'foo'
+
+      cache[:jugyo] = 'jugyo'
+      cache[:jugyo].should == 'jugyo'
+
+      cache.key?(:foo).should be_false
+    end
+  end
+end

data/spec/termtter/optparse_spec.rb

@@ -13,7 +13,7 @@ module Termtter
     end
 
     def run_termtter(opt)
-      `./run_termtter.rb #{opt}`
+      `./bin/termtter #{opt}`
     end
 
     it 'accepts -h option' do

data/spec/termtter/rubytter_proxy_spec.rb

@@ -38,5 +38,43 @@ module Termtter
 
       @twitter.update('test')
     end
+
+    it 'should retry to be success' do
+      config.retry = 3
+      @rubytter_mock.stub!(:update).exactly(1).times
+      @twitter.update('test')
+    end
+
+    it 'should retry when raise TimeoutError' do
+      config.retry = 3
+      @rubytter_mock.stub!(:update).exactly(config.retry).times.and_raise(TimeoutError)
+      @twitter.update('test')
+    end
+
+    it 'should store cache when call "show"' do
+      status = "status"
+      @rubytter_mock.should_receive(:show).exactly(1).and_return(status)
+      @twitter.should_receive(:store_status_cache).with(status)
+      @twitter.show(1)
+    end
+
+    it 'should store cache when call "home_timeline"' do
+      statuses = ["1", "2"]
+      @rubytter_mock.should_receive(:home_timeline).exactly(1).and_return(statuses)
+      @twitter.should_receive(:store_status_cache).exactly(2)
+      @twitter.home_timeline
+    end
+
+    it 'should store cache when call "store_status_cache"' do
+      user = "user"
+      @twitter.should_receive(:store_user_cache).with(user)
+      @twitter.store_status_cache(Rubytter.structize({:user => user}))
+    end
+
+    it 'should not call rubytter if cache exists' do
+      @twitter.status_cache_store[1] = "status"
+      @rubytter_mock.should_receive(:show).exactly(0)
+      @twitter.show(1).should == "status"
+    end
   end
 end