tmuxinator 0.9.0 → 0.10.0

data/spec/lib/tmuxinator/doctor_spec.rb

@@ -0,0 +1,69 @@
+require "spec_helper"
+
+describe Tmuxinator::Doctor do
+  describe ".installed?" do
+    context "tmux is installed" do
+      before do
+        allow(Kernel).to receive(:system) { true }
+      end
+
+      it "returns true" do
+        expect(Tmuxinator::Doctor.installed?).to be_truthy
+      end
+    end
+
+    context "tmux is not installed" do
+      before do
+        allow(Kernel).to receive(:system) { false }
+      end
+
+      it "returns false" do
+        expect(Tmuxinator::Doctor.installed?).to be_falsey
+      end
+    end
+  end
+
+  describe ".editor?" do
+    context "$EDITOR is set" do
+      before do
+        allow(ENV).to receive(:[]).with("EDITOR") { "vim" }
+      end
+
+      it "returns true" do
+        expect(Tmuxinator::Doctor.editor?).to be_truthy
+      end
+    end
+
+    context "$EDITOR is not set" do
+      before do
+        allow(ENV).to receive(:[]).with("EDITOR") { nil }
+      end
+
+      it "returns false" do
+        expect(Tmuxinator::Doctor.editor?).to be_falsey
+      end
+    end
+  end
+
+  describe ".shell?" do
+    context "$SHELL is set" do
+      before do
+        allow(ENV).to receive(:[]).with("SHELL") { "vim" }
+      end
+
+      it "returns true" do
+        expect(Tmuxinator::Doctor.shell?).to be_truthy
+      end
+    end
+
+    context "$SHELL is not set" do
+      before do
+        allow(ENV).to receive(:[]).with("SHELL") { nil }
+      end
+
+      it "returns false" do
+        expect(Tmuxinator::Doctor.shell?).to be_falsey
+      end
+    end
+  end
+end

data/spec/lib/tmuxinator/hooks/project_spec.rb

@@ -0,0 +1,63 @@
+require "spec_helper"
+
+shared_examples_for "a project hook" do
+  let(:project) { FactoryGirl.build(:project) }
+
+  it "calls Hooks.commands_from" do
+    expect(Tmuxinator::Hooks).to receive(:commands_from).
+      with(kind_of(Tmuxinator::Project), hook_name).once
+    project.send("hook_#{hook_name}")
+  end
+
+  context "hook value is string" do
+    before { project.yaml[hook_name] = "echo 'on hook'" }
+
+    it "returns the string" do
+      expect(project.send("hook_#{hook_name}")).to eq("echo 'on hook'")
+    end
+  end
+
+  context "hook value is Array" do
+    before do
+      project.yaml[hook_name] = [
+        "echo 'on hook'",
+        "echo 'another command here'"
+      ]
+    end
+
+    it "joins array using ;" do
+      expect(project.send("hook_#{hook_name}")).
+        to eq("echo 'on hook'; echo 'another command here'")
+    end
+  end
+end
+
+describe Tmuxinator::Hooks::Project do
+  let(:project) { FactoryGirl.build(:project) }
+
+  describe "#hook_on_project_start" do
+    it_should_behave_like "a project hook" do
+      let(:hook_name) { "on_project_start" }
+    end
+  end
+  describe "#hook_on_project_first_start" do
+    it_should_behave_like "a project hook" do
+      let(:hook_name) { "on_project_first_start" }
+    end
+  end
+  describe "#hook_on_project_restart" do
+    it_should_behave_like "a project hook" do
+      let(:hook_name) { "on_project_restart" }
+    end
+  end
+  describe "#hook_on_project_exit" do
+    it_should_behave_like "a project hook" do
+      let(:hook_name) { "on_project_exit" }
+    end
+  end
+  describe "#hook_on_project_stop" do
+    it_should_behave_like "a project hook" do
+      let(:hook_name) { "on_project_stop" }
+    end
+  end
+end

data/spec/lib/tmuxinator/hooks_spec.rb

@@ -0,0 +1,31 @@
+require "spec_helper"
+
+describe Tmuxinator::Hooks do
+  describe "#commands_from" do
+    let(:project) { FactoryGirl.build(:project) }
+    let(:hook_name) { "generic_hook" }
+
+    context "config value is string" do
+      before { project.yaml[hook_name] = "echo 'on hook'" }
+
+      it "returns the string" do
+        expect(subject.commands_from(project, hook_name)).
+          to eq("echo 'on hook'")
+      end
+    end
+
+    context "config value is Array" do
+      before do
+        project.yaml[hook_name] = [
+          "echo 'on hook'",
+          "echo 'another command here'"
+        ]
+      end
+
+      it "joins array using ;" do
+        expect(subject.commands_from(project, hook_name)).
+          to eq("echo 'on hook'; echo 'another command here'")
+      end
+    end
+  end
+end

data/spec/lib/tmuxinator/pane_spec.rb

@@ -1,7 +1,30 @@
 require "spec_helper"
 
 describe Tmuxinator::Pane do
+  let(:klass) { described_class }
+  let(:instance) { klass.new(index, project, window, *commands) }
+  # let(:index) { "vim" }
+  # let(:project) { 0 }
+  # let(:tab) { nil }
+  # let(:commands) { nil }
+  let(:index) { 0 }
+  let(:project) { double }
+  let(:window) { double }
+  let(:commands) { ["vim", "bash"] }
+
+  before do
+    allow(project).to receive(:name).and_return "foo"
+    allow(project).to receive(:base_index).and_return 0
+
+    allow(window).to receive(:project).and_return project
+    allow(window).to receive(:index).and_return 0
+  end
+
+  subject { instance }
+
   it "creates an instance" do
-    expect(Tmuxinator::Pane.new("vim", 0, nil, nil)).to be_a(Tmuxinator::Pane)
+    expect(subject).to be_a(Tmuxinator::Pane)
   end
+
+  it { expect(subject.tmux_window_and_pane_target).to eql "foo:0.0" }
 end

data/spec/lib/tmuxinator/project_spec.rb

@@ -28,6 +28,10 @@ describe Tmuxinator::Project do
     FactoryGirl.build(:nameless_window_project)
   end
 
+  it "should include Hooks" do
+    expect(project).to be_kind_of(Tmuxinator::Hooks::Project)
+  end
+
   describe "#initialize" do
     context "valid yaml" do
       it "creates an instance" do
@@ -166,14 +170,34 @@ describe Tmuxinator::Project do
         rendered = project_with_literals_as_window_name
         expect(rendered.windows.map(&:name)).to match_array(
           %w(222 222333 111222333444555666777 222.3 4e5 4E5
-             true false nil // /sample/))
+             true false nil // /sample/)
+        )
       end
     end
   end
 
   describe "#pre_window" do
-    it "gets the pre_window command" do
-      expect(project.pre_window).to eq "rbenv shell 2.0.0-p247"
+    subject(:pre_window) { project.pre_window }
+
+    context "pre_window in yaml is string" do
+      before { project.yaml["pre_window"] = "mysql.server start" }
+
+      it "returns the string" do
+        expect(pre_window).to eq("mysql.server start")
+      end
+    end
+
+    context "pre_window in yaml is Array" do
+      before do
+        project.yaml["pre_window"] = [
+          "mysql.server start",
+          "memcached -d"
+        ]
+      end
+
+      it "joins array using ;" do
+        expect(pre_window).to eq("mysql.server start; memcached -d")
+      end
     end
 
     context "with deprecations" do
@@ -268,42 +292,53 @@ describe Tmuxinator::Project do
   end
 
   describe "#get_pane_base_index" do
-    it "extracts the pane_base_index from tmux_options" do
-      allow(project).to \
-        receive_messages(show_tmux_options: tmux_config(pane_base_index: 3))
+    it "extracts pane-base-index from the global tmux window options" do
+      allow_any_instance_of(Kernel).to receive(:`).
+        with(Regexp.new("tmux.+ show-window-option -g pane-base-index")).
+        and_return("pane-base-index 3\n")
 
       expect(project.get_pane_base_index).to eq("3")
     end
   end
 
   describe "#get_base_index" do
-    it "extracts the base index from options" do
-      allow(project).to \
-        receive_messages(show_tmux_options: tmux_config(base_index: 1))
+    it "extracts base-index from the global tmux options" do
+      allow_any_instance_of(Kernel).to receive(:`).
+        with(Regexp.new("tmux.+ show-option -g base-index")).
+        and_return("base-index 1\n")
 
       expect(project.get_base_index).to eq("1")
     end
   end
 
   describe "#base_index" do
-    context "pane base index present" do
+    context "when pane_base_index is 1 and base_index is unset" do
       before do
         allow(project).to receive_messages(get_pane_base_index: "1")
+        allow(project).to receive_messages(get_base_index: nil)
+      end
+
+      it "gets the tmux default of 0" do
+        expect(project.base_index).to eq(0)
+      end
+    end
+
+    context "base index present" do
+      before do
         allow(project).to receive_messages(get_base_index: "1")
       end
 
-      it "gets the pane base index" do
+      it "gets the base index" do
         expect(project.base_index).to eq 1
       end
     end
 
-    context "pane base index no present" do
+    context "base index not present" do
       before do
-        allow(project).to receive_messages(get_pane_base_index: nil)
-        allow(project).to receive_messages(get_base_index: "0")
+        allow(project).to receive_messages(get_base_index: nil)
       end
 
-      it "gets the base index" do
+      it "defaults to 0" do
         expect(project.base_index).to eq 0
       end
     end
@@ -314,7 +349,7 @@ describe Tmuxinator::Project do
       it "gets the startup window from project config" do
         project.yaml["startup_window"] = "logs"
 
-        expect(project.startup_window).to eq("logs")
+        expect(project.startup_window).to eq("sample:logs")
       end
     end
 
@@ -322,7 +357,7 @@ describe Tmuxinator::Project do
       it "returns base index instead" do
         allow(project).to receive_messages(base_index: 8)
 
-        expect(project.startup_window).to eq 8
+        expect(project.startup_window).to eq("sample:8")
       end
     end
   end
@@ -332,7 +367,7 @@ describe Tmuxinator::Project do
       it "get the startup pane index from project config" do
         project.yaml["startup_pane"] = 1
 
-        expect(project.startup_pane).to eq(1)
+        expect(project.startup_pane).to eq("sample:0.1")
       end
     end
 
@@ -340,7 +375,7 @@ describe Tmuxinator::Project do
       it "returns the base pane instead" do
         allow(project).to receive_messages(pane_base_index: 4)
 
-        expect(project.startup_pane).to eq(4)
+        expect(project.startup_pane).to eq("sample:0.4")
       end
     end
   end
@@ -502,7 +537,7 @@ describe Tmuxinator::Project do
     let(:window) { project.windows.first.name }
 
     context "when first window has a name" do
-      it "returns command to start a new detatched session" do
+      it "returns command to start a new detached session" do
         expect(project.tmux_new_session_command).to eq command
       end
     end
@@ -511,7 +546,7 @@ describe Tmuxinator::Project do
       let(:project) { nameless_window_project }
       let(:command) { "#{project.tmux} new-session -d -s #{project.name} " }
 
-      it "returns command to for new detatched session without a window name" do
+      it "returns command to for new detached session without a window name" do
         expect(project.tmux_new_session_command).to eq command
       end
     end
@@ -523,7 +558,7 @@ describe Tmuxinator::Project do
     let(:session) { project.name }
 
     context "when first window has a name" do
-      it "returns command to start a new detatched session" do
+      it "returns command to start a new detached session" do
         expect(project.tmux_kill_session_command).to eq command
       end
     end
@@ -541,7 +576,7 @@ describe Tmuxinator::Project do
       expect(File).to receive(:read).with(path) { bad_yaml }
       expect do
         described_class.load(path, options)
-      end.to raise_error RuntimeError, %r{Failed.to.parse.config.file}
+      end.to raise_error RuntimeError, /\AFailed to parse config file: .+\z/
     end
 
     it "should return an instance of the class if the file loads" do

data/spec/lib/tmuxinator/window_spec.rb

@@ -99,11 +99,13 @@ describe Tmuxinator::Window do
           project: project, tab: window
         )
 
-        expect(window.panes).to match([
-          a_pane.with(index: 0).and_commands("vim"),
-          a_pane.with(index: 1).and_commands("ls"),
-          a_pane.with(index: 2).and_commands("top")
-        ])
+        expect(window.panes).to match(
+          [
+            a_pane.with(index: 0).and_commands("vim"),
+            a_pane.with(index: 1).and_commands("ls"),
+            a_pane.with(index: 2).and_commands("top")
+          ]
+        )
       end
     end
 
@@ -341,7 +343,7 @@ describe Tmuxinator::Window do
       allow(Tmuxinator::Config).to receive(:default_path_option) { path_option }
     end
 
-    it "contstructs window command with path, target, and name options" do
+    it "constructs window command with path, target, and name options" do
       expect(window.tmux_new_window_command).to eq full_command
     end
 

data/spec/spec_helper.rb

@@ -1,6 +1,7 @@
 require "coveralls"
-require "simplecov"
 require "pry"
+require "simplecov"
+require "xdg"
 
 formatters = [
   SimpleCov::Formatter::HTMLFormatter,

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: tmuxinator
 version: !ruby/object:Gem::Version
-  version: 0.9.0
+  version: 0.10.0
 platform: ruby
 authors:
 - Allen Bargi
@@ -9,8 +9,22 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-10-08 00:00:00.000000000 Z
+date: 2017-09-19 00:00:00.000000000 Z
 dependencies:
+- !ruby/object:Gem::Dependency
+  name: erubis
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.6'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.6'
 - !ruby/object:Gem::Dependency
   name: thor
   requirement: !ruby/object:Gem::Requirement
@@ -32,19 +46,53 @@ dependencies:
       - !ruby/object:Gem::Version
         version: 0.15.0
 - !ruby/object:Gem::Dependency
-  name: erubis
+  name: xdg
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.6'
+        version: '2.2'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 2.2.3
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.6'
+        version: '2.2'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 2.2.3
+- !ruby/object:Gem::Dependency
+  name: activesupport
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: 5.0.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: 5.0.0
+- !ruby/object:Gem::Dependency
+  name: awesome_print
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.2'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.2'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -59,6 +107,104 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.3'
+- !ruby/object:Gem::Dependency
+  name: coveralls
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.7'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.7'
+- !ruby/object:Gem::Dependency
+  name: factory_girl
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '4.5'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '4.5'
+- !ruby/object:Gem::Dependency
+  name: pry
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.10'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.10'
+- !ruby/object:Gem::Dependency
+  name: rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '10.4'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '10.4'
+- !ruby/object:Gem::Dependency
+  name: rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.3'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.3'
+- !ruby/object:Gem::Dependency
+  name: rubocop
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.46.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.46.0
+- !ruby/object:Gem::Dependency
+  name: simplecov
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.11.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.11.0
 description: Create and manage complex tmux sessions easily.
 email:
 - allen.bargi@gmail.com
@@ -75,11 +221,15 @@ files:
 - completion/tmuxinator.zsh
 - lib/tmuxinator.rb
 - lib/tmuxinator/assets/sample.yml
+- lib/tmuxinator/assets/template-stop.erb
 - lib/tmuxinator/assets/template.erb
 - lib/tmuxinator/assets/wemux_template.erb
 - lib/tmuxinator/cli.rb
 - lib/tmuxinator/config.rb
 - lib/tmuxinator/deprecations.rb
+- lib/tmuxinator/doctor.rb
+- lib/tmuxinator/hooks.rb
+- lib/tmuxinator/hooks/project.rb
 - lib/tmuxinator/pane.rb
 - lib/tmuxinator/project.rb
 - lib/tmuxinator/util.rb
@@ -87,7 +237,12 @@ files:
 - lib/tmuxinator/wemux_support.rb
 - lib/tmuxinator/window.rb
 - spec/factories/projects.rb
+- spec/fixtures/TMUXINATOR_CONFIG/TMUXINATOR_CONFIG.yml
 - spec/fixtures/detach.yml
+- spec/fixtures/dot-tmuxinator/both.yml
+- spec/fixtures/dot-tmuxinator/dup/local-dup.yml
+- spec/fixtures/dot-tmuxinator/home.yml
+- spec/fixtures/dot-tmuxinator/local-dup.yml
 - spec/fixtures/nameless_window.yml
 - spec/fixtures/noname.yml
 - spec/fixtures/noroot.yml
@@ -97,8 +252,13 @@ files:
 - spec/fixtures/sample_literals_as_window_name.yml
 - spec/fixtures/sample_number_as_name.yml
 - spec/fixtures/sample_wemux.yml
+- spec/fixtures/xdg-tmuxinator/both.yml
+- spec/fixtures/xdg-tmuxinator/xdg.yml
 - spec/lib/tmuxinator/cli_spec.rb
 - spec/lib/tmuxinator/config_spec.rb
+- spec/lib/tmuxinator/doctor_spec.rb
+- spec/lib/tmuxinator/hooks/project_spec.rb
+- spec/lib/tmuxinator/hooks_spec.rb
 - spec/lib/tmuxinator/pane_spec.rb
 - spec/lib/tmuxinator/project_spec.rb
 - spec/lib/tmuxinator/util_spec.rb
@@ -123,7 +283,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 1.9.3
+      version: 2.2.7
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
@@ -131,13 +291,18 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: 1.8.23
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.6
+rubygems_version: 2.6.13
 signing_key: 
 specification_version: 4
 summary: Create and manage complex tmux sessions easily.
 test_files:
 - spec/factories/projects.rb
+- spec/fixtures/TMUXINATOR_CONFIG/TMUXINATOR_CONFIG.yml
 - spec/fixtures/detach.yml
+- spec/fixtures/dot-tmuxinator/both.yml
+- spec/fixtures/dot-tmuxinator/dup/local-dup.yml
+- spec/fixtures/dot-tmuxinator/home.yml
+- spec/fixtures/dot-tmuxinator/local-dup.yml
 - spec/fixtures/nameless_window.yml
 - spec/fixtures/noname.yml
 - spec/fixtures/noroot.yml
@@ -147,8 +312,13 @@ test_files:
 - spec/fixtures/sample_literals_as_window_name.yml
 - spec/fixtures/sample_number_as_name.yml
 - spec/fixtures/sample_wemux.yml
+- spec/fixtures/xdg-tmuxinator/both.yml
+- spec/fixtures/xdg-tmuxinator/xdg.yml
 - spec/lib/tmuxinator/cli_spec.rb
 - spec/lib/tmuxinator/config_spec.rb
+- spec/lib/tmuxinator/doctor_spec.rb
+- spec/lib/tmuxinator/hooks/project_spec.rb
+- spec/lib/tmuxinator/hooks_spec.rb
 - spec/lib/tmuxinator/pane_spec.rb
 - spec/lib/tmuxinator/project_spec.rb
 - spec/lib/tmuxinator/util_spec.rb