kumade 0.3.0 → 0.4.0

data/spec/kumade/git_spec.rb

@@ -1,33 +1,46 @@
 require 'spec_helper'
 
 describe Kumade::Git, "#heroku_remote?" do
-  let(:environment)                { 'staging' }
-  let(:another_heroku_environment) { 'another_staging' }
-  let(:not_a_heroku_env)           { 'fake_heroku' }
-  let(:not_a_heroku_url)           { 'git@github.com:gabebw/kumade.git' }
-  let(:another_heroku_url)         { 'git@heroku.work:my-app.git' }
+  context "when the environment is a Heroku repository" do
+    let(:environment) { 'staging' }
 
-  before do
-    force_add_heroku_remote(environment)
-    `git remote add #{not_a_heroku_env} #{not_a_heroku_url}`
-    `git remote add #{another_heroku_environment} #{another_heroku_url}`
-  end
+    before do
+      force_add_heroku_remote(environment)
+      Kumade.configuration.environment = environment
+    end
 
-  after do
-    remove_remote(environment)
-    remove_remote(not_a_heroku_env)
-    remove_remote(another_heroku_environment)
-  end
+    after { remove_remote(environment) }
 
-  it "returns true when the remote is a heroku repository" do
-    Kumade::Git.new(false, environment).heroku_remote?.should be_true
-    Kumade::Git.new(false, another_heroku_environment).heroku_remote?.should be_true
+    its(:heroku_remote?) { should == true }
   end
 
-  it "returns false when the remote is not a heroku repository" do
-    Kumade::Git.new(false, 'kumade').heroku_remote?.should be_false
+  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' }
+
+    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 }
   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) }
+
+    its(:heroku_remote?) { should == false }
+  end
 end
 
 describe Kumade::Git, ".environments" do
@@ -45,7 +58,61 @@ describe Kumade::Git, ".environments" do
     remove_remote(not_a_heroku_env)
   end
 
-  it "returns all environments" do
+  it "returns all Heroku environments" do
     Kumade::Git.environments.should == ["staging"]
   end
 end
+
+describe Kumade::Git, "#branch_exist?" do
+  let(:command_line) { mock("Cocaine::CommandLine") }
+  let(:branch)       { "branch" }
+
+  before do
+    command_line.stubs(:run)
+    Cocaine::CommandLine.expects(:new).with("git show-ref #{branch}").returns(command_line)
+  end
+
+  it "returns true when the branch exists" do
+    subject.branch_exist?("branch").should be_true
+
+    command_line.should have_received(:run)
+  end
+
+  it "returns false if the branch doesn't exist" do
+    command_line.stubs(:run).raises(Cocaine::ExitStatusError)
+
+    subject.branch_exist?("branch").should be_false
+
+    command_line.should have_received(:run)
+  end
+end
+
+describe Kumade::Git, "#dirty?" do
+  context "when dirty" do
+    let(:failing_command_line) { mock("CommandLine instance") }
+
+    before do
+      failing_command_line.stubs(:run).raises(Cocaine::ExitStatusError)
+
+      Cocaine::CommandLine.expects(:new).
+        with("git diff --exit-code").
+        returns(failing_command_line)
+    end
+
+    it "returns true" do
+      subject.dirty?.should == true
+    end
+  end
+
+  context "when clean" do
+    before do
+      Cocaine::CommandLine.expects(:new).
+        with("git diff --exit-code").
+        returns(stub("Successful CommandLine", :run => true))
+    end
+
+    it "returns false" do
+      subject.dirty?.should == false
+    end
+  end
+end

data/spec/kumade/heroku_spec.rb

@@ -0,0 +1,121 @@
+require 'spec_helper'
+
+describe Kumade::Heroku, "DEPLOY_BRANCH" do
+  subject { Kumade::Heroku::DEPLOY_BRANCH }
+
+  it { should == "deploy" }
+end
+
+describe Kumade::Heroku, "#sync" do
+  let(:environment) { 'staging' }
+
+  before do
+    force_add_heroku_remote(environment)
+    subject.git.stubs(:create)
+    subject.git.stubs(:push)
+  end
+
+  it "creates and pushes the deploy branch" do
+    subject.sync
+
+    subject.git.should have_received(:create).with("deploy")
+    subject.git.should have_received(:push).with("deploy:master", environment, true)
+  end
+end
+
+describe Kumade::Heroku, "#migrate_database" do
+  let(:environment) { 'staging' }
+
+  before do
+    STDOUT.stubs(:puts)
+    subject.stubs(:heroku)
+    force_add_heroku_remote(environment)
+  end
+
+  it "runs db:migrate with the correct app" do
+    subject.migrate_database
+
+    subject.should have_received(:heroku).with("rake db:migrate")
+  end
+
+  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)
+    end
+
+    it "prints a message" do
+      subject.migrate_database
+
+      STDOUT.should have_received(:puts).with(regexp_matches(/Migrated #{environment}/))
+    end
+  end
+end
+
+describe Kumade::Heroku, "#heroku" do
+  let(:command_line_instance) { stub("Cocaine::CommandLine instance", :run => true) }
+
+  before do
+    STDOUT.stubs(:puts)
+  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")
+    end
+  end
+
+  context "when not on Cedar" 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")
+    end
+  end
+end
+
+describe Kumade::Heroku, "#cedar?" do
+  context "when on Cedar" do
+    include_context "when on Cedar"
+
+    it "returns true" do
+      subject.cedar?.should == true
+    end
+  end
+
+  context "when not on Cedar" do
+    include_context "when not on Cedar"
+
+    it "returns false" do
+      subject.cedar?.should == false
+    end
+  end
+end
+
+describe Kumade::Heroku, "#delete_deploy_branch" do
+  before { STDOUT.stubs(:puts) }
+
+  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
+  end
+end

data/spec/kumade/packager_spec.rb

@@ -0,0 +1,318 @@
+require 'spec_helper'
+
+require 'jammit'
+
+shared_context "with a fake Git" do
+  let(:git) { stub() }
+  subject   { Kumade::Packager.new(git) }
+end
+
+describe Kumade::Packager, "#run" do
+  include_context "with a fake Git"
+
+  before do
+    subject.stubs(:package_with_jammit)
+  end
+
+  context "with Jammit installed" do
+    it "calls package_with_jammit" do
+      subject.run
+
+      subject.should have_received(:package_with_jammit)
+    end
+  end
+
+  context "with Jammit not installed" do
+    before { subject.stubs(:jammit_installed? => false) }
+
+    it "does not call package_with_jammit" do
+      subject.run
+
+      subject.should_not have_received(:package_with_jammit)
+    end
+  end
+
+  context "with More installed" do
+    before do
+      subject.stubs(:jammit_installed? => false)
+      subject.stubs(:more_installed? => true)
+      subject.stubs(:package_with_more)
+    end
+
+    it "calls package_with_more" do
+      subject.run
+
+      subject.should have_received(:package_with_more)
+    end
+  end
+
+  context "with More not installed" do
+    before do
+      subject.stubs(:jammit_installed? => false)
+      subject.stubs(:more_installed? => false)
+    end
+
+    it "does not call package_with_more" do
+      subject.run
+
+      subject.should_not have_received(:package_with_more)
+    end
+  end
+
+  context "with custom rake task installed" do
+    before do
+      subject.stubs(:jammit_installed?  => false,
+                    :more_installed?    => false,
+                    :invoke_custom_task => nil,
+                    :custom_task?       => true)
+    end
+
+    it "invokes custom task" do
+      subject.run
+
+      subject.should have_received(:invoke_custom_task)
+    end
+  end
+
+  context "with custom rake task not installed" do
+    before do
+      subject.stubs(:jammit_installed?  => false,
+                    :more_installed?    => false,
+                    :invoke_custom_task => nil,
+                    :custom_task?       => false)
+    end
+
+    it "does not invoke custom task" do
+      subject.run
+
+      subject.should_not have_received(:invoke_custom_task)
+    end
+  end
+end
+
+describe Kumade::Packager, "#invoke_custom_task" do
+  include_context "with a fake Git"
+
+  let(:task) { stub('kumade:before_asset_compilation task', :invoke => nil) }
+
+  before do
+    subject.stubs(:say)
+    Rake::Task.stubs(:[] => task)
+  end
+
+  it "calls deploy task" do
+    Rake::Task.expects(:[]).with("kumade:before_asset_compilation").returns(task)
+
+    subject.invoke_custom_task
+
+    task.should have_received(:invoke)
+  end
+end
+
+describe Kumade::Packager, "#custom_task?" do
+  include_context "with a fake Git"
+
+  before do
+    Rake::Task.clear
+  end
+
+  it "returns true if it task found" do
+    namespace :kumade do
+      task :before_asset_compilation do
+      end
+    end
+
+    subject.custom_task?.should be_true
+  end
+
+  it "returns false if task not found" do
+    subject.custom_task?.should be_false
+  end
+end
+
+describe Kumade::Packager, "#package_with_jammit" do
+  include_context "with a fake Git"
+
+  before do
+    subject.stubs(:git_add_and_commit_all_assets_in)
+    subject.stubs(:success)
+    subject.stubs(:error)
+    Jammit.stubs(:package!)
+  end
+
+  it "calls Jammit.package!" do
+    subject.package_with_jammit
+
+    Jammit.should have_received(:package!).once
+  end
+
+  context "with updated assets" do
+    before { subject.git.stubs(:dirty? => true) }
+
+    it "prints the correct message" do
+      subject.package_with_jammit
+
+      subject.should have_received(:success).with("Packaged assets with Jammit")
+    end
+
+    it "calls git_add_and_commit_all_assets_in" do
+      subject.stubs(:jammit_assets_path => 'jammit-assets')
+      subject.expects(:git_add_and_commit_all_assets_in).
+        with('jammit-assets').
+        returns(true)
+
+      subject.package_with_jammit
+    end
+  end
+
+  it "prints an error if packaging failed" do
+    Jammit.expects(:package!).raises(Jammit::MissingConfiguration.new("random Jammit error"))
+
+    subject.package_with_jammit
+
+    subject.should have_received(:error).with("Error: Jammit::MissingConfiguration: random Jammit error")
+  end
+end
+
+describe Kumade::Packager, "#package_with_more" do
+  include_context "with a fake Git"
+
+  before do
+    subject.stubs(:git_add_and_commit_all_assets_in => true,
+                 :more_assets_path                 => 'assets')
+    subject.stubs(:say)
+  end
+
+  it "calls the more:generate task" do
+    git.expects(:dirty?).returns(true)
+    subject.expects(:run).with("bundle exec rake more:generate")
+    subject.package_with_more
+  end
+
+  context "with changed assets" do
+    before do
+      git.stubs(:dirty? => true)
+    end
+
+    it "prints a success message" do
+      subject.stubs(:run).with("bundle exec rake more:generate")
+      subject.expects(:success).with("Packaged assets with More")
+
+      subject.package_with_more
+    end
+
+    it "calls git_add_and_commit_all_assets_in if assets were added" do
+      subject.stubs(:more_assets_path => 'blerg')
+      subject.stubs(:run).with("bundle exec rake more:generate")
+      subject.expects(:git_add_and_commit_all_assets_in).
+        with('blerg').
+        returns(true)
+
+      subject.package_with_more
+    end
+  end
+
+  context "with no changed assets" do
+    it "prints no message" do
+      subject.stubs(:run).with("bundle exec rake more:generate")
+      subject.stubs(:git => mock(:dirty? => false))
+      subject.expects(:say).never
+
+      subject.package_with_more
+    end
+
+    it "does not call git_add_and_commit_all_more_assets" do
+      subject.stubs(:run).with("bundle exec rake more:generate")
+      subject.stubs(:git => mock(:dirty? => false))
+      subject.expects(:git_add_and_commit_all_assets_in).never
+
+      subject.package_with_more
+    end
+  end
+
+  it "prints an error if packaging failed" do
+    subject.stubs(:run).with("bundle exec rake more:generate").raises("blerg")
+
+    subject.expects(:error).with("Error: RuntimeError: blerg")
+
+    subject.package_with_more
+  end
+end
+
+describe Kumade::Packager, "#git_add_and_commit_all_assets_in" do
+  include_context "with a fake Git"
+
+  it "should call git.add_and_commit_all_in" do
+    git.expects(:add_and_commit_all_in).with("dir", 'deploy', 'Compiled assets', "Added and committed all assets", "couldn't commit assets")
+    subject.git_add_and_commit_all_assets_in("dir")
+  end
+end
+
+describe Kumade::Packager, "#jammit_assets_path" do
+  let(:git)      { stub() }
+  let(:packager) { Kumade::Packager.new(git) }
+
+  before do
+    Jammit.stubs(:package_path).returns('blerg')
+  end
+
+  subject { packager.jammit_assets_path }
+
+  it { should == File.join(Jammit::PUBLIC_ROOT, 'blerg') }
+end
+
+describe Kumade::Packager, "#more_assets_path" do
+  include_context "with a fake Git"
+
+  it "returns the correct asset path" do
+    module ::Less
+      class More
+        def self.destination_path
+          'blerg'
+        end
+      end
+    end
+    subject.more_assets_path.should == 'public/blerg'
+  end
+end
+
+describe Kumade::Packager, "#jammit_installed?" do
+  include_context "with a fake Git"
+
+  it "returns true because it's loaded by the Gemfile" do
+    subject.jammit_installed?.should be_true
+  end
+
+  it "returns false if jammit is not installed" do
+    subject.jammit_installed?.should be_true
+  end
+end
+
+describe Kumade::Packager, "#more_installed?" do
+  include_context "with a fake Git"
+
+  context "when More is not installed" do
+    before do
+      if defined?(Less)
+        Object.send(:remove_const, :Less)
+      end
+    end
+
+    it "returns false" do
+      subject.more_installed?.should be_false
+    end
+  end
+
+  context "when More is installed" do
+    before do
+      module Less
+        class More
+        end
+      end
+    end
+
+    it "returns true" do
+      subject.more_installed?.should be_true
+    end
+  end
+end