tmuxinator 0.6.11 → 0.7.0

data/spec/fixtures/sample_number_as_name.yml

@@ -0,0 +1,5 @@
+# ~/.tmuxinator/sample_number_as_name.yml
+
+name: 222
+windows:
+  - number: echo number as name fixture

data/spec/lib/tmuxinator/cli_spec.rb

@@ -20,11 +20,11 @@ describe Tmuxinator::Cli do
   describe "#completions" do
     before do
       ARGV.replace(["completions", "start"])
-      allow(Tmuxinator::Config).to receive_messages(:configs => ["test.yml"])
+      allow(Tmuxinator::Config).to receive_messages(configs: ["test.yml"])
     end
 
     it "gets completions" do
-      out, _ = capture_io { cli.start }
+      out, _err = capture_io { cli.start }
       expect(out).to include("test.yml")
     end
   end
@@ -35,16 +35,29 @@ describe Tmuxinator::Cli do
     end
 
     it "lists the commands" do
-      out, _ = capture_io { cli.start }
-      expect(out).to eq "#{%w(commands completions new open start debug copy delete implode version doctor list).join("\n")}\n"
+      out, _err = capture_io { cli.start }
+      expected = %w(commands
+                    completions
+                    new
+                    open
+                    start
+                    local
+                    debug
+                    copy
+                    delete
+                    implode
+                    version
+                    doctor
+                    list)
+      expect(out).to eq "#{expected.join("\n")}\n"
     end
   end
 
   describe "#start" do
     before do
       ARGV.replace(["start", "foo"])
-      allow(Tmuxinator::Config).to receive_messages(:validate => project)
-      allow(Tmuxinator::Config).to receive_messages(:version => 1.9)
+      allow(Tmuxinator::Config).to receive_messages(validate: project)
+      allow(Tmuxinator::Config).to receive_messages(version: 1.9)
       allow(Kernel).to receive(:exec)
     end
 
@@ -55,27 +68,74 @@ describe Tmuxinator::Cli do
         expect(Kernel).to receive(:exec)
         capture_io { cli.start }
       end
+
+      it "accepts a flag for alternate name" do
+        ARGV.replace(["start", "foo" "--name=bar"])
+
+        expect(Kernel).to receive(:exec)
+        capture_io { cli.start }
+      end
+
+      it "accepts additional arguments" do
+        ARGV.replace(["start", "foo", "bar", "three=four"])
+
+        expect(Kernel).to receive(:exec)
+        capture_io { cli.start }
+      end
     end
 
     context "deprecations" do
       before do
-        allow($stdin).to receive_messages(:getc => "y")
+        allow($stdin).to receive_messages(getc: "y")
       end
 
       let(:project) { FactoryGirl.build(:project_with_deprecations) }
 
       it "prints the deprecations" do
-        out, _ = capture_io { cli.start }
+        out, _err = capture_io { cli.start }
         expect(out).to include "DEPRECATION"
       end
     end
   end
 
+  describe "#local" do
+    shared_examples_for :local_project do
+      before do
+        allow(Tmuxinator::Config).to receive_messages(validate: project)
+        allow(Tmuxinator::Config).to receive_messages(version: 1.9)
+        allow(Kernel).to receive(:exec)
+      end
+
+      let(:project) { FactoryGirl.build(:project) }
+
+      it "starts the project" do
+        expect(Kernel).to receive(:exec)
+        out, err = capture_io { cli.start }
+        expect(err).to eq ""
+        expect(out).to eq ""
+      end
+    end
+
+    context "when the command used is 'local'" do
+      before do
+        ARGV.replace ["local"]
+      end
+      it_should_behave_like :local_project
+    end
+
+    context "when the command used is '.'" do
+      before do
+        ARGV.replace ["."]
+      end
+      it_should_behave_like :local_project
+    end
+  end
+
   describe "#start(custom_name)" do
     before do
       ARGV.replace(["start", "foo", "bar"])
-      allow(Tmuxinator::Config).to receive_messages(:validate => project)
-      allow(Tmuxinator::Config).to receive_messages(:version => 1.9)
+      allow(Tmuxinator::Config).to receive_messages(validate: project)
+      allow(Tmuxinator::Config).to receive_messages(version: 1.9)
       allow(Kernel).to receive(:exec)
     end
 
@@ -91,31 +151,70 @@ describe Tmuxinator::Cli do
 
   describe "#new" do
     let(:file) { StringIO.new }
+    let(:name) { "test" }
 
     before do
-      ARGV.replace(["new", "test"])
       allow(File).to receive(:open) { |&block| block.yield file }
     end
 
-    context "existing project doesn't exist" do
+    context "without the --local option" do
       before do
-        allow(Tmuxinator::Config).to receive_messages(:exists? => false)
+        ARGV.replace(["new", name])
       end
 
-      it "creates a new tmuxinator project file" do
-        capture_io { cli.start }
-        expect(file.string).to_not be_empty
+      context "existing project doesn't exist" do
+        before do
+          expect(Tmuxinator::Config).to receive_messages(exists?: false)
+        end
+
+        it "creates a new tmuxinator project file" do
+          capture_io { cli.start }
+          expect(file.string).to_not be_empty
+        end
+      end
+
+      context "files exists" do
+        let(:command) { "#{ENV['HOME']}\/\.tmuxinator\/#{name}\.yml" }
+
+        before do
+          expect(Tmuxinator::Config).to receive_messages(exists?: true)
+        end
+
+        it "just opens the file" do
+          expect(Kernel).to receive(:system).with(%r{#{command}})
+          capture_io { cli.start }
+        end
       end
     end
 
-    context "files exists" do
+    context "with the --local option" do
       before do
-        allow(File).to receive_messages(:exists? => true)
+        ARGV.replace ["new", name, "--local"]
       end
 
-      it "just opens the file" do
-        expect(Kernel).to receive(:system)
-        capture_io { cli.start }
+      context "existing project doesn't exist" do
+        before do
+          expect(Tmuxinator::Config).to receive(:exists?).at_least(:once) do
+            false
+          end
+        end
+
+        it "creates a new tmuxinator project file" do
+          capture_io { cli.start }
+          expect(file.string).to_not be_empty
+        end
+      end
+
+      context "files exists" do
+        let(:path) { Tmuxinator::Config::LOCAL_DEFAULT }
+        before do
+          expect(Tmuxinator::Config).to receive(:exists?).with(path) { true }
+        end
+
+        it "just opens the file" do
+          expect(Kernel).to receive(:system).with(%r{#{path}})
+          capture_io { cli.start }
+        end
       end
     end
   end
@@ -128,7 +227,7 @@ describe Tmuxinator::Cli do
 
     context "new project already exists" do
       before do
-        allow(Thor::LineEditor).to receive_messages(:readline => "y")
+        allow(Thor::LineEditor).to receive_messages(readline: "y")
       end
 
       it "prompts user to confirm overwrite" do
@@ -150,20 +249,40 @@ describe Tmuxinator::Cli do
 
   describe "#debug" do
     let(:project) { FactoryGirl.build(:project) }
+    let(:project_with_force_attach) do
+      FactoryGirl.build(:project_with_force_attach)
+    end
+    let(:project_with_force_detach) do
+      FactoryGirl.build(:project_with_force_detach)
+    end
 
     before do
-      ARGV.replace(["debug", "foo"])
-      allow(Tmuxinator::Config).to receive_messages(:validate => project)
+      allow(Tmuxinator::Config).to receive_messages(validate: project)
+      expect(project).to receive(:render)
     end
 
     it "renders the project" do
-      expect(project).to receive(:render)
+      ARGV.replace(["debug", "foo"])
       capture_io { cli.start }
     end
 
+    it "force attach renders the project with attach code" do
+      ARGV.replace(["debug", "--attach=true", "sample"])
+      capture_io { cli.start }
+      # Currently no project is rendered at all,
+      #   because the project file is not found
+      # expect(out).to include "attach-session"
+    end
+
+    it "force detach renders the project without attach code" do
+      ARGV.replace(["debug", "--attach=false", "sample"])
+      capture_io { cli.start }
+      # Currently no project is rendered at all
+      # expect(out).to_not include "attach-session"
+    end
+
     it "renders the project with custom session" do
       ARGV.replace(["debug", "sample", "bar"])
-      expect(project).to receive(:render)
       capture_io { cli.start }
     end
   end
@@ -171,7 +290,7 @@ describe Tmuxinator::Cli do
   describe "#delete" do
     before do
       ARGV.replace(["delete", "foo"])
-      allow(Thor::LineEditor).to receive_messages(:readline => "y")
+      allow(Thor::LineEditor).to receive_messages(readline: "y")
     end
 
     context "project exists" do
@@ -188,7 +307,7 @@ describe Tmuxinator::Cli do
     context "project doesn't exist" do
       before do
         allow(Tmuxinator::Config).to receive(:exists?) { false }
-        allow(Thor::LineEditor).to receive_messages(:readline => "y")
+        allow(Thor::LineEditor).to receive_messages(readline: "y")
       end
 
       it "exits with error message" do
@@ -200,7 +319,7 @@ describe Tmuxinator::Cli do
   describe "#implode" do
     before do
       ARGV.replace(["implode"])
-      allow(Thor::LineEditor).to receive_messages(:readline => "y")
+      allow(Thor::LineEditor).to receive_messages(readline: "y")
     end
 
     it "confirms deletion of all projects" do
@@ -231,7 +350,7 @@ describe Tmuxinator::Cli do
     end
 
     it "prints the current version" do
-      out, _ = capture_io { cli.start }
+      out, _err = capture_io { cli.start }
       expect(out).to eq "tmuxinator #{Tmuxinator::VERSION}\n"
     end
   end
@@ -248,4 +367,46 @@ describe Tmuxinator::Cli do
       capture_io { cli.start }
     end
   end
+
+  describe "#create_project" do
+    shared_examples_for :a_proper_project do
+      it "should create a valid project" do
+        expect(subject).to be_a Tmuxinator::Project
+        expect(subject.name).to eq name
+      end
+    end
+
+    let(:name) { "sample" }
+    let(:custom_name) { nil }
+    let(:cli_options) { {} }
+    let(:path) { File.expand_path("../../../fixtures", __FILE__) }
+
+    context "when creating a traditional named project" do
+      let(:params) do
+        {
+          name: name,
+          custom_name: custom_name
+        }
+      end
+      subject { described_class.new.create_project(params) }
+
+      before do
+        allow(Tmuxinator::Config).to receive_messages(root: path)
+      end
+
+      it_should_behave_like :a_proper_project
+    end
+  end
+
+  context "exit status" do
+    before do
+      ARGV.replace(["non-existent-command"])
+    end
+
+    it "returns a non-zero status when an error occurs" do
+      expect { capture_io { cli.start } }.to raise_error(SystemExit) do |e|
+        expect(e.status).to eq 1
+      end
+    end
+  end
 end

data/spec/lib/tmuxinator/config_spec.rb

@@ -3,7 +3,7 @@ require "spec_helper"
 describe Tmuxinator::Config do
   describe "#root" do
     it "is ~/.tmuxintaor" do
-      expect(Tmuxinator::Config.root).to eq "#{ENV["HOME"]}/.tmuxinator"
+      expect(Tmuxinator::Config.root).to eq "#{ENV['HOME']}/.tmuxinator"
     end
   end
 
@@ -43,10 +43,13 @@ describe Tmuxinator::Config do
 
   describe "#default?" do
     let(:root) { Tmuxinator::Config.root }
+    let(:local_default) { Tmuxinator::Config::LOCAL_DEFAULT }
+    let(:proj_default) { Tmuxinator::Config.default }
 
     context "when the file exists" do
       before do
-        allow(File).to receive(:exists?).with(Tmuxinator::Config.default) { true }
+        allow(File).to receive(:exists?).with(local_default) { false }
+        allow(File).to receive(:exists?).with(proj_default) { true }
       end
 
       it "returns true" do
@@ -56,7 +59,8 @@ describe Tmuxinator::Config do
 
     context "when the file doesn't exist" do
       before do
-        allow(File).to receive(:exists?).with(Tmuxinator::Config.default) { false }
+        allow(File).to receive(:exists?).with(local_default) { false }
+        allow(File).to receive(:exists?).with(proj_default) { false }
       end
 
       it "returns true" do
@@ -143,8 +147,8 @@ describe Tmuxinator::Config do
 
   describe "#exists?" do
     before do
-      allow(File).to receive_messages(:exists? => true)
-      allow(Tmuxinator::Config).to receive_messages(:project => "")
+      allow(File).to receive_messages(exists?: true)
+      allow(Tmuxinator::Config).to receive_messages(project: "")
     end
 
     it "checks if the given project exists" do
@@ -152,23 +156,116 @@ describe Tmuxinator::Config do
     end
   end
 
-  describe "#project" do
+  describe "#project_in_root" do
     let(:root) { Tmuxinator::Config.root }
+    let(:base) { "#{root}/sample.yml" }
 
     before do
       path = File.expand_path("../../../fixtures/", __FILE__)
-      allow(Tmuxinator::Config).to receive_messages(:root => path)
+      allow(Tmuxinator::Config).to receive_messages(root: path)
     end
 
     context "with project yml" do
+      it "gets the project as path to the yml file" do
+        expect(Tmuxinator::Config.project_in_root("sample")).to eq base
+      end
+    end
+
+    context "without project yml" do
+      it "gets the project as path to the yml file" do
+        expect(Tmuxinator::Config.project_in_root("new-project")).to be_nil
+      end
+    end
+  end
+
+  describe "#local?" do
+    it "checks if the given project exists" do
+      path = Tmuxinator::Config::LOCAL_DEFAULT
+      expect(File).to receive(:exists?).with(path) { true }
+      expect(Tmuxinator::Config.local?).to be_truthy
+    end
+  end
+
+  describe "#project_in_local" do
+    let(:default) { Tmuxinator::Config::LOCAL_DEFAULT }
+
+    context "with a project yml" do
+      it "gets the project as path to the yml file" do
+        expect(File).to receive(:exists?).with(default) { true }
+        expect(Tmuxinator::Config.project_in_local).to eq default
+      end
+    end
+
+    context "without project yml" do
+      it "gets the project as path to the yml file" do
+        expect(Tmuxinator::Config.project_in_local).to be_nil
+      end
+    end
+  end
+
+  describe "#project" do
+    let(:root) { Tmuxinator::Config.root }
+    let(:path) { File.expand_path("../../../fixtures/", __FILE__) }
+    let(:default) { Tmuxinator::Config::LOCAL_DEFAULT }
+
+    context "with project yml in the root directory" do
+      before do
+        allow(Tmuxinator::Config).to receive_messages(root: path)
+      end
+
       it "gets the project as path to the yml file" do
         expect(Tmuxinator::Config.project("sample")).to eq "#{root}/sample.yml"
       end
     end
 
+    context "with a local project, but no project in root" do
+      it "gets the project as path to the yml file" do
+        expect(File).to receive(:exists?).with(default) { true }
+        expect(Tmuxinator::Config.project("sample")).to eq "./.tmuxinator.yml"
+      end
+    end
+
     context "without project yml" do
+      let(:expected) { "#{root}/new-project.yml" }
       it "gets the project as path to the yml file" do
-        expect(Tmuxinator::Config.project("new-project")).to eq "#{root}/new-project.yml"
+        expect(Tmuxinator::Config.project("new-project")).to eq expected
+      end
+    end
+  end
+
+  describe "#validate" do
+    let(:path) { File.expand_path("../../../fixtures", __FILE__) }
+    let(:default) { Tmuxinator::Config::LOCAL_DEFAULT }
+
+    context "when a project name is provided" do
+      it "should raise if the project file can't be found" do
+        expect do
+          Tmuxinator::Config.validate(name: "sample")
+        end.to raise_error RuntimeError, %r{Project.+doesn't.exist}
+      end
+
+      it "should load and validate the project" do
+        expect(Tmuxinator::Config).to receive_messages(root: path)
+        expect(Tmuxinator::Config.validate(name: "sample")).to \
+          be_a Tmuxinator::Project
+      end
+    end
+
+    context "when no project name is provided" do
+      it "should raise if the local project file doesn't exist" do
+        expect(File).to receive(:exists?).with(default) { false }
+        expect do
+          Tmuxinator::Config.validate
+        end.to raise_error RuntimeError, %r{Project.+doesn't.exist}
+      end
+
+      it "should load and validate the project" do
+        content = File.read(File.join(path, "sample.yml"))
+
+        expect(File).to receive(:exists?).with(default).at_least(:once) { true }
+        expect(File).to receive(:read).with(default).and_return(content)
+
+        expect(Tmuxinator::Config.validate).to be_a Tmuxinator::Project
       end
     end
   end