termtter 1.5.0 → 1.6.0

data/spec/termtter/config_spec.rb

@@ -8,28 +8,54 @@ module Termtter
       @config = Config.new
     end
 
-    it 'should be able to store value to new storage' do
+    it 'can store value to new storage' do
       @config.new_storage = :value
       @config.new_storage.should == :value
     end
 
-    it 'should be able to make subb.key and store value' do
+    it 'can make subb.key and store value' do
       @config.subb.key = :value
       @config.subb.key.should == :value
     end
 
-    it 'should be able to make multiple storage' do
+    it 'can make multiple storage' do
       @config.subb.more.for.test = 'value'
       @config.subb.more.for.test.should == 'value'
     end
 
-    it 'should be able to change value in storage' do
+    it 'can change value in storage' do
       @config.storage = :value1
       @config.storage = :value2
       @config.storage.should == :value2
     end
 
-    it 'should be able to store any data' do
+    it 'can inspect' do
+      @config.storage = :value
+      @config.inspect.should == {:storage => :value}.inspect
+    end
+
+    it 'can get __values__' do
+      @config.storage = :value
+      @config.__values__.should == {:storage => :value}
+    end
+
+    it 'can __clear__' do
+      @config.storage = :value
+      @config.storage.should == :value
+      @config.__clear__
+      result = @config.__values__
+      result.should be_empty
+      result.should be_an_instance_of Hash
+    end 
+
+    it 'can be called __clear__ with name' do
+      @config.foo = 'foo'
+      @config.foo.should_not be_empty
+      @config.__clear__(:foo)
+      @config.foo.should be_empty
+    end
+
+    it 'can store any data' do
       [
         ['string',  'value'   ],
         ['symbol',  :value    ],
@@ -44,14 +70,14 @@ module Termtter
       end
     end
 
-    it 'should raise error when add by prohibited name' do
+    it 'can raise error when add by prohibited name' do
       lambda {
         @config.set_default('sub.aaa', :value)
         @config.sub.aaa
       }.should raise_error
     end
 
-    it 'should raise error when add subb-storage to existed key' do
+    it 'can raise error when add subb-storage to existed key' do
       @config.subb = 'original value'
       lambda {
         @config.subb.key = 'invalid subbstitution'
@@ -61,7 +87,7 @@ module Termtter
       )
     end
 
-    it 'should set intermediate defult configs' do
+    it 'can set intermediate defult configs' do
       @config.set_default 'subb.more', 'value'
       @config.subb.class.should == Config
       @config.subb.more.should == 'value'
@@ -70,31 +96,29 @@ module Termtter
       @config.proxy.port.should == 'value'
     end
 
-    # FIXME: not work
+    # FIXME: Is this need spec?
 #     it 'should have :undefined value in un-assigned key' do
 #       @config.aaaa.should == :undefined
 #     end
 
-    it 'should be empty when something is assigned' do
-      @config.empty?.should be_true
-
+    it 'can examin that storage is empty' do
+      @config.should be_empty
       @config.aaa = 1
-      @config.empty?.should be_false
-
-      @config.bbb.empty?.should be_true
+      @config.should_not be_empty
+      @config.bbb.should be_empty
     end
 
-    it 'should be empty when assigned nil' do
+    it 'can examin that storage is not empty (nil)' do
       @config.bbb = nil
-      @config.empty?.should be_false
+      @config.should_not be_empty
     end
 
-    it 'should be empty when set_defaulted' do
+    it 'can examin that storage is not empty (default)' do
       @config.set_default('aaa', 1)
-      @config.empty?.should be_false
+      @config.should_not be_empty
     end
 
-    it 'should use in expression' do
+    it 'can use in expression' do
       @config.set_default(:ssb, 'hoge')
       lambda {
         res = @config.ssb + ' piyo'
@@ -102,20 +126,26 @@ module Termtter
       }.should_not raise_error
     end
 
-    it 'should not change value when call set_default twice' do
+    it 'can not change value when call set_default twice' do
       @config.plugins.set_default :only, 'before_value'
       @config.plugins.set_default :only, 'after_value'
       @config.plugins.only.should == 'before_value'
     end
 
-    it 'should be called set_default with int multiple times' do
-      @config.set_default(:foo, 1)
-      @config.set_default(:foo, 2)
+    it 'can be called set_default with int multiple times' do
+      lambda {
+        @config.set_default(:foo, 1)
+        @config.set_default(:foo, 2)
+        @config.foo.should == 1
+      }.should_not raise_error
     end
 
-    it 'should be called set_default with string multiple times' do
-      @config.set_default(:foo, 'foo')
-      @config.set_default(:foo, 'bar')
+    it 'can be called set_default with string multiple times' do
+      lambda {
+        @config.set_default(:foo, 'foo')
+        @config.set_default(:foo, 'bar')
+        @config.foo.should == 'foo'
+      }.should_not raise_error
     end
   end
 end

data/spec/termtter/hook_spec.rb

@@ -62,6 +62,18 @@ module Termtter
       hook.match?(:'_bar_').should == false
     end
 
+    it 'should not match invalid point' do
+      hook = Hook.new(
+        :name => :spam,
+        :points => [1, { }, nil],
+        :exec_proc => lambda{|cmd, arg|
+        }
+        )
+      hook.match?(1).should == false
+      hook.match?({ }).should == false
+      hook.match?(nil).should == false
+    end
+
     it 'call hook proc' do
       proc_args = nil
       hook = Hook.new(

data/spec/termtter/hookable_spec.rb

@@ -0,0 +1,53 @@
+# -*- coding: utf-8 -*-
+require File.dirname(__FILE__) + '/../spec_helper'
+
+module Dummy
+  include Termtter::Hookable
+end
+
+module Termtter
+  describe Hookable do
+
+    it 'takes new_hook' do
+      hook = Hook.new(:name => :test)
+      Dummy.register_hook(hook)
+      Dummy.get_hook(:test).should == hook
+    end
+
+    it 'takes hook as Hash' do
+      Dummy.register_hook(:name => :test)
+      Dummy.get_hook(:test).name.should == :test
+    end
+
+    it 'calls new_hook' do
+      hook_called = false
+      Dummy.register_hook(:name => :test1, :points => [:point1], :exec_proc => lambda {hook_called = true})
+      hook_called.should == false
+      Dummy.call_hooks(:point1)
+      hook_called.should == true
+    end
+
+    it 'calls new_hook with args' do
+      arg1 = nil
+      arg2 = nil
+      Dummy.register_hook(:name => :test1, :points => [:point1], :exec_proc => lambda {|a1, a2| arg1 = a1; arg2 = a2})
+      arg1.should == nil
+      arg2.should == nil
+      Dummy.call_hooks(:point1, 'foo', 'bar')
+      arg1.should == 'foo'
+      arg2.should == 'bar'
+    end
+
+    it 'return hooks when call get_hooks' do
+      hook1 = Dummy.register_hook(:name => :test1, :points => [:point1])
+      hook2 = Dummy.register_hook(:name => :test2, :points => [:point1])
+      hook3 = Dummy.register_hook(:name => :test3, :points => [:point2])
+
+      hooks = Dummy.get_hooks(:point1)
+      hooks.size.should == 2
+      hooks.include?(hook1).should == true
+      hooks.include?(hook2).should == true
+      hooks.include?(hook3).should == false
+    end
+  end
+end

data/spec/termtter/optparse_spec.rb

@@ -1,16 +1,71 @@
 # -*- coding: utf-8 -*-
 require File.dirname(__FILE__) + '/../spec_helper'
 
-describe 'optparse' do
-  def run_termtter(opt)
-    `./run_termtter.rb #{opt}`
-  end
+module Termtter
+  describe OptParser do
 
-  it 'accepts -h option' do
-    run_termtter('-h').should match(/Usage/)
-  end
+    before do
+      @original_conf ||= config
+    end
+
+    after do
+      conf = @original_conf
+    end
+
+    def run_termtter(opt)
+      `./run_termtter.rb #{opt}`
+    end
+
+    it 'accepts -h option' do
+      run_termtter('-h').should match(/Usage/)
+    end
+
+    it 'accepts -f option' do
+      Termtter::OptParser::parse!(%w{-f ~/config})
+      config.system.conf_file.should == '~/config'
+    end
+
+    it 'accepts -t option' do
+      Termtter::OptParser::parse!(%w{-t ~/termtter_directory})
+      config.system.conf_dir.should == '~/termtter_directory'
+    end
+
+    it 'accepts -d option' do
+      Termtter::OptParser::parse!(%w{-d})
+      config.devel.should be_true
+    end
+
+    it 'accepts -c option' do
+      Termtter::OptParser::parse!(%w{-c})
+      config.system.cmd_mode.should be_true
+    end
+
+    it 'accepts -r option' do
+      Termtter::OptParser::parse!(%w{-r cool})
+      config.system.run_commands.should == %w{cool}
+    end
+
+    it 'accepts -p option' do
+      Termtter::OptParser::parse!(%w{-p cool})
+      config.system.load_plugins.should == %w{cool}
+    end
+
+    it 'accepts -e option' do
+      Termtter::OptParser::parse!(%w{-e foo})
+      config.system.eval_scripts.should == %w{foo}
+    end
+
+    it 'accepts -m option' do
+      require 'termcolor'
+      TermColor.parse('<red>cool</red>').should == "\e[31mcool\e[0m"
+      Termtter::OptParser::parse!(%w{-m})
+      TermColor.parse('<red>cool</red>').should == 'cool'
+      module ::TermColor
+        class << self
+          alias parse parse_orig
+        end
+      end
+    end
 
-  it 'accepts -m option' do
-    pending
   end
 end

data/spec/termtter/rubytter_proxy_spec.rb

@@ -0,0 +1,42 @@
+# -*- coding: utf-8 -*-
+require File.dirname(__FILE__) + '/../spec_helper'
+
+module Termtter
+  describe RubytterProxy do
+    before do
+      @rubytter_mock = Object.new
+      Rubytter.stub!(:new).and_return(@rubytter_mock)
+      @twitter = RubytterProxy.new('foo', 'bar', {})
+    end
+
+    it 'should call a Rubytter\'s method' do
+      @rubytter_mock.should_receive(:update).with('test').once
+      @twitter.update('test')
+    end
+
+    it 'should call a Rubytter\'s method with block' do
+      block = Proc.new{}
+      @rubytter_mock.should_receive(:update).with('test').once
+      @twitter.update('test')
+    end
+
+    it 'should call hooks' do
+      pre_hook = RubytterProxy.register_hook(:pre, :point => :pre_update) {}
+      pre_hook.should_receive(:call).with('test').and_return(["TEST"])
+      post_hook = RubytterProxy.register_hook(:post, :point => :post_update) {}
+      post_hook.should_receive(:call).with('test')
+      @rubytter_mock.should_receive(:update).with("TEST")
+
+      @twitter.update('test')
+    end
+
+    it 'should cancel to call method' do
+      pre_hook = RubytterProxy.register_hook(:pre, :point => :pre_update) {raise HookCanceled}
+      post_hook = RubytterProxy.register_hook(:post, :point => :post_update) {}
+      post_hook.should_receive(:call).with('test').never
+      @rubytter_mock.should_receive(:update).never
+
+      @twitter.update('test')
+    end
+  end
+end

data/spec/termtter/system_extensions/windows_spec.rb

@@ -0,0 +1,9 @@
+# -*- coding: utf-8 -*-
+
+require File.dirname(__FILE__) + '/../../spec_helper'
+
+describe 'Extensions for Windows' do
+
+  it 'needs spec'
+end
+

data/spec/termtter/system_extensions_spec.rb

@@ -0,0 +1,61 @@
+# -*- coding: utf-8 -*-
+
+require File.dirname(__FILE__) + '/../spec_helper'
+require 'highline'
+
+describe Termtter do
+
+  it 'provides create_highline' do
+    h = create_highline
+    h.class.should == HighLine
+  end
+
+  it 'provides win?' do
+    be_quiet do
+      original_ruby_platform = RUBY_PLATFORM
+      RUBY_PLATFORM = 'darwin'
+      win?.should == false
+      RUBY_PLATFORM = 'mswin'
+      win?.should == true
+      RUBY_PLATFORM = original_ruby_platform
+    end
+  end
+
+  it 'Readline can refresh line' do
+    Readline::LIBREADLINE.should_receive(:rl_refresh_line).with(0, 0)
+    Readline.refresh_line
+  end
+
+  it 'extend DL::Impoter when not be able to find DL::Importable' do
+    DL::Importer = mock(:importer)
+    DL.stub(:const_defined?).with(:Importable).and_return(false)
+    Readline::LIBREADLINE.should_receive(:extend).with(DL::Importer)
+    load 'termtter/system_extensions.rb'
+  end
+
+  it 'can handle error when difine LIBREADLINE' do
+    Readline::LIBREADLINE.stub(:extend) { raise }
+    load 'termtter/system_extensions.rb'
+    Readline::LIBREADLINE.should_not_receive(:rl_refresh_line)
+    Readline.refresh_line
+  end
+
+  it 'can open browser that suites platform' do
+    be_quiet(:stdout => false) do
+      url = 'example.com'
+      [
+        ['linux',   'firefox'],
+        ['mswin',   'explorer'],
+        ['mingw',   'explorer'],
+        ['bccwin',  'explorer'],
+        ['darwin',  'open'],
+        ['hogehog', 'open'],
+      ].each do |platform, browser|
+        RUBY_PLATFORM = platform
+        self.should_receive(:system).with(browser, url)
+        open_browser(url)
+      end
+    end
+  end
+end
+

data/spec/termtter/task_manager_spec.rb

@@ -68,11 +68,69 @@ module Termtter
       @task_manager.instance_eval('@tasks').size.should == 1
     end
 
+    it 'should run (not pause)' do
+      be_quiet do
+        counter = 0
+        TaskManager::INTERVAL = 0
+        @task_manager.stub(:step) { counter += 1 }
+        @task_manager.instance_variable_set(:@work, true)
+        @task_manager.run
+        100.times do
+          break if counter != 0
+          sleep 0.1
+        end
+        @task_manager.instance_variable_set(:@work, false)
+        counter.should > 0
+      end
+    end
+
+    it 'should run (pause)' do
+      be_quiet do
+        counter = 0
+        TaskManager::INTERVAL = 0.1
+        @task_manager.stub(:step) { counter += 1 }
+        @task_manager.instance_variable_set(:@work, true)
+        @task_manager.instance_variable_set(:@pause, true)
+        @task_manager.run
+        sleep 0.1
+        counter.should == 0
+        @task_manager.instance_variable_set(:@pause, false)
+        100.times do
+          break if counter != 0
+          sleep 0.1
+        end
+        @task_manager.instance_variable_set(:@work, false)
+        counter.should > 0
+      end
+    end
+
     it 'should add task with :name' do
       @task_manager.add_task(:name => 'foo')
       @task_manager.get_task('foo').name.should == 'foo'
       @task_manager.delete_task('foo')
       @task_manager.get_task('foo').should == nil
     end
+
+    it 'should be killed' do
+      @task_manager.instance_eval('@work').should == true
+      @task_manager.kill
+      @task_manager.instance_eval('@work').should == false
+    end
+
+    it 'should be paused or resumed' do
+      @task_manager.instance_eval('@pause').should == false
+      @task_manager.pause
+      @task_manager.instance_eval('@pause').should == true
+      @task_manager.resume
+      @task_manager.instance_eval('@pause').should == false
+    end
+
+    it 'invoke_later calls invoke_and_wait' do
+      called = false
+      block = lambda { called = true}
+      @task_manager.should_receive(:invoke_and_wait).with(&block)
+      @task_manager.invoke_later &block
+      called.should be_true
+    end
   end
 end

data/spec/termtter_spec.rb

@@ -26,3 +26,14 @@ describe Termtter, 'when plugin is called (with init option)' do
     config.plugins.aaa.bbb.should == :ccc
   end
 end
+
+describe Termtter, 'when termtter is loaded' do
+  it 'will add load path' do
+    termtter_path = File.expand_path(File.dirname(__FILE__) + '/../lib/termtter.rb')
+    original_load_path = $:.dup
+    $:.clear
+    be_quiet { load termtter_path }
+    $:.include?(File.dirname(termtter_path)).should == true
+    $:.concat(original_load_path)
+  end
+end