consular-iterm 1.0.1 → 1.0.2

data/consular-iterm.gemspec

@@ -3,7 +3,7 @@ $:.push File.expand_path("../lib", __FILE__)
 
 Gem::Specification.new do |s|
   s.name        = "consular-iterm"
-  s.version     = "1.0.1"
+  s.version     = "1.0.2"
   s.authors     = ["Arthur Chiu"]
   s.email       = ["mr.arthur.chiu@gmail.com"]
   s.homepage    = "http://www.github.com/achiu/consular-iterm"
@@ -20,6 +20,7 @@ Gem::Specification.new do |s|
   s.add_dependency 'consular'
   s.add_dependency 'rb-appscript'
   s.add_development_dependency 'minitest'
+  s.add_development_dependency 'mocha'
 
   # specify any dependencies here; for example:
   # s.add_development_dependency "rspec"

data/lib/consular/iterm.rb

@@ -13,7 +13,7 @@ module Consular
 
     class << self
 
-      # Checks to see if the current system is darwin and 
+      # Checks to see if the current system is darwin and
       # if $TERM_PROGRAM is iTerm.app
       #
       # @api public
@@ -80,15 +80,14 @@ module Consular
 
     # Prepend a title setting command prior to the other commands.
     #
-    # @param [String] title
-    #   The title to set for the context of the commands.
-    # @param [Array<String>] commands
-    #   The context of commands to preprend to.
+    # @param [String]
+    #   The tab title.
+    # @param [Appscript::Reference]
+    #   The tab instance resulting from #open_tab.
     #
     # @api public
-    def set_title(title, commands)
-      cmd = "PS1=\"$PS1\\[\\e]2;#{title}\\a\\]\""
-      title ? commands.insert(0, cmd) : commands
+    def set_title(title, tab)
+      tab.name.set(title) if title
     end
 
     # Executes the commands for each designated window.
@@ -117,31 +116,31 @@ module Consular
     # @api public
     def execute_window(content, options = {})
       window_options = content[:options]
-      _contents      = content[:tabs]
-      _first_run     = true
+      tab_contents   = content[:tabs]
+      first_run      = true
 
-      _contents.keys.sort.each do |key|
-        _content = _contents[key]
-        _options = content[:options]
-        _name    = options[:name]
+      tab_contents.keys.sort.each do |key|
+        tab_content = tab_contents[key]
+        tab_options = tab_content[:options] || {}
+        tab_name    = tab_options[:name]
 
-        _tab =
-        if _first_run && !options[:default]
-          open_window options.merge(window_options)
-        else
-          key == 'default' ? active_tab : open_tab(_options) && active_tab
-        end
+        tab =
+          if first_run && !options[:default]
+            open_window options.merge(window_options)
+          else
+            key == 'default' ? active_tab : open_tab(tab_options) && active_tab
+          end
+        set_title tab_name, tab
 
-        _first_run = false
-        commands = prepend_befores _content[:commands], content[:before]
-        commands = set_title _name, commands
+        first_run = false
+        commands = prepend_befores tab_content[:commands], content[:before]
 
         if content.key? :panes
-          commands.each { |cmd| execute_command cmd, :in => _tab }
+          commands.each { |cmd| execute_command cmd, :in => tab }
           execute_panes content
           content.delete :panes
         else
-          commands.each { |cmd| execute_command cmd, :in => _tab }
+          commands.each { |cmd| execute_command cmd, :in => tab }
         end
       end
     end
@@ -233,7 +232,7 @@ module Consular
 
     # to select panes; iTerm's Appscript select method does not work
     # as expected, we have to select via menu instead
-    # 
+    #
     # @param [String] direction
     #   Direction to split the pane. The valid directions are:
     #   'Above', 'Below', 'Left', 'Right'

data/spec/iterm_spec.rb

@@ -23,9 +23,16 @@ describe Consular::ITerm do
     assert_equal ['ls'],       @core.prepend_befores(['ls'])
   end
 
-  it "should set .set_title" do
-    assert_equal ["PS1=\"$PS1\\[\\e]2;hey\\a\\]\"",'ls'], @core.set_title('hey', ['ls'])
-    assert_equal ['ls'],                                  @core.set_title(nil,   ['ls'])
+  it "should set the title of the tab if one given" do
+    (name = Object.new).expects(:set)
+    (tab = Object.new).stubs(:name).returns(name)
+    @core.set_title('the title', tab)
+  end
+
+  it "should not bother setting the tab title if not one given" do
+    (name = Object.new).expects(:set).never
+    (tab = Object.new).stubs(:name).returns(name)
+    @core.set_title(nil, tab)
   end
 
 end

data/spec/spec_helper.rb

@@ -1,6 +1,7 @@
 require 'rubygems'
 gem 'minitest'
 require 'minitest/autorun'
+require 'mocha'
 require File.expand_path('../../lib/consular/iterm', __FILE__)
 
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: consular-iterm
 version: !ruby/object:Gem::Version
-  version: 1.0.1
+  version: 1.0.2
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2011-11-17 00:00:00.000000000Z
+date: 2011-12-09 00:00:00.000000000Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: consular
-  requirement: &2168686940 !ruby/object:Gem::Requirement
+  requirement: &2152532600 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *2168686940
+  version_requirements: *2152532600
 - !ruby/object:Gem::Dependency
   name: rb-appscript
-  requirement: &2168686220 !ruby/object:Gem::Requirement
+  requirement: &2152532160 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,10 +32,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *2168686220
+  version_requirements: *2152532160
 - !ruby/object:Gem::Dependency
   name: minitest
-  requirement: &2168685680 !ruby/object:Gem::Requirement
+  requirement: &2152531740 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -43,7 +43,18 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *2168685680
+  version_requirements: *2152531740
+- !ruby/object:Gem::Dependency
+  name: mocha
+  requirement: &2152485220 !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: *2152485220
 description: Terminal Automation for ITerm via Consular
 email:
 - mr.arthur.chiu@gmail.com