kumade 0.4.0 → 0.5.0

data/spec/kumade/git_spec.rb

@@ -1,49 +1,26 @@
 require 'spec_helper'
 
-describe Kumade::Git, "#heroku_remote?" do
+describe Kumade::Git, "#heroku_remote?", :with_mock_outputter do
   context "when the environment is a Heroku repository" do
-    let(:environment) { 'staging' }
+    include_context "with Heroku environment"
 
-    before do
-      force_add_heroku_remote(environment)
-      Kumade.configuration.environment = environment
-    end
-
-    after { remove_remote(environment) }
-
-    its(:heroku_remote?) { should == true }
+    it { should be_heroku_remote }
   end
 
   context "when the environment is a Heroku repository managed with heroku-accounts" do
-    let(:another_heroku_environment) { 'another_staging' }
-    let(:another_heroku_url)         { 'git@heroku.work:my-app.git' }
+    include_context "with Heroku-accounts environment"
 
-    before do
-      force_add_heroku_remote(another_heroku_environment)
-      Kumade.configuration.environment = another_heroku_environment
-    end
-
-    after { remove_remote(another_heroku_environment) }
-
-    its(:heroku_remote?) { should == true }
+    it { should be_heroku_remote }
   end
 
   context "when the environment is not a Heroku repository" do
-    let(:not_a_heroku_env) { 'fake_heroku' }
-    let(:not_a_heroku_url) { 'git@github.com:gabebw/kumade.git' }
-
-    before do
-      `git remote add #{not_a_heroku_env} #{not_a_heroku_url}`
-      Kumade.configuration.environment = not_a_heroku_env
-    end
-
-    after { remove_remote(not_a_heroku_env) }
+    include_context "with non-Heroku environment"
 
-    its(:heroku_remote?) { should == false }
+    it { should_not be_heroku_remote }
   end
 end
 
-describe Kumade::Git, ".environments" do
+describe Kumade::Git, ".environments", :with_mock_outputter do
   let(:environment)      { 'staging' }
   let(:not_a_heroku_env) { 'fake_heroku' }
   let(:not_a_heroku_url) { 'git@github.com:gabebw/kumade.git' }
@@ -63,56 +40,190 @@ describe Kumade::Git, ".environments" do
   end
 end
 
-describe Kumade::Git, "#branch_exist?" do
-  let(:command_line) { mock("Cocaine::CommandLine") }
-  let(:branch)       { "branch" }
+describe Kumade::Git, "#push", :with_mock_outputter do
+  let(:branch)       { 'branch' }
+  let(:remote)       { 'my-remote' }
+  let(:command_line) { stub("Kumade::CommandLine instance", :run_or_error => true) }
 
   before do
-    command_line.stubs(:run)
-    Cocaine::CommandLine.expects(:new).with("git show-ref #{branch}").returns(command_line)
+    Kumade::CommandLine.stubs(:new => command_line)
+  end
+
+  it "pushes to the correct remote" do
+    subject.push(branch, remote)
+    Kumade::CommandLine.should have_received(:new).with("git push #{remote} #{branch}")
+    command_line.should have_received(:run_or_error).once
+  end
+
+  it "can force push" do
+    subject.push(branch, remote, true)
+    Kumade::CommandLine.should have_received(:new).with("git push -f #{remote} #{branch}")
+    command_line.should have_received(:run_or_error).once
+  end
+
+  it "prints a success message" do
+    subject.push(branch, remote)
+    Kumade.configuration.outputter.should have_received(:success).with("Pushed #{branch} -> #{remote}")
+  end
+end
+
+describe Kumade::Git, "#create", :with_mock_outputter do
+  let(:branch) { "my-new-branch" }
+  it "creates a branch" do
+    subject.create(branch)
+    system("git show-ref #{branch} > /dev/null").should be_true
   end
 
-  it "returns true when the branch exists" do
-    subject.branch_exist?("branch").should be_true
+  context "when the branch already exists" do
+    before do
+      subject.create(branch)
+    end
 
-    command_line.should have_received(:run)
+    it "does not error" do
+      subject.create(branch)
+      Kumade.configuration.outputter.should have_received(:error).never
+    end
   end
+end
 
-  it "returns false if the branch doesn't exist" do
-    command_line.stubs(:run).raises(Cocaine::ExitStatusError)
+describe Kumade::Git, "#delete", :with_mock_outputter do
+  let(:branch_to_delete)   { 'branch_to_delete' }
+  let(:branch_to_checkout) { 'branch_to_checkout' }
+
+  before do
+    subject.create(branch_to_delete)
+    subject.create(branch_to_checkout)
+  end
 
-    subject.branch_exist?("branch").should be_false
+  it "switches to a branch" do
+    subject.delete(branch_to_delete, branch_to_checkout)
+    subject.current_branch.should == branch_to_checkout
+  end
 
-    command_line.should have_received(:run)
+  it "deletes a branch" do
+    subject.delete(branch_to_delete, branch_to_checkout)
+    `git show-ref #{branch_to_delete}`.strip.should be_empty
   end
 end
 
-describe Kumade::Git, "#dirty?" do
-  context "when dirty" do
-    let(:failing_command_line) { mock("CommandLine instance") }
+describe Kumade::Git, "#add_and_commit_all_assets_in", :with_mock_outputter do
+  let(:directory) { 'assets' }
+
+  before do
+    Dir.mkdir(directory)
+    Dir.chdir(directory) do
+      File.open('new-file', 'w') do |f|
+        f.write('some content')
+      end
+    end
+  end
+
+  it "switches to the deploy branch" do
+    subject.add_and_commit_all_assets_in(directory)
+    subject.current_branch.should == Kumade::Heroku::DEPLOY_BRANCH
+  end
 
+  it "uses a bland commit message" do
+    subject.add_and_commit_all_assets_in(directory)
+    `git log -n1 --pretty=format:%s`.should == 'Compiled assets.'
+  end
+
+  it "commits everything in the dir" do
+    subject.add_and_commit_all_assets_in(directory)
+    subject.should_not be_dirty
+  end
+
+  it "prints a success message" do
+    subject.add_and_commit_all_assets_in(directory)
+    Kumade.configuration.outputter.should have_received(:success).with('Added and committed all assets')
+  end
+
+  context "if the command fails" do
+    let(:command_line) { mock('CommandLine', :run_or_error => nil) }
+    before do
+      Kumade::CommandLine.stubs(:new => command_line)
+    end
+
+    it "prints an error message if something goes wrong" do
+      subject.add_and_commit_all_assets_in(directory)
+      command_line.should have_received(:run_or_error).once
+    end
+  end
+end
+
+describe Kumade::Git, "#current_branch", :with_mock_outputter do
+  it "returns the current branch" do
+    subject.current_branch.should == 'master'
+    `git checkout -b new-branch 2>/dev/null`
+    subject.current_branch.should == 'new-branch'
+  end
+end
+
+describe Kumade::Git, "#remote_exists?", :with_mock_outputter do
+  context "when pretending" do
+    before { Kumade.configuration.pretending = true }
+    it "returns true no matter what" do
+      subject.remote_exists?('not-a-remote').should be_true
+    end
+  end
+
+  context "when not pretending" do
+    let(:good_remote) { 'good-remote' }
+    let(:bad_remote)  { 'bad-remote' }
     before do
-      failing_command_line.stubs(:run).raises(Cocaine::ExitStatusError)
+      Kumade.configuration.pretending = false
+      force_add_heroku_remote(good_remote)
+    end
 
-      Cocaine::CommandLine.expects(:new).
-        with("git diff --exit-code").
-        returns(failing_command_line)
+    it "returns true if the remote exists" do
+      subject.remote_exists?(good_remote).should be_true
     end
 
-    it "returns true" do
-      subject.dirty?.should == true
+    it "returns false if the remote does not exist" do
+      subject.remote_exists?(bad_remote).should be_false
     end
   end
+end
+
+describe Kumade::Git, "#dirty?", :with_mock_outputter do
+  context "when dirty" do
+    before { dirty_the_repo }
+
+    it { should be_dirty }
+  end
 
   context "when clean" do
+    it { should_not be_dirty }
+  end
+end
+
+
+describe Kumade::Git, "#ensure_clean_git", :with_mock_outputter do
+  context "when pretending" do
     before do
-      Cocaine::CommandLine.expects(:new).
-        with("git diff --exit-code").
-        returns(stub("Successful CommandLine", :run => true))
+      Kumade.configuration.pretending = true
+      dirty_the_repo
     end
 
-    it "returns false" do
-      subject.dirty?.should == false
+    it "prints a success message" do
+      subject.ensure_clean_git
+      Kumade.configuration.outputter.should have_received(:success).with("Git repo is clean")
+    end
+  end
+
+  context "when repo is clean" do
+    it "prints a success message" do
+      subject.ensure_clean_git
+      Kumade.configuration.outputter.should have_received(:success).with("Git repo is clean")
+    end
+  end
+
+  context "when repo is dirty" do
+    before { dirty_the_repo }
+
+    it "prints an error message" do
+      subject.ensure_clean_git
+      Kumade.configuration.outputter.should have_received(:error).with("Cannot deploy: repo is not clean.")
     end
   end
 end

data/spec/kumade/heroku_spec.rb

@@ -1,12 +1,12 @@
 require 'spec_helper'
 
-describe Kumade::Heroku, "DEPLOY_BRANCH" do
+describe Kumade::Heroku, "DEPLOY_BRANCH", :with_mock_outputter do
   subject { Kumade::Heroku::DEPLOY_BRANCH }
 
   it { should == "deploy" }
 end
 
-describe Kumade::Heroku, "#sync" do
+describe Kumade::Heroku, "#sync", :with_mock_outputter do
   let(:environment) { 'staging' }
 
   before do
@@ -23,11 +23,10 @@ describe Kumade::Heroku, "#sync" do
   end
 end
 
-describe Kumade::Heroku, "#migrate_database" do
+describe Kumade::Heroku, "#migrate_database", :with_mock_outputter do
   let(:environment) { 'staging' }
 
   before do
-    STDOUT.stubs(:puts)
     subject.stubs(:heroku)
     force_add_heroku_remote(environment)
   end
@@ -40,40 +39,38 @@ describe Kumade::Heroku, "#migrate_database" do
 
   context "when pretending" do
     before do
-      STDOUT.stubs(:puts)
       Kumade.configuration.pretending = true
     end
 
     it "does not run the command" do
       subject.migrate_database
 
-      subject.should_not have_received(:heroku)
+      subject.should have_received(:heroku).never
     end
 
     it "prints a message" do
       subject.migrate_database
 
-      STDOUT.should have_received(:puts).with(regexp_matches(/Migrated #{environment}/))
+      Kumade.configuration.outputter.should have_received(:success).with(regexp_matches(/Migrated #{environment}/))
     end
   end
 end
 
-describe Kumade::Heroku, "#heroku" do
-  let(:command_line_instance) { stub("Cocaine::CommandLine instance", :run => true) }
+describe Kumade::Heroku, "#heroku", :with_mock_outputter do
+  let(:command_instance) { stub("Kumade::CommandLine instance", :run_or_error => true) }
 
   before do
-    STDOUT.stubs(:puts)
+    Kumade::CommandLine.stubs(:new => command_instance)
   end
 
   context "when on Cedar" do
     include_context "when on Cedar"
 
     it "runs commands with `run`" do
-      Cocaine::CommandLine.expects(:new).
-        with(regexp_matches(/bundle exec heroku run/)).
-        returns(command_line_instance)
-
       subject.heroku("rake")
+
+      Kumade::CommandLine.should have_received(:new).with(regexp_matches(/bundle exec heroku run rake/)).once
+      command_instance.should have_received(:run_or_error).once
     end
   end
 
@@ -81,16 +78,15 @@ describe Kumade::Heroku, "#heroku" do
     include_context "when not on Cedar"
 
     it "runs commands without `run`" do
-      Cocaine::CommandLine.expects(:new).
-        with(regexp_matches(/bundle exec heroku rake/)).
-        returns(command_line_instance)
-
       subject.heroku("rake")
+
+      Kumade::CommandLine.should have_received(:new).with(regexp_matches(/bundle exec heroku rake/)).once
+      command_instance.should have_received(:run_or_error).once
     end
   end
 end
 
-describe Kumade::Heroku, "#cedar?" do
+describe Kumade::Heroku, "#cedar?", :with_mock_outputter do
   context "when on Cedar" do
     include_context "when on Cedar"
 
@@ -108,14 +104,12 @@ describe Kumade::Heroku, "#cedar?" do
   end
 end
 
-describe Kumade::Heroku, "#delete_deploy_branch" do
-  before { STDOUT.stubs(:puts) }
+describe Kumade::Heroku, "#delete_deploy_branch", :with_mock_outputter do
+  before { subject.git.stubs(:delete) }
 
   it "deletes the deploy branch" do
-    Cocaine::CommandLine.expects(:new).
-      with("git checkout master && git branch -D deploy").
-      returns(stub(:run => true))
-
     subject.delete_deploy_branch
+
+    subject.git.should have_received(:delete).with(Kumade::Heroku::DEPLOY_BRANCH, 'master').once
   end
 end

data/spec/kumade/outputter_spec.rb

@@ -0,0 +1,50 @@
+require 'spec_helper'
+
+describe Kumade::Outputter, "#success" do
+  before { STDOUT.stubs(:puts) }
+
+  it "prints a message to STDOUT" do
+    subject.success("woo hoo")
+    STDOUT.should have_received(:puts).with(regexp_matches(/==> woo hoo/))
+  end
+end
+
+describe Kumade::Outputter, "#error" do
+  before { STDOUT.stubs(:puts) }
+
+  it "raises a DeploymentError with the given message" do
+    lambda { subject.error("uh oh") }.should raise_error(Kumade::DeploymentError, "uh oh")
+  end
+
+  it "prints a message to STDOUT" do
+    subject.error("uh oh") rescue nil
+    STDOUT.should have_received(:puts).with(regexp_matches(/==> ! uh oh/))
+  end
+end
+
+describe Kumade::Outputter, "#info" do
+  before { STDOUT.stubs(:puts) }
+
+  it "prints a message to STDOUT" do
+    subject.info("the more you know")
+    STDOUT.should have_received(:puts).with(regexp_matches(/==> the more you know/))
+  end
+end
+
+describe Kumade::Outputter, "#say_command" do
+  before { STDOUT.stubs(:puts) }
+
+  it "prints a formatted message to STDOUT" do
+    subject.say_command("git checkout master")
+    STDOUT.should have_received(:puts).with(" " * 8 + "git checkout master")
+  end
+end
+
+describe Kumade::Outputter, "#info" do
+  before { STDOUT.stubs(:puts) }
+
+  it "prints a message to STDOUT" do
+    subject.info("the more you know")
+    STDOUT.should have_received(:puts).with(regexp_matches(/==> the more you know/))
+  end
+end

data/spec/kumade/packager_list_spec.rb

@@ -0,0 +1,31 @@
+require 'spec_helper'
+
+describe Kumade::PackagerList, "detecting packages", :with_mock_outputter do
+  it "returns an array containing the Jammit packager if Jammit is installed" do
+    Kumade::JammitPackager.stubs(:installed? => true)
+    Kumade::MorePackager.stubs(:installed? => false)
+
+    Kumade::PackagerList.new.to_a.should == [Kumade::JammitPackager]
+  end
+
+  it "returns an array containing the More packager if More is installed" do
+    Kumade::JammitPackager.stubs(:installed? => false)
+    Kumade::MorePackager.stubs(:installed? => true)
+
+    Kumade::PackagerList.new.to_a.should == [Kumade::MorePackager]
+  end
+
+  it "returns multiple packagers if they are installed" do
+    Kumade::JammitPackager.stubs(:installed? => true)
+    Kumade::MorePackager.stubs(:installed? => true)
+
+    Kumade::PackagerList.new.to_a.should =~ [Kumade::JammitPackager, Kumade::MorePackager]
+  end
+
+  it "returns an array containing the no-op packager if no other packagers are found" do
+    Kumade::JammitPackager.stubs(:installed? => false)
+    Kumade::MorePackager.stubs(:installed? => false)
+
+    Kumade::PackagerList.new.to_a.should == [Kumade::NoopPackager]
+  end
+end