shipit-engine 0.5.2 → 0.6.0

data/test/unit/commands_test.rb

@@ -1,11 +1,13 @@
 require 'test_helper'
 
-class CommandsTest < ActiveSupport::TestCase
-  def setup
-    @commands = Commands.new
-  end
+module Shipit
+  class CommandsTest < ActiveSupport::TestCase
+    def setup
+      @commands = Commands.new
+    end
 
-  test 'SHIPIT gets added to the environment variables' do
-    assert_equal '1', @commands.env['SHIPIT']
+    test 'SHIPIT gets added to the environment variables' do
+      assert_equal '1', @commands.env['SHIPIT']
+    end
   end
 end

data/test/unit/csv_serializer_test.rb

@@ -1,36 +1,38 @@
 require 'test_helper'
 
-class CSVSerializerTest < ActiveSupport::TestCase
-  test "blank values are dumped as nil" do
-    assert_dumped nil, ''
-    assert_dumped nil, '  '
-    assert_dumped nil, nil
-    assert_dumped nil, []
-  end
+module Shipit
+  class CSVSerializerTest < ActiveSupport::TestCase
+    test "blank values are dumped as nil" do
+      assert_dumped nil, ''
+      assert_dumped nil, '  '
+      assert_dumped nil, nil
+      assert_dumped nil, []
+    end
 
-  test "blank values are loaded as an empty array" do
-    assert_loaded [], ''
-    assert_loaded [], '  '
-    assert_loaded [], nil
-  end
+    test "blank values are loaded as an empty array" do
+      assert_loaded [], ''
+      assert_loaded [], '  '
+      assert_loaded [], nil
+    end
 
-  test "load split the words by comma" do
-    assert_loaded %w(foo bar), 'foo,bar'
-  end
+    test "load split the words by comma" do
+      assert_loaded %w(foo bar), 'foo,bar'
+    end
 
-  test "dump join the words with a comma" do
-    assert_dumped 'foo,bar', %w(foo bar)
-  end
+    test "dump join the words with a comma" do
+      assert_dumped 'foo,bar', %w(foo bar)
+    end
 
-  private
+    private
 
-  def assert_dumped(expected, object)
-    message = "Expected CSVSerializer.dump(#{object.inspect}) to eq #{expected.inspect}"
-    assert_equal(expected, Shipit::CSVSerializer.dump(object), message)
-  end
+    def assert_dumped(expected, object)
+      message = "Expected CSVSerializer.dump(#{object.inspect}) to eq #{expected.inspect}"
+      assert_equal(expected, Shipit::CSVSerializer.dump(object), message)
+    end
 
-  def assert_loaded(expected, payload)
-    message = "Expected CSVSerializer.load(#{payload.inspect}) to eq #{expected.inspect}"
-    assert_equal(expected, Shipit::CSVSerializer.load(payload), message)
+    def assert_loaded(expected, payload)
+      message = "Expected CSVSerializer.load(#{payload.inspect}) to eq #{expected.inspect}"
+      assert_equal(expected, Shipit::CSVSerializer.load(payload), message)
+    end
   end
 end

data/test/unit/deploy_commands_test.rb

@@ -1,180 +1,183 @@
 require 'test_helper'
 
-class DeployCommandsTest < ActiveSupport::TestCase
-  def setup
-    @stack = stacks(:shipit)
-    @deploy = deploys(:shipit_pending)
-    @commands = DeployCommands.new(@deploy)
-    @deploy_spec = stub(
-      dependencies_steps!: ['bundle install --some-args'],
-      deploy_steps!: ['bundle exec cap $ENVIRONMENT deploy'],
-      rollback_steps!: ['bundle exec cap $ENVIRONMENT deploy:rollback'],
-      machine_env: {'GLOBAL' => '1'},
-      directory: nil,
-    )
-    @commands.stubs(:deploy_spec).returns(@deploy_spec)
-
-    StackCommands.stubs(git_version: Gem::Version.new('1.8.4.3'))
-  end
-
-  test "#fetch calls git fetch if repository cache already exist" do
-    Dir.expects(:exist?).with(@stack.git_path).returns(true)
-    command = @commands.fetch
-    assert_equal %w(git fetch origin --tags master), command.args
-  end
-
-  test "#fetch calls git fetch in git_path directory if repository cache already exist" do
-    Dir.expects(:exist?).with(@stack.git_path).returns(true)
-    command = @commands.fetch
-    assert_equal @stack.git_path, command.chdir
-  end
-
-  test "#fetch calls git clone if repository cache do not exist" do
-    Dir.expects(:exist?).with(@stack.git_path).returns(false)
-    command = @commands.fetch
-    expected = %W(git clone --single-branch --recursive --branch master #{@stack.repo_git_url} #{@stack.git_path})
-    assert_equal expected, command.args
-  end
-
-  test "#fetch does not use --single-branch if git is outdated" do
-    Dir.expects(:exist?).with(@stack.git_path).returns(false)
-    StackCommands.stubs(git_version: Gem::Version.new('1.7.2.30'))
-    command = @commands.fetch
-    expected = %W(git clone --recursive --branch master #{@stack.repo_git_url} #{@stack.git_path})
-    assert_equal expected, command.args
-  end
-
-  test "#fetch calls git fetch in base_path directory if repository cache do not exist" do
-    Dir.expects(:exist?).with(@stack.git_path).returns(false)
-    command = @commands.fetch
-    assert_equal @stack.deploys_path, command.chdir
-  end
-
-  test "#fetch merges Shipit.env in ENVIRONMENT" do
-    Shipit.stubs(:env).returns("SPECIFIC_CONFIG" => 5)
-    command = @commands.fetch
-    assert_equal 5, command.env["SPECIFIC_CONFIG"]
-  end
-
-  test "#clone clones the repository cache into the working directory" do
-    command = @commands.clone
-    assert_equal ['git', 'clone', '--local', @stack.git_path, @deploy.working_directory], command.args
-  end
-
-  test "#clone clones the repository cache from the deploys_path" do
-    command = @commands.clone
-    assert_equal @stack.deploys_path, command.chdir
-  end
-
-  test "#checkout checks out the deployed commit" do
-    command = @commands.checkout(@deploy.until_commit)
-    assert_equal ['git', 'checkout', @deploy.until_commit.sha], command.args
-  end
-
-  test "#checkout checks out the deployed commit from the working directory" do
-    command = @commands.checkout(@deploy.until_commit)
-    assert_equal @deploy.working_directory, command.chdir
-  end
-
-  test "#perform calls cap $environment deploy" do
-    commands = @commands.perform
-    assert_equal 1, commands.length
-    command = commands.first
-    assert_equal ['bundle exec cap $ENVIRONMENT deploy'], command.args
-  end
-
-  test "#perform calls cap $environment deploy:rollback for a rollback of a capistrano stack" do
-    @rollback = @deploy.build_rollback
-    @rollback.save!
-    @commands = Commands.for(@rollback)
-    @commands.stubs(:deploy_spec).returns(@deploy_spec)
-
-    steps = @commands.perform
-    assert_equal 1, steps.length
-    step = steps.first
-    assert_equal ['bundle exec cap $ENVIRONMENT deploy:rollback'], step.args
-  end
-
-  test "#perform calls cap $environment deploy from the working_directory" do
-    commands = @commands.perform
-    assert_equal 1, commands.length
-    command = commands.first
-    assert_equal @deploy.working_directory, command.chdir
-  end
-
-  test "the working_directory can be overriten in the spec" do
-    @deploy_spec.stubs(:directory).returns('my_directory')
-    commands = @commands.perform
-    assert_equal 1, commands.length
-    command = commands.first
-    assert_equal File.join(@deploy.working_directory, 'my_directory'), command.chdir
-  end
-
-  test "#perform calls cap $environment deploy with the SHA in the environment" do
-    commands = @commands.perform
-    assert_equal 1, commands.length
-    command = commands.first
-    assert_equal @deploy.until_commit.sha, command.env['SHA']
-  end
-
-  test "#perform calls cap $environment deploy with the SHIPIT_LINK in the environment" do
-    commands = @commands.perform
-    assert_equal 1, commands.length
-    command = commands.first
-    assert_equal "http://shipit.com/shopify/shipit-engine/production/deploys/#{@deploy.id}", command.env['SHIPIT_LINK']
-  end
-
-  test "#perform calls cap $environment deploy with the LAST_DEPLOYED_SHA in the environment" do
-    commands = @commands.perform
-    assert_equal 1, commands.length
-    command = commands.first
-    assert_equal @deploy.stack.last_deployed_commit.sha, command.env['LAST_DEPLOYED_SHA']
-  end
-
-  test "#perform transliterates the user name" do
-    @deploy.user = User.new(login: 'Sirupsen', name: "Simon Hørup Eskildsen")
-    commands = @commands.perform
-    assert_equal 1, commands.length
-    command = commands.first
-    assert_equal "Sirupsen (Simon Horup Eskildsen) via Shipit", command.env['USER']
-  end
-
-  test "#perform calls cap $environment deploy with the ENVIRONMENT in the environment" do
-    commands = @commands.perform
-    assert_equal 1, commands.length
-    command = commands.first
-    assert_equal @stack.environment, command.env['ENVIRONMENT']
-  end
-
-  test "#perform merges Shipit.env in ENVIRONMENT" do
-    Shipit.stubs(:env).returns("SPECIFIC_CONFIG" => 5)
-    assert_equal 5, @commands.env["SPECIFIC_CONFIG"]
-  end
-
-  test "#perform merges shipit.yml machine_env in ENVIRONMENT" do
-    assert_equal '1', @commands.env['GLOBAL']
-  end
-
-  test "#install_dependencies calls bundle install" do
-    commands = @commands.install_dependencies
-    assert_equal 1, commands.length
-    assert_equal ['bundle install --some-args'], commands.first.args
-  end
-
-  test "#install_dependencies merges Shipit.env in ENVIRONMENT" do
-    Shipit.stubs(:env).returns("SPECIFIC_CONFIG" => 5)
-    command = @commands.install_dependencies.first
-    assert_equal 5, command.env["SPECIFIC_CONFIG"]
-  end
-
-  test "#install_dependencies merges machine_env in ENVIRONMENT" do
-    command = @commands.install_dependencies.first
-    assert_equal '1', command.env['GLOBAL']
-  end
-
-  test "the deploy's `env` is merged in ENVIRONMENT" do
-    @deploy.env = {'FOO' => 'BAR'}
-    command = @commands.install_dependencies.first
-    assert_equal 'BAR', command.env['FOO']
+module Shipit
+  class DeployCommandsTest < ActiveSupport::TestCase
+    def setup
+      @stack = shipit_stacks(:shipit)
+      @deploy = shipit_deploys(:shipit_pending)
+      @commands = DeployCommands.new(@deploy)
+      @deploy_spec = stub(
+        dependencies_steps!: ['bundle install --some-args'],
+        deploy_steps!: ['bundle exec cap $ENVIRONMENT deploy'],
+        rollback_steps!: ['bundle exec cap $ENVIRONMENT deploy:rollback'],
+        machine_env: {'GLOBAL' => '1'},
+        directory: nil,
+      )
+      @commands.stubs(:deploy_spec).returns(@deploy_spec)
+
+      StackCommands.stubs(git_version: Gem::Version.new('1.8.4.3'))
+    end
+
+    test "#fetch calls git fetch if repository cache already exist" do
+      Dir.expects(:exist?).with(@stack.git_path).returns(true)
+      command = @commands.fetch
+      assert_equal %w(git fetch origin --tags master), command.args
+    end
+
+    test "#fetch calls git fetch in git_path directory if repository cache already exist" do
+      Dir.expects(:exist?).with(@stack.git_path).returns(true)
+      command = @commands.fetch
+      assert_equal @stack.git_path, command.chdir
+    end
+
+    test "#fetch calls git clone if repository cache do not exist" do
+      Dir.expects(:exist?).with(@stack.git_path).returns(false)
+      command = @commands.fetch
+      expected = %W(git clone --single-branch --recursive --branch master #{@stack.repo_git_url} #{@stack.git_path})
+      assert_equal expected, command.args
+    end
+
+    test "#fetch does not use --single-branch if git is outdated" do
+      Dir.expects(:exist?).with(@stack.git_path).returns(false)
+      StackCommands.stubs(git_version: Gem::Version.new('1.7.2.30'))
+      command = @commands.fetch
+      expected = %W(git clone --recursive --branch master #{@stack.repo_git_url} #{@stack.git_path})
+      assert_equal expected, command.args
+    end
+
+    test "#fetch calls git fetch in base_path directory if repository cache do not exist" do
+      Dir.expects(:exist?).with(@stack.git_path).returns(false)
+      command = @commands.fetch
+      assert_equal @stack.deploys_path, command.chdir
+    end
+
+    test "#fetch merges Shipit.env in ENVIRONMENT" do
+      Shipit.stubs(:env).returns("SPECIFIC_CONFIG" => 5)
+      command = @commands.fetch
+      assert_equal 5, command.env["SPECIFIC_CONFIG"]
+    end
+
+    test "#clone clones the repository cache into the working directory" do
+      command = @commands.clone
+      assert_equal ['git', 'clone', '--local', @stack.git_path, @deploy.working_directory], command.args
+    end
+
+    test "#clone clones the repository cache from the deploys_path" do
+      command = @commands.clone
+      assert_equal @stack.deploys_path, command.chdir
+    end
+
+    test "#checkout checks out the deployed commit" do
+      command = @commands.checkout(@deploy.until_commit)
+      assert_equal ['git', 'checkout', @deploy.until_commit.sha], command.args
+    end
+
+    test "#checkout checks out the deployed commit from the working directory" do
+      command = @commands.checkout(@deploy.until_commit)
+      assert_equal @deploy.working_directory, command.chdir
+    end
+
+    test "#perform calls cap $environment deploy" do
+      commands = @commands.perform
+      assert_equal 1, commands.length
+      command = commands.first
+      assert_equal ['bundle exec cap $ENVIRONMENT deploy'], command.args
+    end
+
+    test "#perform calls cap $environment deploy:rollback for a rollback of a capistrano stack" do
+      @rollback = @deploy.build_rollback
+      @rollback.save!
+      @commands = Commands.for(@rollback)
+      @commands.stubs(:deploy_spec).returns(@deploy_spec)
+
+      steps = @commands.perform
+      assert_equal 1, steps.length
+      step = steps.first
+      assert_equal ['bundle exec cap $ENVIRONMENT deploy:rollback'], step.args
+    end
+
+    test "#perform calls cap $environment deploy from the working_directory" do
+      commands = @commands.perform
+      assert_equal 1, commands.length
+      command = commands.first
+      assert_equal @deploy.working_directory, command.chdir
+    end
+
+    test "the working_directory can be overriten in the spec" do
+      @deploy_spec.stubs(:directory).returns('my_directory')
+      commands = @commands.perform
+      assert_equal 1, commands.length
+      command = commands.first
+      assert_equal File.join(@deploy.working_directory, 'my_directory'), command.chdir
+    end
+
+    test "#perform calls cap $environment deploy with the SHA in the environment" do
+      commands = @commands.perform
+      assert_equal 1, commands.length
+      command = commands.first
+      assert_equal @deploy.until_commit.sha, command.env['SHA']
+    end
+
+    test "#perform calls cap $environment deploy with the SHIPIT_LINK in the environment" do
+      commands = @commands.perform
+      assert_equal 1, commands.length
+      command = commands.first
+      url = "http://shipit.com/shopify/shipit-engine/production/deploys/#{@deploy.id}"
+      assert_equal url, command.env['SHIPIT_LINK']
+    end
+
+    test "#perform calls cap $environment deploy with the LAST_DEPLOYED_SHA in the environment" do
+      commands = @commands.perform
+      assert_equal 1, commands.length
+      command = commands.first
+      assert_equal @deploy.stack.last_deployed_commit.sha, command.env['LAST_DEPLOYED_SHA']
+    end
+
+    test "#perform transliterates the user name" do
+      @deploy.user = User.new(login: 'Sirupsen', name: "Simon Hørup Eskildsen")
+      commands = @commands.perform
+      assert_equal 1, commands.length
+      command = commands.first
+      assert_equal "Sirupsen (Simon Horup Eskildsen) via Shipit", command.env['USER']
+    end
+
+    test "#perform calls cap $environment deploy with the ENVIRONMENT in the environment" do
+      commands = @commands.perform
+      assert_equal 1, commands.length
+      command = commands.first
+      assert_equal @stack.environment, command.env['ENVIRONMENT']
+    end
+
+    test "#perform merges Shipit.env in ENVIRONMENT" do
+      Shipit.stubs(:env).returns("SPECIFIC_CONFIG" => 5)
+      assert_equal 5, @commands.env["SPECIFIC_CONFIG"]
+    end
+
+    test "#perform merges shipit.yml machine_env in ENVIRONMENT" do
+      assert_equal '1', @commands.env['GLOBAL']
+    end
+
+    test "#install_dependencies calls bundle install" do
+      commands = @commands.install_dependencies
+      assert_equal 1, commands.length
+      assert_equal ['bundle install --some-args'], commands.first.args
+    end
+
+    test "#install_dependencies merges Shipit.env in ENVIRONMENT" do
+      Shipit.stubs(:env).returns("SPECIFIC_CONFIG" => 5)
+      command = @commands.install_dependencies.first
+      assert_equal 5, command.env["SPECIFIC_CONFIG"]
+    end
+
+    test "#install_dependencies merges machine_env in ENVIRONMENT" do
+      command = @commands.install_dependencies.first
+      assert_equal '1', command.env['GLOBAL']
+    end
+
+    test "the deploy's `env` is merged in ENVIRONMENT" do
+      @deploy.env = {'FOO' => 'BAR'}
+      command = @commands.install_dependencies.first
+      assert_equal 'BAR', command.env['FOO']
+    end
   end
 end

data/test/unit/deploy_spec_test.rb

@@ -1,288 +1,290 @@
 require 'test_helper'
 
-class DeploySpecTest < ActiveSupport::TestCase
-  setup do
-    @spec = DeploySpec::FileSystem.new('/tmp/', 'env')
-    @spec.stubs(:load_config).returns({})
-  end
+module Shipit
+  class DeploySpecTest < ActiveSupport::TestCase
+    setup do
+      @spec = DeploySpec::FileSystem.new('/tmp/', 'env')
+      @spec.stubs(:load_config).returns({})
+    end
 
-  test '#supports_fetch_deployed_revision? returns false by default' do
-    refute @spec.supports_fetch_deployed_revision?
-  end
+    test '#supports_fetch_deployed_revision? returns false by default' do
+      refute @spec.supports_fetch_deployed_revision?
+    end
 
-  test '#supports_fetch_deployed_revision? returns true if steps are defined' do
-    @spec.stubs(:load_config).returns('fetch' => ['curl --silent https://example.com/status/version'])
-    assert @spec.supports_fetch_deployed_revision?
-  end
+    test '#supports_fetch_deployed_revision? returns true if steps are defined' do
+      @spec.stubs(:load_config).returns('fetch' => ['curl --silent https://example.com/status/version'])
+      assert @spec.supports_fetch_deployed_revision?
+    end
 
-  test '#supports_rollback? returns false by default' do
-    refute @spec.supports_rollback?
-  end
+    test '#supports_rollback? returns false by default' do
+      refute @spec.supports_rollback?
+    end
 
-  test '#supports_rollback? returns true if steps are defined' do
-    @spec.stubs(:load_config).returns('rollback' => {'override' => ['rm -rf /usr /lib/nvidia-current/xorg/xorg']})
-    assert @spec.supports_rollback?
-  end
+    test '#supports_rollback? returns true if steps are defined' do
+      @spec.stubs(:load_config).returns('rollback' => {'override' => ['rm -rf /usr /lib/nvidia-current/xorg/xorg']})
+      assert @spec.supports_rollback?
+    end
 
-  test '#supports_rollback? returns true if stack is detected as capistrano' do
-    @spec.expects(:capistrano?).returns(true)
-    assert @spec.supports_rollback?
-  end
+    test '#supports_rollback? returns true if stack is detected as capistrano' do
+      @spec.expects(:capistrano?).returns(true)
+      assert @spec.supports_rollback?
+    end
 
-  test '#dependencies_steps returns `dependencies.override` if present' do
-    @spec.stubs(:load_config).returns('dependencies' => {'override' => %w(foo bar baz)})
-    assert_equal %w(foo bar baz), @spec.dependencies_steps
-  end
+    test '#dependencies_steps returns `dependencies.override` if present' do
+      @spec.stubs(:load_config).returns('dependencies' => {'override' => %w(foo bar baz)})
+      assert_equal %w(foo bar baz), @spec.dependencies_steps
+    end
 
-  test '#dependencies_steps returns `bundle install` if a `Gemfile` is present' do
-    @spec.expects(:bundler?).returns(true)
-    @spec.expects(:bundle_install).returns(:bundle_install)
-    assert_equal :bundle_install, @spec.dependencies_steps
-  end
+    test '#dependencies_steps returns `bundle install` if a `Gemfile` is present' do
+      @spec.expects(:bundler?).returns(true)
+      @spec.expects(:bundle_install).returns(:bundle_install)
+      assert_equal :bundle_install, @spec.dependencies_steps
+    end
 
-  test '#fetch_deployed_revision_steps! is unknown by default' do
-    assert_raises DeploySpec::Error do
-      @spec.fetch_deployed_revision_steps!
+    test '#fetch_deployed_revision_steps! is unknown by default' do
+      assert_raises DeploySpec::Error do
+        @spec.fetch_deployed_revision_steps!
+      end
     end
-  end
 
-  test '#fetch_deployed_revision_steps use `fetch` is present' do
-    @spec.stubs(:load_config).returns('fetch' => ['echo l33t'])
-    assert_equal ['echo l33t'], @spec.fetch_deployed_revision_steps
-  end
+    test '#fetch_deployed_revision_steps use `fetch` is present' do
+      @spec.stubs(:load_config).returns('fetch' => ['echo l33t'])
+      assert_equal ['echo l33t'], @spec.fetch_deployed_revision_steps
+    end
 
-  test '#bundle_install return a sane default bundle install command' do
-    @spec.stubs(:gemfile_lock_exists?).returns(true)
-    command = %(
-      bundle check --path=#{DeploySpec.bundle_path} ||
-      bundle install
-      --frozen
-      --path=#{DeploySpec.bundle_path}
-      --retry=2
-      --without=default:production:development:test:staging:benchmark:debug
-    ).gsub(/\s+/, ' ').strip
-    assert_equal command, @spec.bundle_install.last
-  end
+    test '#bundle_install return a sane default bundle install command' do
+      @spec.stubs(:gemfile_lock_exists?).returns(true)
+      command = %(
+        bundle check --path=#{DeploySpec.bundle_path} ||
+        bundle install
+        --frozen
+        --path=#{DeploySpec.bundle_path}
+        --retry=2
+        --without=default:production:development:test:staging:benchmark:debug
+      ).gsub(/\s+/, ' ').strip
+      assert_equal command, @spec.bundle_install.last
+    end
 
-  test '#bundle_install use `dependencies.bundler.without` if present to build the --without argument' do
-    @spec.stubs(:gemfile_lock_exists?).returns(true)
-    @spec.stubs(:load_config).returns('dependencies' => {'bundler' => {'without' => %w(some custom groups)}})
-    command = %(
-      bundle check --path=#{DeploySpec.bundle_path} ||
-      bundle install
-      --frozen
-      --path=#{DeploySpec.bundle_path}
-      --retry=2
-      --without=some:custom:groups
-    ).gsub(/\s+/, ' ').strip
-    assert_equal command, @spec.bundle_install.last
-  end
+    test '#bundle_install use `dependencies.bundler.without` if present to build the --without argument' do
+      @spec.stubs(:gemfile_lock_exists?).returns(true)
+      @spec.stubs(:load_config).returns('dependencies' => {'bundler' => {'without' => %w(some custom groups)}})
+      command = %(
+        bundle check --path=#{DeploySpec.bundle_path} ||
+        bundle install
+        --frozen
+        --path=#{DeploySpec.bundle_path}
+        --retry=2
+        --without=some:custom:groups
+      ).gsub(/\s+/, ' ').strip
+      assert_equal command, @spec.bundle_install.last
+    end
 
-  test '#bundle_install has --frozen option if Gemfile.lock is present' do
-    @spec.stubs(:load_config).returns('dependencies' => {'bundler' => {'without' => %w(some custom groups)}})
-    @spec.stubs(:gemfile_lock_exists?).returns(true)
-    assert @spec.bundle_install.last.include?('--frozen')
-  end
+    test '#bundle_install has --frozen option if Gemfile.lock is present' do
+      @spec.stubs(:load_config).returns('dependencies' => {'bundler' => {'without' => %w(some custom groups)}})
+      @spec.stubs(:gemfile_lock_exists?).returns(true)
+      assert @spec.bundle_install.last.include?('--frozen')
+    end
 
-  test '#bundle_install does not have --frozen option if Gemfile.lock is not present' do
-    @spec.stubs(:load_config).returns('dependencies' => {'bundler' => {'without' => %w(some custom groups)}})
-    @spec.stubs(:gemfile_lock_exists?).returns(false)
-    refute @spec.bundle_install.last.include?('--frozen')
-  end
+    test '#bundle_install does not have --frozen option if Gemfile.lock is not present' do
+      @spec.stubs(:load_config).returns('dependencies' => {'bundler' => {'without' => %w(some custom groups)}})
+      @spec.stubs(:gemfile_lock_exists?).returns(false)
+      refute @spec.bundle_install.last.include?('--frozen')
+    end
 
-  test '#bundle_install does not have --frozen if overridden in shipit.yml' do
-    @spec.stubs(:load_config).returns('dependencies' => {'bundler' => {'frozen' => false}})
-    @spec.stubs(:gemfile_lock_exists?).returns(true)
-    refute @spec.bundle_install.last.include?('--frozen')
-  end
+    test '#bundle_install does not have --frozen if overridden in shipit.yml' do
+      @spec.stubs(:load_config).returns('dependencies' => {'bundler' => {'frozen' => false}})
+      @spec.stubs(:gemfile_lock_exists?).returns(true)
+      refute @spec.bundle_install.last.include?('--frozen')
+    end
 
-  test '#deploy_steps returns `deploy.override` if present' do
-    @spec.stubs(:load_config).returns('deploy' => {'override' => %w(foo bar baz)})
-    assert_equal %w(foo bar baz), @spec.deploy_steps
-  end
+    test '#deploy_steps returns `deploy.override` if present' do
+      @spec.stubs(:load_config).returns('deploy' => {'override' => %w(foo bar baz)})
+      assert_equal %w(foo bar baz), @spec.deploy_steps
+    end
 
-  test '#deploy_steps returns `cap $ENVIRONMENT deploy` if a `Capfile` is present' do
-    @spec.expects(:bundler?).returns(true)
-    @spec.expects(:capistrano?).returns(true)
-    assert_equal ['bundle exec cap $ENVIRONMENT deploy'], @spec.deploy_steps
-  end
+    test '#deploy_steps returns `cap $ENVIRONMENT deploy` if a `Capfile` is present' do
+      @spec.expects(:bundler?).returns(true)
+      @spec.expects(:capistrano?).returns(true)
+      assert_equal ['bundle exec cap $ENVIRONMENT deploy'], @spec.deploy_steps
+    end
 
-  test '#deploy_steps raise a DeploySpec::Error! if it dont know how to deploy the app' do
-    @spec.expects(:capistrano?).returns(false)
-    assert_raise DeploySpec::Error do
-      @spec.deploy_steps!
+    test '#deploy_steps raise a DeploySpec::Error! if it dont know how to deploy the app' do
+      @spec.expects(:capistrano?).returns(false)
+      assert_raise DeploySpec::Error do
+        @spec.deploy_steps!
+      end
     end
-  end
 
-  test '#rollback_steps returns `rollback.override` if present' do
-    @spec.stubs(:load_config).returns('rollback' => {'override' => %w(foo bar baz)})
-    assert_equal %w(foo bar baz), @spec.rollback_steps
-  end
+    test '#rollback_steps returns `rollback.override` if present' do
+      @spec.stubs(:load_config).returns('rollback' => {'override' => %w(foo bar baz)})
+      assert_equal %w(foo bar baz), @spec.rollback_steps
+    end
 
-  test '#rollback_steps returns `cap $ENVIRONMENT deploy:rollback` if a `Capfile` is present' do
-    @spec.expects(:bundler?).returns(true)
-    @spec.expects(:capistrano?).returns(true)
-    assert_equal ['bundle exec cap $ENVIRONMENT deploy:rollback'], @spec.rollback_steps
-  end
+    test '#rollback_steps returns `cap $ENVIRONMENT deploy:rollback` if a `Capfile` is present' do
+      @spec.expects(:bundler?).returns(true)
+      @spec.expects(:capistrano?).returns(true)
+      assert_equal ['bundle exec cap $ENVIRONMENT deploy:rollback'], @spec.rollback_steps
+    end
 
-  test '#machine_env return an environment hash' do
-    @spec.stubs(:load_config).returns('machine' => {'environment' => {'GLOBAL' => '1'}})
-    assert_equal({'GLOBAL' => '1'}, @spec.machine_env)
-  end
+    test '#machine_env return an environment hash' do
+      @spec.stubs(:load_config).returns('machine' => {'environment' => {'GLOBAL' => '1'}})
+      assert_equal({'GLOBAL' => '1'}, @spec.machine_env)
+    end
 
-  test '#load_config can grab the env-specific shipit.yml file' do
-    config = {}
-    config.expects(:exist?).returns(true)
-    config.expects(:read).returns({'dependencies' => {'override' => %w(foo bar baz)}}.to_yaml)
-    spec = DeploySpec::FileSystem.new('.', 'staging')
-    spec.expects(:file).with('shipit.staging.yml').returns(config)
-    assert_equal %w(foo bar baz), spec.dependencies_steps
-  end
+    test '#load_config can grab the env-specific shipit.yml file' do
+      config = {}
+      config.expects(:exist?).returns(true)
+      config.expects(:read).returns({'dependencies' => {'override' => %w(foo bar baz)}}.to_yaml)
+      spec = DeploySpec::FileSystem.new('.', 'staging')
+      spec.expects(:file).with('shipit.staging.yml').returns(config)
+      assert_equal %w(foo bar baz), spec.dependencies_steps
+    end
 
-  test '#load_config grabs the global shipit.yml file if there is no env-specific file' do
-    not_config = {}
-    not_config.expects(:exist?).returns(false)
+    test '#load_config grabs the global shipit.yml file if there is no env-specific file' do
+      not_config = {}
+      not_config.expects(:exist?).returns(false)
 
-    config = {}
-    config.expects(:exist?).returns(true)
-    config.expects(:read).returns({'dependencies' => {'override' => %w(foo bar baz)}}.to_yaml)
+      config = {}
+      config.expects(:exist?).returns(true)
+      config.expects(:read).returns({'dependencies' => {'override' => %w(foo bar baz)}}.to_yaml)
 
-    spec = DeploySpec::FileSystem.new('.', 'staging')
-    spec.expects(:file).with('shipit.staging.yml').returns(not_config)
-    spec.expects(:file).with('shipit.yml').returns(config)
-    assert_equal %w(foo bar baz), spec.dependencies_steps
-  end
+      spec = DeploySpec::FileSystem.new('.', 'staging')
+      spec.expects(:file).with('shipit.staging.yml').returns(not_config)
+      spec.expects(:file).with('shipit.yml').returns(config)
+      assert_equal %w(foo bar baz), spec.dependencies_steps
+    end
 
-  test '#gemspec gives the path of the repo gemspec if present' do
-    spec = DeploySpec::FileSystem.new('foobar/', 'production')
-    Dir.expects(:[]).with('foobar/*.gemspec').returns(['foobar/foobar.gemspec'])
-    assert_equal 'foobar/foobar.gemspec', spec.gemspec
-  end
+    test '#gemspec gives the path of the repo gemspec if present' do
+      spec = DeploySpec::FileSystem.new('foobar/', 'production')
+      Dir.expects(:[]).with('foobar/*.gemspec').returns(['foobar/foobar.gemspec'])
+      assert_equal 'foobar/foobar.gemspec', spec.gemspec
+    end
 
-  test '#gem? is true if a gemspec is present' do
-    @spec.expects(:gemspec).returns('something')
-    assert @spec.gem?
-  end
+    test '#gem? is true if a gemspec is present' do
+      @spec.expects(:gemspec).returns('something')
+      assert @spec.gem?
+    end
 
-  test '#gem? is false if there is no gemspec' do
-    @spec.expects(:gemspec).returns(nil)
-    refute @spec.gem?
-  end
+    test '#gem? is false if there is no gemspec' do
+      @spec.expects(:gemspec).returns(nil)
+      refute @spec.gem?
+    end
 
-  test '#publish_gem first check if version tag have been created, and then invoke bundler release task' do
-    @spec.stubs(:gemspec).returns('/tmp/shipit.gemspec')
-    refute @spec.capistrano?
-    assert_equal ['assert-gem-version-tag /tmp/shipit.gemspec', 'bundle exec rake release'], @spec.deploy_steps
-  end
+    test '#publish_gem first check if version tag have been created, and then invoke bundler release task' do
+      @spec.stubs(:gemspec).returns('/tmp/shipit.gemspec')
+      refute @spec.capistrano?
+      assert_equal ['assert-gem-version-tag /tmp/shipit.gemspec', 'bundle exec rake release'], @spec.deploy_steps
+    end
 
-  test '#setup_dot_py gives the path of the repo setup.py if present' do
-    spec = DeploySpec::FileSystem.new('foobar/', 'production')
-    assert_equal Pathname.new('foobar/setup.py'), spec.setup_dot_py
-  end
+    test '#setup_dot_py gives the path of the repo setup.py if present' do
+      spec = DeploySpec::FileSystem.new('foobar/', 'production')
+      assert_equal Pathname.new('foobar/setup.py'), spec.setup_dot_py
+    end
 
-  test '#egg? is true if a setup.py is present' do
-    @spec.expects(:setup_dot_py).returns(Shipit::Engine.root.join('Gemfile'))
-    assert @spec.egg?
-  end
+    test '#egg? is true if a setup.py is present' do
+      @spec.expects(:setup_dot_py).returns(Shipit::Engine.root.join('Gemfile'))
+      assert @spec.egg?
+    end
 
-  test '#egg? is false if there is no setup.py' do
-    @spec.expects(:setup_dot_py).returns(Shipit::Engine.root.join("tmp-#{SecureRandom.hex}"))
-    refute @spec.egg?
-  end
+    test '#egg? is false if there is no setup.py' do
+      @spec.expects(:setup_dot_py).returns(Shipit::Engine.root.join("tmp-#{SecureRandom.hex}"))
+      refute @spec.egg?
+    end
 
-  test '#publish_egg first check if version tag have been created and then invoke setup.py upload' do
-    file = Pathname.new('/tmp/fake_setup.py')
-    file.write('foo')
-    @spec.stubs(:setup_dot_py).returns(file)
-    steps = ['assert-egg-version-tag /tmp/fake_setup.py', 'python setup.py register sdist upload']
-    assert_equal steps, @spec.deploy_steps
-  end
+    test '#publish_egg first check if version tag have been created and then invoke setup.py upload' do
+      file = Pathname.new('/tmp/fake_setup.py')
+      file.write('foo')
+      @spec.stubs(:setup_dot_py).returns(file)
+      steps = ['assert-egg-version-tag /tmp/fake_setup.py', 'python setup.py register sdist upload']
+      assert_equal steps, @spec.deploy_steps
+    end
 
-  test '#cacheable returns a DeploySpec instance that can be serialized' do
-    assert_instance_of DeploySpec::FileSystem, @spec
-    assert_instance_of DeploySpec, @spec.cacheable
-    config = {
-      'ci' => {'hide' => [], 'allow_failures' => [], 'require' => []},
-      'machine' => {'environment' => {}, 'directory' => nil},
-      'review' => {'checklist' => [], 'monitoring' => [], 'checks' => []},
-      'dependencies' => {'override' => []},
-      'plugins' => {},
-      'deploy' => {'override' => nil, 'variables' => []},
-      'rollback' => {'override' => nil},
-      'fetch' => nil,
-      'tasks' => {},
-    }
-    assert_equal config, @spec.cacheable.config
-  end
+    test '#cacheable returns a DeploySpec instance that can be serialized' do
+      assert_instance_of DeploySpec::FileSystem, @spec
+      assert_instance_of DeploySpec, @spec.cacheable
+      config = {
+        'ci' => {'hide' => [], 'allow_failures' => [], 'require' => []},
+        'machine' => {'environment' => {}, 'directory' => nil},
+        'review' => {'checklist' => [], 'monitoring' => [], 'checks' => []},
+        'dependencies' => {'override' => []},
+        'plugins' => {},
+        'deploy' => {'override' => nil, 'variables' => []},
+        'rollback' => {'override' => nil},
+        'fetch' => nil,
+        'tasks' => {},
+      }
+      assert_equal config, @spec.cacheable.config
+    end
 
-  test "#deploy_variables returns an empty array by default" do
-    assert_equal [], @spec.deploy_variables
-  end
+    test "#deploy_variables returns an empty array by default" do
+      assert_equal [], @spec.deploy_variables
+    end
 
-  test "#deploy_variables returns an array of VariableDefinition instances" do
-    @spec.stubs(:load_config).returns('deploy' => {'variables' => [{
-      'name' => 'SAFETY_DISABLED',
-      'title' => 'Set to 1 to do dangerous things',
-      'default' => 0,
-    }]})
+    test "#deploy_variables returns an array of VariableDefinition instances" do
+      @spec.stubs(:load_config).returns('deploy' => {'variables' => [{
+        'name' => 'SAFETY_DISABLED',
+        'title' => 'Set to 1 to do dangerous things',
+        'default' => 0,
+      }]})
 
-    assert_equal 1, @spec.deploy_variables.size
-    variable_definition = @spec.deploy_variables.first
-    assert_equal 'SAFETY_DISABLED', variable_definition.name
-  end
+      assert_equal 1, @spec.deploy_variables.size
+      variable_definition = @spec.deploy_variables.first
+      assert_equal 'SAFETY_DISABLED', variable_definition.name
+    end
 
-  test "task definitions prepend bundle exec if necessary" do
-    @spec.expects(:load_config).returns('tasks' => {'restart' => {'steps' => %w(foo)}})
-    @spec.expects(:bundler?).returns(true).at_least_once
-    definition = @spec.find_task_definition('restart')
+    test "task definitions prepend bundle exec if necessary" do
+      @spec.expects(:load_config).returns('tasks' => {'restart' => {'steps' => %w(foo)}})
+      @spec.expects(:bundler?).returns(true).at_least_once
+      definition = @spec.find_task_definition('restart')
 
-    assert_equal ['bundle exec foo'], definition.steps
-  end
+      assert_equal ['bundle exec foo'], definition.steps
+    end
 
-  test "task definitions prepend bundle exec before serialization" do
-    @spec.expects(:load_config).returns('tasks' => {'restart' => {'steps' => %w(foo)}})
-    @spec.expects(:bundler?).returns(true).at_least_once
+    test "task definitions prepend bundle exec before serialization" do
+      @spec.expects(:load_config).returns('tasks' => {'restart' => {'steps' => %w(foo)}})
+      @spec.expects(:bundler?).returns(true).at_least_once
 
-    cached_spec = DeploySpec.load(DeploySpec.dump(@spec))
-    definition = cached_spec.find_task_definition('restart')
-    assert_equal ['bundle exec foo'], definition.steps
-  end
+      cached_spec = DeploySpec.load(DeploySpec.dump(@spec))
+      definition = cached_spec.find_task_definition('restart')
+      assert_equal ['bundle exec foo'], definition.steps
+    end
 
-  test "#review_checklist returns an array" do
-    @spec.expects(:load_config).returns('review' => {'checklist' => %w(foo bar)})
-    assert_equal %w(foo bar), @spec.review_checklist
-  end
+    test "#review_checklist returns an array" do
+      @spec.expects(:load_config).returns('review' => {'checklist' => %w(foo bar)})
+      assert_equal %w(foo bar), @spec.review_checklist
+    end
 
-  test "#review_checklist returns an empty array if the section is missing" do
-    assert_equal [], @spec.review_checklist
-  end
+    test "#review_checklist returns an empty array if the section is missing" do
+      assert_equal [], @spec.review_checklist
+    end
 
-  test "#review_monitoring returns an array of hashes" do
-    @spec.expects(:load_config).returns('review' => {'monitoring' => [
-      {'image' => 'http://example.com/foo.png', 'width' => 200, 'height' => 400},
-      {'iframe' => 'http://example.com/', 'width' => 200, 'height' => 400},
-    ]})
-    assert_equal [
-      {'image' => 'http://example.com/foo.png', 'width' => 200, 'height' => 400},
-      {'iframe' => 'http://example.com/', 'width' => 200, 'height' => 400},
-    ], @spec.review_monitoring
-  end
+    test "#review_monitoring returns an array of hashes" do
+      @spec.expects(:load_config).returns('review' => {'monitoring' => [
+        {'image' => 'http://example.com/foo.png', 'width' => 200, 'height' => 400},
+        {'iframe' => 'http://example.com/', 'width' => 200, 'height' => 400},
+      ]})
+      assert_equal [
+        {'image' => 'http://example.com/foo.png', 'width' => 200, 'height' => 400},
+        {'iframe' => 'http://example.com/', 'width' => 200, 'height' => 400},
+      ], @spec.review_monitoring
+    end
 
-  test "#review_monitoring returns an empty array if the section is missing" do
-    assert_equal [], @spec.review_monitoring
-  end
+    test "#review_monitoring returns an empty array if the section is missing" do
+      assert_equal [], @spec.review_monitoring
+    end
 
-  test "#hidden_statuses is empty by default" do
-    assert_equal [], @spec.hidden_statuses
-  end
+    test "#hidden_statuses is empty by default" do
+      assert_equal [], @spec.hidden_statuses
+    end
 
-  test "#hidden_statuses is an array even if the value is a string" do
-    @spec.expects(:load_config).returns('ci' => {'hide' => 'ci/circleci'})
-    assert_equal %w(ci/circleci), @spec.hidden_statuses
-  end
+    test "#hidden_statuses is an array even if the value is a string" do
+      @spec.expects(:load_config).returns('ci' => {'hide' => 'ci/circleci'})
+      assert_equal %w(ci/circleci), @spec.hidden_statuses
+    end
 
-  test "#hidden_statuses is an array even if the value is present" do
-    @spec.expects(:load_config).returns('ci' => {'hide' => %w(ci/circleci ci/jenkins)})
-    assert_equal %w(ci/circleci ci/jenkins), @spec.hidden_statuses
+    test "#hidden_statuses is an array even if the value is present" do
+      @spec.expects(:load_config).returns('ci' => {'hide' => %w(ci/circleci ci/jenkins)})
+      assert_equal %w(ci/circleci ci/jenkins), @spec.hidden_statuses
+    end
   end
 end